Skip to content

Commit

Permalink
Fix nohover pointerevent attributes test
Browse files Browse the repository at this point in the history
Summary:
Changelog: [RNTester][Internal] nohover pointerevent attributes platform test fixes

In implementing full support of nohover pointerevents I discovered a couple issues with my initial port of the web platform test — specifically I forgot to update the calls to `checkPointerEventAttributes` to be "touch" instead of "mouse" and I had forgotten to remove the `pointerMove` event from the expected pointer event order.

Reviewed By: lunaleaps

Differential Revision: D38718994

fbshipit-source-id: d189a4b5cf3042c9f493ac876062f4f60219ae2b
  • Loading branch information
vincentriemer authored and facebook-github-bot committed Aug 16, 2022
1 parent 4dce39d commit 6442543
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ import {check_PointerEvent, useTestEventHandler} from './PointerEventSupport';
const eventList = [
'pointerOver',
'pointerEnter',
'pointerMove',
'pointerDown',
'pointerUp',
'pointerOut',
Expand Down Expand Up @@ -151,7 +150,7 @@ function PointerEventAttributesNoHoverPointersTestCase(
eventType,
{x: pageX, y: pageY, width, height},
'',
'mouse',
'touch',
);
if (
Object.keys(detected_eventTypesRef.current).length ===
Expand All @@ -176,7 +175,7 @@ function PointerEventAttributesNoHoverPointersTestCase(
eventType,
{x: pageX, y: pageY, width, height},
'Inner frame ',
'mouse',
'touch',
);
if (
Object.keys(detected_eventTypesRef.current).length ===
Expand Down

0 comments on commit 6442543

Please sign in to comment.