Temporarily use fireEvent on div elements

It looks like there's a recent addition to user-event that checks the pointer events of elements. It looks like no pointer events are detected on divs (but do get detected when pointer-events: auto is explicitly applied). Further investigation needed to see if user-event introduced a bug See: https://github.com/testing-library/user-event/blob/7143ae30353e0cc7f8f12e405763eaf2257ef03e/src/utils/misc/hasPointerEvents.ts#L3
zendeskgarden · Mar 31, 2021 · 5511916 · 5511916
1 parent f3bd095
commit 5511916
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/packages/datepickers/src/elements/Datepicker/Datepicker.spec.tsx b/packages/datepickers/src/elements/Datepicker/Datepicker.spec.tsx
@@ -105,7 +105,7 @@ describe('Datepicker', () => {
       const { getByTestId } = render(<Example value={DEFAULT_DATE} />);
 
       userEvent.click(getByTestId('input'));
-      userEvent.click(getByTestId('previous-month'));
+      fireEvent.click(getByTestId('previous-month'));
 
       expect(getByTestId('month-display')).toHaveTextContent('January 2019');
     });
@@ -114,7 +114,7 @@ describe('Datepicker', () => {
       const { getByTestId } = render(<Example value={DEFAULT_DATE} />);
 
       userEvent.click(getByTestId('input'));
-      userEvent.click(getByTestId('next-month'));
+      fireEvent.click(getByTestId('next-month'));
 
       expect(getByTestId('month-display')).toHaveTextContent('March 2019');
     });
@@ -135,7 +135,7 @@ describe('Datepicker', () => {
       );
 
       userEvent.click(getByTestId('input'));
-      userEvent.click(getAllByTestId('day')[1]);
+      fireEvent.click(getAllByTestId('day')[1]);
 
       expect(onChangeSpy).toHaveBeenCalledWith(new Date(2019, 0, 28));
     });
@@ -148,7 +148,7 @@ describe('Datepicker', () => {
       const input = getByTestId('input');
 
       userEvent.click(input);
-      userEvent.click(getAllByTestId('day')[1]);
+      fireEvent.click(getAllByTestId('day')[1]);
 
       expect(input).toHaveValue('January 28, 2019');
     });

diff --git a/packages/dropdowns/src/elements/Dropdown/Dropdown.spec.tsx b/packages/dropdowns/src/elements/Dropdown/Dropdown.spec.tsx
@@ -172,7 +172,7 @@ describe('Dropdown', () => {
       );
 
       userEvent.click(getByTestId('trigger'));
-      userEvent.click(getAllByTestId('item')[0]);
+      fireEvent.click(getAllByTestId('item')[0]);
       expect(onSelectSpy.mock.calls[0][0]).toBe('previous-item');
     });
 
@@ -183,7 +183,7 @@ describe('Dropdown', () => {
       );
 
       userEvent.click(getByTestId('trigger'));
-      userEvent.click(getAllByTestId('item')[0]);
+      fireEvent.click(getAllByTestId('item')[0]);
       expect(onSelectSpy.mock.calls[0][0]).toStrictEqual(['item-1', 'item-2', 'previous-item']);
     });
 
@@ -197,7 +197,7 @@ describe('Dropdown', () => {
       );
 
       userEvent.click(getByTestId('trigger'));
-      userEvent.click(getAllByTestId('item')[0]);
+      fireEvent.click(getAllByTestId('item')[0]);
       expect(onSelectSpy.mock.calls[0][0]).toStrictEqual(['item-1', 'item-2']);
     });
 

diff --git a/packages/dropdowns/src/elements/Menu/Items/AddItem.spec.tsx b/packages/dropdowns/src/elements/Menu/Items/AddItem.spec.tsx
@@ -7,7 +7,7 @@
 
 import React from 'react';
 import userEvent from '@testing-library/user-event';
-import { render } from 'garden-test-utils';
+import { render, fireEvent } from 'garden-test-utils';
 import { Dropdown, Trigger, Menu, AddItem } from '../../..';
 
 describe('AddItem', () => {
@@ -28,7 +28,7 @@ describe('AddItem', () => {
     );
 
     userEvent.click(getByTestId('trigger'));
-    userEvent.click(getByTestId('add-item'));
+    fireEvent.click(getByTestId('add-item'));
 
     expect(onSelectSpy.mock.calls[0][0]).toBe('add-item');
   });

diff --git a/packages/dropdowns/src/elements/Menu/Items/MediaItem.spec.tsx b/packages/dropdowns/src/elements/Menu/Items/MediaItem.spec.tsx
@@ -7,7 +7,7 @@
 
 import React from 'react';
 import userEvent from '@testing-library/user-event';
-import { render } from 'garden-test-utils';
+import { render, fireEvent } from 'garden-test-utils';
 import { Dropdown, Trigger, Menu, MediaItem } from '../../..';
 
 describe('MediaItem', () => {
@@ -28,7 +28,7 @@ describe('MediaItem', () => {
     );
 
     userEvent.click(getByTestId('trigger'));
-    userEvent.click(getByTestId('media-item'));
+    fireEvent.click(getByTestId('media-item'));
 
     expect(onSelectSpy.mock.calls[0][0]).toBe('media-item');
   });