refactor: remove real timers warning in User Event

Author: mdjastrzebski

src/user-event/press/__tests__/longPress.real-timers.test.tsx

@@ -2,13 +2,11 @@ import React from 'react';
 import { Pressable, Text } from 'react-native';
 import { render, screen } from '../../../pure';
 import { userEvent } from '../..';
-import * as WarnAboutRealTimers from '../../utils/warn-about-real-timers';
 
 describe('userEvent.longPress with real timers', () => {
   beforeEach(() => {
     jest.useRealTimers();
     jest.restoreAllMocks();
-    jest.spyOn(WarnAboutRealTimers, 'warnAboutRealTimersIfNeeded').mockImplementation();
   });
 
   test('calls onLongPress if the delayLongPress is the default one', async () => {
@@ -90,18 +88,3 @@ describe('userEvent.longPress with real timers', () => {
     expect(mockOnLongPress).toHaveBeenCalled();
   });
 });
-
-test('warns about using real timers with userEvent', async () => {
-  jest.restoreAllMocks();
-  const mockConsoleWarn = jest.spyOn(console, 'warn').mockImplementation();
-
-  render(<Pressable testID="pressable" />);
-
-  await userEvent.longPress(screen.getByTestId('pressable'));
-
-  expect(mockConsoleWarn.mock.calls[0][0]).toMatchInlineSnapshot(`
-    "It is recommended to use userEvent with fake timers
-    Some events involve duration so your tests may take a long time to run.
-    For instance calling userEvent.longPress with real timers will take 500 ms."
-  `);
-});

src/user-event/press/__tests__/press.real-timers.test.tsx

@@ -10,13 +10,11 @@ import {
 import { createEventLogger, getEventsNames } from '../../../test-utils';
 import { render, screen } from '../../..';
 import { userEvent } from '../..';
-import * as WarnAboutRealTimers from '../../utils/warn-about-real-timers';
 
 describe('userEvent.press with real timers', () => {
   beforeEach(() => {
     jest.useRealTimers();
     jest.restoreAllMocks();
-    jest.spyOn(WarnAboutRealTimers, 'warnAboutRealTimersIfNeeded').mockImplementation();
   });
 
   test('calls onPressIn, onPress and onPressOut prop of touchable', async () => {
@@ -301,18 +299,3 @@ describe('userEvent.press with real timers', () => {
     expect(mockOnPress).toHaveBeenCalled();
   });
 });
-
-test('warns about using real timers with userEvent', async () => {
-  jest.restoreAllMocks();
-  const mockConsoleWarn = jest.spyOn(console, 'warn').mockImplementation();
-
-  render(<Pressable testID="pressable" />);
-
-  await userEvent.press(screen.getByTestId('pressable'));
-
-  expect(mockConsoleWarn.mock.calls[0][0]).toMatchInlineSnapshot(`
-    "It is recommended to use userEvent with fake timers
-    Some events involve duration so your tests may take a long time to run.
-    For instance calling userEvent.longPress with real timers will take 500 ms."
-  `);
-});

src/user-event/press/press.ts

@@ -6,7 +6,7 @@ import { isPointerEventEnabled } from '../../helpers/pointer-events';
 import { isHostText, isHostTextInput } from '../../helpers/host-component-names';
 import { EventBuilder } from '../event-builder';
 import { UserEventConfig, UserEventInstance } from '../setup';
-import { dispatchEvent, wait, warnAboutRealTimersIfNeeded } from '../utils';
+import { dispatchEvent, wait } from '../utils';
 import { DEFAULT_MIN_PRESS_DURATION } from './constants';
 
 export interface PressOptions {
@@ -69,8 +69,6 @@ const emitPressablePressEvents = async (
   element: ReactTestInstance,
   options: BasePressOptions,
 ) => {
-  warnAboutRealTimersIfNeeded();
-
   await wait(config);
 
   dispatchEvent(element, 'responderGrant', EventBuilder.Common.responderGrant());

src/user-event/utils/__tests__/warn-about-real-timers.test.ts

@@ -2,4 +2,3 @@ export * from './content-size';
 export * from './dispatch-event';
 export * from './text-range';
 export * from './wait';
-export * from './warn-about-real-timers';