Skip to content

Commit 3e19180

Browse files
committed
Move internalAct to internal-test-utils package
This is not a public API. We only use it for our internal tests, the ones in this repo. Let's move it to this private package. Practically speaking this will also let us use async/await in the implementation.
1 parent d814473 commit 3e19180

File tree

82 files changed

+87
-86
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

82 files changed

+87
-86
lines changed

packages/internal-test-utils/ReactInternalTestUtils.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ import {diff} from 'jest-diff';
1212
import {equals} from '@jest/expect-utils';
1313
import enqueueTask from './enqueueTask';
1414

15+
export {act} from './internalAct';
16+
1517
function assertYieldsWereCleared(Scheduler) {
1618
const actualYields = Scheduler.unstable_clearLog();
1719
if (actualYields.length !== 0) {

packages/internal-test-utils/__tests__/ReactInternalTestUtils-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ const {
2020
waitForThrow,
2121
assertLog,
2222
} = require('internal-test-utils');
23-
const act = require('jest-react').act;
23+
const act = require('internal-test-utils').act;
2424
const Scheduler = require('scheduler/unstable_mock');
2525

2626
describe('ReactInternalTestUtils', () => {

packages/jest-react/src/internalAct.js renamed to packages/internal-test-utils/internalAct.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,7 @@ export function act<T>(scope: () => Thenable<T>): Thenable<T> {
6464
}
6565
};
6666

67-
// TODO: This would be way simpler if we could use async/await. Move this
68-
// function to the internal-test-utils package.
67+
// TODO: This would be way simpler if we used async/await.
6968
try {
7069
const result = scope();
7170
if (

packages/react-client/src/__tests__/ReactFlight-test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ describe('ReactFlight', () => {
3232
ReactNoop = require('react-noop-renderer');
3333
ReactNoopFlightServer = require('react-noop-renderer/flight-server');
3434
ReactNoopFlightClient = require('react-noop-renderer/flight-client');
35-
act = require('jest-react').act;
35+
act = require('internal-test-utils').act;
3636
Scheduler = require('scheduler');
3737
const InternalTestUtils = require('internal-test-utils');
3838
assertLog = InternalTestUtils.assertLog;
@@ -1184,7 +1184,7 @@ describe('ReactFlight', () => {
11841184
ReactNoop = require('react-noop-renderer');
11851185
ReactNoopFlightServer = require('react-noop-renderer/flight-server');
11861186
ReactNoopFlightClient = require('react-noop-renderer/flight-client');
1187-
act = require('jest-react').act;
1187+
act = require('internal-test-utils').act;
11881188
Scheduler = require('scheduler');
11891189

11901190
await act(async () => {

packages/react-debug-tools/src/__tests__/ReactHooksInspectionIntegration-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ describe('ReactHooksInspectionIntegration', () => {
2222
React = require('react');
2323
ReactTestRenderer = require('react-test-renderer');
2424
Scheduler = require('scheduler');
25-
act = require('jest-react').act;
25+
act = require('internal-test-utils').act;
2626
ReactDebugTools = require('react-debug-tools');
2727
});
2828

packages/react-devtools-shared/src/__tests__/inspectedElement-test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,9 @@ describe('InspectedElement', () => {
5555
ReactDOM = require('react-dom');
5656
ReactDOMClient = require('react-dom/client');
5757
PropTypes = require('prop-types');
58-
TestUtilsAct = require('jest-react').act;
58+
TestUtilsAct = require('internal-test-utils').act;
5959
TestRenderer = utils.requireTestRenderer();
60-
TestRendererAct = require('jest-react').act;
60+
TestRendererAct = require('internal-test-utils').act;
6161

6262
BridgeContext =
6363
require('react-devtools-shared/src/devtools/views/context').BridgeContext;

packages/react-devtools-shared/src/__tests__/storeComponentFilters-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ describe('Store component filters', () => {
3434
React = require('react');
3535
Types = require('react-devtools-shared/src/types');
3636
utils = require('./utils');
37-
internalAct = require('jest-react').act;
37+
internalAct = require('internal-test-utils').act;
3838

3939
legacyRender = utils.legacyRender;
4040
});

packages/react-dom/src/__tests__/ReactDOMConsoleErrorReporting-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ describe('ReactDOMConsoleErrorReporting', () => {
2121

2222
beforeEach(() => {
2323
jest.resetModules();
24-
act = require('jest-react').act;
24+
act = require('internal-test-utils').act;
2525
React = require('react');
2626
ReactDOM = require('react-dom');
2727
ReactDOMClient = require('react-dom/client');

packages/react-dom/src/__tests__/ReactDOMFiberAsync-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ describe('ReactDOMFiberAsync', () => {
3131
React = require('react');
3232
ReactDOM = require('react-dom');
3333
ReactDOMClient = require('react-dom/client');
34-
act = require('jest-react').act;
34+
act = require('internal-test-utils').act;
3535
Scheduler = require('scheduler');
3636

3737
const InternalTestUtils = require('internal-test-utils');

packages/react-dom/src/__tests__/ReactDOMFizzShellHydration-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ describe('ReactDOMFizzShellHydration', () => {
3232
React = require('react');
3333
ReactDOMClient = require('react-dom/client');
3434
Scheduler = require('scheduler');
35-
clientAct = require('jest-react').act;
35+
clientAct = require('internal-test-utils').act;
3636
ReactDOMFizzServer = require('react-dom/server');
3737
Stream = require('stream');
3838

0 commit comments

Comments
 (0)