flatten testGateReact19

Author: mdjastrzebski

eslint.config.mjs

@@ -50,7 +50,6 @@ export default [
       'react-native-a11y/has-valid-accessibility-ignores-invert-colors': 'off',
       'react-native-a11y/has-valid-accessibility-value': 'off',
       '@typescript-eslint/no-explicit-any': 'off',
-      'jest/no-standalone-expect': ['error', { additionalTestBlockFunctions: ['testGateReact19'] }],
     },
   },
 ];

src/__tests__/react-native-api.test.tsx

@@ -4,9 +4,6 @@ import { FlatList, Image, Modal, ScrollView, Switch, Text, TextInput, View } fro
 import { render, screen } from '..';
 import { mapJsonProps } from '../test-utils/json';
 
-const isReact19 = React.version.startsWith('19.');
-const testGateReact19 = isReact19 ? test : test.skip;
-
 /**
  * Tests in this file are intended to give us an proactive warning that React Native behavior has
  * changed in a way that may impact our code like queries or event handling.
@@ -239,7 +236,7 @@ test('React Native API assumption: <FlatList> renders a single host <ScrollView>
   `);
 });
 
-testGateReact19('React Native API assumption: <Modal> renders a single host element', () => {
+test('React Native API assumption: <Modal> renders a single host element', () => {
   render(
     <Modal testID="test">
       <Text>Modal Content</Text>

src/__tests__/render-hook-async.test.tsx

@@ -5,8 +5,6 @@ import { Text } from 'react-native';
 import { act, renderHookAsync } from '..';
 import { excludeConsoleMessage } from '../test-utils/console';
 
-const testGateReact19 = React.version.startsWith('19.') ? test : test.skip;
-
 // eslint-disable-next-line no-console
 const originalConsoleError = console.error;
 afterEach(() => {
@@ -121,7 +119,7 @@ test('handles multiple state updates in effects', async () => {
   expect(result.current).toEqual({ first: 10, second: 20 });
 });
 
-testGateReact19('handles hook with suspense', async () => {
+test('handles hook with suspense', async () => {
   let resolvePromise: (value: string) => void;
   const promise = new Promise<string>((resolve) => {
     resolvePromise = resolve;
@@ -160,7 +158,7 @@ class ErrorBoundary extends React.Component<
   }
 }
 
-testGateReact19('handles hook suspense with error boundary', async () => {
+test('handles hook suspense with error boundary', async () => {
   const ERROR_MESSAGE = 'Hook Promise Rejected In Test';
   // eslint-disable-next-line no-console
   console.error = excludeConsoleMessage(console.error, ERROR_MESSAGE);

src/__tests__/render-hook.test.tsx

@@ -84,20 +84,3 @@ test('props type is inferred correctly when initial props is explicitly undefine
   rerender(6);
   expect(result.current.param).toBe(6);
 });
-
-/**
- * This test makes sure that calling renderHook does
- * not try to detect host component names in any form.
- * But since there are numerous methods that could trigger that
- * we check the count of renders using React Test Renderers.
- */
-test('does render only once', () => {
-  jest.spyOn(UniversalTestRenderer, 'createRoot');
-
-  renderHook(() => {
-    const [state, setState] = React.useState(1);
-    return [state, setState];
-  });
-
-  expect(UniversalTestRenderer.createRoot).toHaveBeenCalledTimes(1);
-});

src/__tests__/suspense-fake-timers.test.tsx

@@ -6,8 +6,6 @@ import { excludeConsoleMessage } from '../test-utils/console';
 
 jest.useFakeTimers();
 
-const testGateReact19 = React.version.startsWith('19.') ? test : test.skip;
-
 // eslint-disable-next-line no-console
 const originalConsoleError = console.error;
 afterEach(() => {
@@ -22,7 +20,7 @@ function Suspending({ promise, testID }: { promise: Promise<unknown>; testID: st
   return <View testID={testID} />;
 }
 
-testGateReact19('resolves manually-controlled promise', async () => {
+test('resolves manually-controlled promise', async () => {
   let resolvePromise: (value: unknown) => void;
   const promise = new Promise((resolve) => {
     resolvePromise = resolve;
@@ -47,7 +45,7 @@ testGateReact19('resolves manually-controlled promise', async () => {
   expect(screen.queryByText('Loading...')).not.toBeOnTheScreen();
 });
 
-testGateReact19('resolves timer-controlled promise', async () => {
+test('resolves timer-controlled promise', async () => {
   const promise = new Promise((resolve) => {
     setTimeout(() => resolve(null), 100);
   });
@@ -88,7 +86,7 @@ class ErrorBoundary extends React.Component<
   }
 }
 
-testGateReact19('handles promise rejection with error boundary', async () => {
+test('handles promise rejection with error boundary', async () => {
   const ERROR_MESSAGE = 'Promise Rejected In Test';
   // eslint-disable-next-line no-console
   console.error = excludeConsoleMessage(console.error, ERROR_MESSAGE);
@@ -117,7 +115,7 @@ testGateReact19('handles promise rejection with error boundary', async () => {
   expect(screen.queryByTestId('error-content')).not.toBeOnTheScreen();
 });
 
-testGateReact19('handles multiple suspending components', async () => {
+test('handles multiple suspending components', async () => {
   let resolvePromise1: (value: unknown) => void;
   let resolvePromise2: (value: unknown) => void;
 
@@ -154,7 +152,7 @@ testGateReact19('handles multiple suspending components', async () => {
   expect(screen.queryByText('Loading...')).not.toBeOnTheScreen();
 });
 
-testGateReact19('handles multiple suspense boundaries independently', async () => {
+test('handles multiple suspense boundaries independently', async () => {
   let resolvePromise1: (value: unknown) => void;
   let resolvePromise2: (value: unknown) => void;
 

src/__tests__/suspense.test.tsx

@@ -4,8 +4,6 @@ import { Text, View } from 'react-native';
 import { act, renderAsync, screen } from '..';
 import { excludeConsoleMessage } from '../test-utils/console';
 
-const testGateReact19 = React.version.startsWith('19.') ? test : test.skip;
-
 // eslint-disable-next-line no-console
 const originalConsoleError = console.error;
 afterEach(() => {
@@ -20,7 +18,7 @@ function Suspending({ promise, testID }: { promise: Promise<unknown>; testID: st
   return <View testID={testID} />;
 }
 
-testGateReact19('resolves manually-controlled promise', async () => {
+test('resolves manually-controlled promise', async () => {
   let resolvePromise: (value: unknown) => void;
   const promise = new Promise((resolve) => {
     resolvePromise = resolve;
@@ -45,7 +43,7 @@ testGateReact19('resolves manually-controlled promise', async () => {
   expect(screen.queryByText('Loading...')).not.toBeOnTheScreen();
 });
 
-testGateReact19('resolves timer-controlled promise', async () => {
+test('resolves timer-controlled promise', async () => {
   const promise = new Promise((resolve) => {
     setTimeout(() => resolve(null), 100);
   });
@@ -85,7 +83,7 @@ class ErrorBoundary extends React.Component<
   }
 }
 
-testGateReact19('handles promise rejection with error boundary', async () => {
+test('handles promise rejection with error boundary', async () => {
   const ERROR_MESSAGE = 'Promise Rejected In Test';
   // eslint-disable-next-line no-console
   console.error = excludeConsoleMessage(console.error, ERROR_MESSAGE);
@@ -114,7 +112,7 @@ testGateReact19('handles promise rejection with error boundary', async () => {
   expect(screen.queryByTestId('error-content')).not.toBeOnTheScreen();
 });
 
-testGateReact19('handles multiple suspending components', async () => {
+test('handles multiple suspending components', async () => {
   let resolvePromise1: (value: unknown) => void;
   let resolvePromise2: (value: unknown) => void;
 
@@ -151,7 +149,7 @@ testGateReact19('handles multiple suspending components', async () => {
   expect(screen.queryByText('Loading...')).not.toBeOnTheScreen();
 });
 
-testGateReact19('handles multiple suspense boundaries independently', async () => {
+test('handles multiple suspense boundaries independently', async () => {
   let resolvePromise1: (value: unknown) => void;
   let resolvePromise2: (value: unknown) => void;