Skip to content

Commit 0441809

Browse files
committed
Call Scheduler.flushAll unconditionally in tests
Instead of wrapping in enableNewScheduler flag.
1 parent 5161ef8 commit 0441809

File tree

1 file changed

+3
-11
lines changed

1 file changed

+3
-11
lines changed

packages/react-reconciler/src/__tests__/ReactHooks-test.internal.js

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ let ReactTestRenderer;
1818
let Scheduler;
1919
let ReactDOMServer;
2020
let act;
21-
let enableNewScheduler;
2221

2322
// Additional tests can be found in ReactHooksWithNoopRenderer. Plan is to
2423
// gradually migrate those to this file.
@@ -33,7 +32,6 @@ describe('ReactHooks', () => {
3332
Scheduler = require('scheduler');
3433
ReactDOMServer = require('react-dom/server');
3534
act = ReactTestRenderer.act;
36-
enableNewScheduler = ReactFeatureFlags.enableNewScheduler;
3735
});
3836

3937
if (__DEV__) {
@@ -1756,9 +1754,7 @@ describe('ReactHooks', () => {
17561754
);
17571755
expect(root).toMatchRenderedOutput('loading');
17581756
await Promise.resolve();
1759-
if (enableNewScheduler) {
1760-
Scheduler.flushAll();
1761-
}
1757+
Scheduler.flushAll();
17621758
expect(root).toMatchRenderedOutput('hello');
17631759
});
17641760

@@ -1790,9 +1786,7 @@ describe('ReactHooks', () => {
17901786
);
17911787
expect(root).toMatchRenderedOutput('loading');
17921788
await Promise.resolve();
1793-
if (enableNewScheduler) {
1794-
Scheduler.flushAll();
1795-
}
1789+
Scheduler.flushAll();
17961790
expect(root).toMatchRenderedOutput('hello');
17971791
});
17981792

@@ -1824,9 +1818,7 @@ describe('ReactHooks', () => {
18241818
);
18251819
expect(root).toMatchRenderedOutput('loading');
18261820
await Promise.resolve();
1827-
if (enableNewScheduler) {
1828-
Scheduler.flushAll();
1829-
}
1821+
Scheduler.flushAll();
18301822
expect(root).toMatchRenderedOutput('hello');
18311823
});
18321824
});

0 commit comments

Comments
 (0)