From 2e0507694ec04d24045b11c368f0a6f6ddcc9088 Mon Sep 17 00:00:00 2001 From: Noah Lemen Date: Wed, 21 Feb 2024 13:31:43 -0500 Subject: [PATCH] remove RTR from ReactCacheOld-test (#28409) ## Summary swaps `react-test-renderer` for `react-noop-rendererer` in ReactCacheOld-test ## How did you test this change? `yarn test-www ReactCacheOld` --- .../__tests__/ReactCacheOld-test.internal.js | 49 +++++++------------ 1 file changed, 18 insertions(+), 31 deletions(-) diff --git a/packages/react-cache/src/__tests__/ReactCacheOld-test.internal.js b/packages/react-cache/src/__tests__/ReactCacheOld-test.internal.js index 2403d2de286b1..f114d30b87201 100644 --- a/packages/react-cache/src/__tests__/ReactCacheOld-test.internal.js +++ b/packages/react-cache/src/__tests__/ReactCacheOld-test.internal.js @@ -13,7 +13,7 @@ let ReactCache; let createResource; let React; let ReactFeatureFlags; -let ReactTestRenderer; +let ReactNoop; let Scheduler; let Suspense; let TextResource; @@ -34,7 +34,7 @@ describe('ReactCache', () => { Suspense = React.Suspense; ReactCache = require('react-cache'); createResource = ReactCache.unstable_createResource; - ReactTestRenderer = require('react-test-renderer'); + ReactNoop = require('react-noop-renderer'); Scheduler = require('scheduler'); const InternalTestUtils = require('internal-test-utils'); @@ -120,9 +120,8 @@ describe('ReactCache', () => { ); } - ReactTestRenderer.create(, { - isConcurrent: true, - }); + const root = ReactNoop.createRoot(); + root.render(); await waitForAll(['Suspend! [Hi]', 'Loading...']); @@ -140,9 +139,8 @@ describe('ReactCache', () => { ); } - const root = ReactTestRenderer.create(, { - isConcurrent: true, - }); + const root = ReactNoop.createRoot(); + root.render(); await waitForAll(['Suspend! [Hi]', 'Loading...']); @@ -157,7 +155,7 @@ describe('ReactCache', () => { assertLog(['Promise rejected [Hi]', 'Error! [Hi]', 'Error! [Hi]']); // Should throw again on a subsequent read - root.update(); + root.render(); await waitForThrow('Failed to load: Hi'); assertLog(['Error! [Hi]', 'Error! [Hi]']); }); @@ -176,13 +174,11 @@ describe('ReactCache', () => { return BadTextResource.read(['Hi', 100]); } - ReactTestRenderer.create( + const root = ReactNoop.createRoot(); + root.render( }> , - { - isConcurrent: true, - }, ); if (__DEV__) { @@ -202,16 +198,14 @@ describe('ReactCache', () => { it('evicts least recently used values', async () => { ReactCache.unstable_setGlobalCacheLimit(3); + const root = ReactNoop.createRoot(); // Render 1, 2, and 3 - const root = ReactTestRenderer.create( + root.render( }> , - { - isConcurrent: true, - }, ); await waitForAll(['Suspend! [1]', 'Loading...']); jest.advanceTimersByTime(100); @@ -228,7 +222,7 @@ describe('ReactCache', () => { expect(root).toMatchRenderedOutput('123'); // Render 1, 4, 5 - root.update( + root.render( }> @@ -255,7 +249,7 @@ describe('ReactCache', () => { // We've now rendered values 1, 2, 3, 4, 5, over our limit of 3. The least // recently used values are 2 and 3. They should have been evicted. - root.update( + root.render( }> @@ -293,13 +287,11 @@ describe('ReactCache', () => { return ; } - const root = ReactTestRenderer.create( + const root = ReactNoop.createRoot(); + root.render( }> , - { - isConcurrent: true, - }, ); await waitForAll(['Loading...']); @@ -351,13 +343,11 @@ describe('ReactCache', () => { } } - const root = ReactTestRenderer.create( + const root = ReactNoop.createRoot(); + root.render( }> , - { - isConcurrent: true, - }, ); await waitForAll(['Suspend! [Hi]', 'Loading...']); @@ -367,13 +357,10 @@ describe('ReactCache', () => { // cached value. resolveThenable('Hi muahahaha I am different'); - root.update( + root.render( }> , - { - isConcurrent: true, - }, ); assertLog([]);