[noop] Fail tests on unasserted recoverable errors

Author: eps1lon

packages/react-noop-renderer/src/createReactNoop.js

@@ -1173,8 +1173,8 @@ function createReactNoop(
   }
 
   function onRecoverableError(error: mixed): void {
-    // TODO: Turn this on once tests are fixed
-    // console.error(error);
+    // eslint-disable-next-line react-internal/warning-args, react-internal/no-production-logging -- renderer is only used for testing.
+    console.error(error);
   }
   function onDefaultTransitionIndicator(): void | (() => void) {}
 

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

@@ -287,6 +287,10 @@ describe('ReactIncrementalErrorHandling', () => {
       'commit',
       'commit',
     ]);
+    assertConsoleErrorDev([
+      'Error: There was an error during concurrent rendering but React was able to recover by instead synchronously rendering the entire root.' +
+        '\n    in <stack>',
+    ]);
     expect(ReactNoop).toMatchRenderedOutput(
       <span prop="Everything is fine." />,
     );
@@ -339,6 +343,10 @@ describe('ReactIncrementalErrorHandling', () => {
       'commit',
       'commit',
     ]);
+    assertConsoleErrorDev([
+      'Error: There was an error during concurrent rendering but React was able to recover by instead synchronously rendering the entire root.' +
+        '\n    in <stack>',
+    ]);
     // This should not include the offscreen content
     expect(ReactNoop).toMatchRenderedOutput(
       <>
@@ -1786,6 +1794,10 @@ describe('ReactIncrementalErrorHandling', () => {
     });
 
     // Should finish without throwing.
+    assertConsoleErrorDev([
+      'Error: There was an error during concurrent rendering but React was able to recover by instead synchronously rendering the entire root.' +
+        '\n    in <stack>',
+    ]);
     expect(root).toMatchRenderedOutput('Everything is fine.');
   });
 
@@ -1832,6 +1844,10 @@ describe('ReactIncrementalErrorHandling', () => {
     });
     // Should render the final state without throwing the error.
     assertLog(['Everything is fine.']);
+    assertConsoleErrorDev([
+      'Error: There was an error during concurrent rendering but React was able to recover by instead synchronously rendering the entire root.' +
+        '\n    in <stack>',
+    ]);
     expect(root).toMatchRenderedOutput('Everything is fine.');
   });
 

packages/react-reconciler/src/__tests__/ReactIncrementalErrorReplay-test.js

@@ -12,6 +12,7 @@
 
 let React;
 let ReactNoop;
+let assertConsoleErrorDev;
 let waitForAll;
 let waitForThrow;
 
@@ -22,6 +23,7 @@ describe('ReactIncrementalErrorReplay', () => {
     ReactNoop = require('react-noop-renderer');
 
     const InternalTestUtils = require('internal-test-utils');
+    assertConsoleErrorDev = InternalTestUtils.assertConsoleErrorDev;
     waitForAll = InternalTestUtils.waitForAll;
     waitForThrow = InternalTestUtils.waitForThrow;
   });
@@ -50,5 +52,9 @@ describe('ReactIncrementalErrorReplay', () => {
     }
     ReactNoop.render(<App />);
     await waitForAll([]);
+    assertConsoleErrorDev([
+      'Error: There was an error during concurrent rendering but React was able to recover by instead synchronously rendering the entire root.' +
+        '\n    in <stack>',
+    ]);
   });
 });

packages/react-reconciler/src/__tests__/useMemoCache-test.js

@@ -8,6 +8,8 @@
  * @jest-environment node
  */
 
+import {assertConsoleErrorDev} from 'internal-test-utils';
+
 let React;
 let ReactNoop;
 let Scheduler;
@@ -256,8 +258,6 @@ describe('useMemoCache()', () => {
       return `${data.text} (n=${props.n})`;
     });
 
-    spyOnDev(console, 'error');
-
     const root = ReactNoop.createRoot();
     await act(() => {
       root.render(
@@ -274,6 +274,10 @@ describe('useMemoCache()', () => {
       // this triggers a throw.
       setN(1);
     });
+    assertConsoleErrorDev([
+      'Error: There was an error during concurrent rendering but React was able to recover by instead synchronously rendering the entire root.' +
+        '\n    in <stack>',
+    ]);
     expect(root).toMatchRenderedOutput('Count 0 (n=1)');
     expect(Text).toBeCalledTimes(2);
     expect(data).toBe(data0);