Fix some test coverage

Author: gethinwebster

src/internal/components/sortable-area/keyboard-sensor/index.ts

@@ -31,7 +31,7 @@ export type KeyboardAndUAPCoordinateGetter = (
 ) => Coordinates | void;
 
 type KeyboardAndUAPSensorOptions = KeyboardSensorOptions & {
-  coordinateGetter?: KeyboardAndUAPCoordinateGetter;
+  coordinateGetter: KeyboardAndUAPCoordinateGetter;
   onActivation?({ event }: { event: KeyboardEvent | MouseEvent }): void;
 };
 
@@ -131,10 +131,6 @@ export class KeyboardAndUAPSensor implements SensorInstance {
       this.referenceCoordinates = currentCoordinates;
     }
 
-    if (!coordinateGetter) {
-      return;
-    }
-
     const newCoordinates = coordinateGetter(event, {
       active,
       context: context.current,

src/list/__tests__/list-sortable.test.tsx

@@ -135,6 +135,17 @@ describe('List - Sortable', () => {
     await expectAnnouncement('Item moved from position 1 to position 2 of 3');
   });
 
+  test('ignores other keys', async () => {
+    const { wrapper } = renderSortableList();
+    const dragHandle = wrapper.findItemByIndex(1)!.findDragHandle()!.getElement();
+    pressKey(dragHandle, 'Space');
+    await expectAnnouncement('Picked up item at position 1 of 3');
+    pressKey(dragHandle, 'D');
+    pressKey(dragHandle, 'Space');
+    expect(wrapper.findItemByIndex(1)!.getElement()).toHaveTextContent('Item 1');
+    expect(wrapper.findItemByIndex(2)!.getElement()).toHaveTextContent('Item 2');
+  });
+
   test('can move an item with UAP buttons', async () => {
     const { wrapper } = renderSortableList();
     const dragHandle = wrapper.findItemByIndex(1)!.findDragHandle()!.getElement();
@@ -157,4 +168,14 @@ describe('List - Sortable', () => {
     expect(wrapper.findItemByIndex(2)!.getElement()).toHaveTextContent('Item 1');
     await expectAnnouncement('Item moved from position 1 to position 2 of 3');
   });
+
+  test('non-primary pointer events do not activate UAP', () => {
+    const { wrapper } = renderSortableList();
+    const dragHandle = wrapper.findItemByIndex(1)!.findDragHandle()!;
+    const dragWrapper = new InternalDragHandleWrapper(document.body);
+    expect(dragWrapper.findVisibleDirectionButtonBlockEnd()).toBeFalsy();
+
+    dragHandle.click({ button: 1 });
+    expect(dragWrapper.findVisibleDirectionButtonBlockEnd()).toBeFalsy();
+  });
 });