Skip to content

Commit f1dc626

Browse files
committed
Call Scheduler.flushAll unconditionally in tests
Instead of wrapping in enableNewScheduler flag.
1 parent be3bec7 commit f1dc626

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__) {
@@ -1749,9 +1747,7 @@ describe('ReactHooks', () => {
17491747
);
17501748
expect(root).toMatchRenderedOutput('loading');
17511749
await Promise.resolve();
1752-
if (enableNewScheduler) {
1753-
Scheduler.flushAll();
1754-
}
1750+
Scheduler.flushAll();
17551751
expect(root).toMatchRenderedOutput('hello');
17561752
});
17571753

@@ -1783,9 +1779,7 @@ describe('ReactHooks', () => {
17831779
);
17841780
expect(root).toMatchRenderedOutput('loading');
17851781
await Promise.resolve();
1786-
if (enableNewScheduler) {
1787-
Scheduler.flushAll();
1788-
}
1782+
Scheduler.flushAll();
17891783
expect(root).toMatchRenderedOutput('hello');
17901784
});
17911785

@@ -1817,9 +1811,7 @@ describe('ReactHooks', () => {
18171811
);
18181812
expect(root).toMatchRenderedOutput('loading');
18191813
await Promise.resolve();
1820-
if (enableNewScheduler) {
1821-
Scheduler.flushAll();
1822-
}
1814+
Scheduler.flushAll();
18231815
expect(root).toMatchRenderedOutput('hello');
18241816
});
18251817
});

0 commit comments

Comments
 (0)