fix(pointer): check PointerCoords.x in isDifferentPointerPosition (…

…#1216)
testing-library · Jan 15, 2025 · 75edef5 · 75edef5
1 parent 8528972
commit 75edef5
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/src/system/pointer/shared.ts b/src/system/pointer/shared.ts
@@ -27,7 +27,7 @@ export function isDifferentPointerPosition(
 ) {
   return (
     positionA.target !== positionB.target ||
-    positionA.coords?.x !== positionB.coords?.y ||
+    positionA.coords?.x !== positionB.coords?.x ||
     positionA.coords?.y !== positionB.coords?.y ||
     positionA.caret?.node !== positionB.caret?.node ||
     positionA.caret?.offset !== positionB.caret?.offset

diff --git a/tests/pointer/move.ts b/tests/pointer/move.ts
@@ -129,3 +129,24 @@ test('declare pointer coordinates', async () => {
     expect(getEvents('mouseover')[0]).toHaveProperty(prop, value)
   })
 })
+
+test('move pointer by x/y coords', async () => {
+  const {elements, getEventSnapshot, user} = setup('<div></div>')
+
+  await user.pointer([
+    {keys: '[MouseLeft>]', target: elements[0], coords: {x: 20, y: 20}},
+    {coords: {x: 40, y: 20}},
+    {coords: {x: 40, y: 40}},
+  ])
+
+  expect(getEventSnapshot()).toMatchInlineSnapshot(`
+    Events fired on: div
+
+    div - pointerdown
+    div - mousedown: primary
+    div - pointermove
+    div - mousemove
+    div - pointermove
+    div - mousemove
+  `)
+})