[tests] add assertLog for legacy mode tests (facebook#28814)

A few more tests for facebook#28737
EdisonVan · Apr 15, 2024 · 9cd2d04 · 9cd2d04
1 parent 68ec6df
commit 9cd2d04
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/packages/react-dom/src/__tests__/ReactDOMSuspensePlaceholder-test.js b/packages/react-dom/src/__tests__/ReactDOMSuspensePlaceholder-test.js
@@ -132,7 +132,7 @@ describe('ReactDOMSuspensePlaceholder', () => {
     expect(window.getComputedStyle(divs[0].current).display).toEqual('none');
     expect(window.getComputedStyle(divs[1].current).display).toEqual('none');
     expect(window.getComputedStyle(divs[2].current).display).toEqual('none');
-
+    assertLog(['A', 'Suspend! [B]', 'C', 'Loading...']);
     await act(async () => {
       await resolveText('B');
     });
@@ -141,6 +141,7 @@ describe('ReactDOMSuspensePlaceholder', () => {
     expect(window.getComputedStyle(divs[1].current).display).toEqual('block');
     // This div's display was set with a prop.
     expect(window.getComputedStyle(divs[2].current).display).toEqual('inline');
+    assertLog(['B']);
   });
 
   it('hides and unhides timed out text nodes', async () => {
@@ -203,6 +204,7 @@ describe('ReactDOMSuspensePlaceholder', () => {
         '<span style="display: none;">Sibling</span><span style=' +
           '"display: none;"></span>Loading...',
       );
+      assertLog(['Suspend! [Async]', 'Loading...']);
 
       // Update the inline display style. It will be overridden because it's
       // inside a hidden fallback.
@@ -211,6 +213,7 @@ describe('ReactDOMSuspensePlaceholder', () => {
         '<span style="display: none;">Sibling</span><span style=' +
           '"display: none;"></span>Loading...',
       );
+      assertLog(['Suspend! [Async]']);
 
       // Unsuspend. The style should now match the inline prop.
       await act(() => resolveText('Async'));

diff --git a/packages/react-dom/src/__tests__/ReactLegacyUpdates-test.js b/packages/react-dom/src/__tests__/ReactLegacyUpdates-test.js
@@ -1668,6 +1668,7 @@ describe('ReactLegacyUpdates', () => {
       await act(() => {
         ReactDOM.render(<Terminating />, container);
       });
+      assertLog(Array.from({length: LIMIT + 1}, (_, k) => k));
       expect(container.textContent).toBe('50');
       await act(() => {
         _setStep(0);