Mark TestRendererAsync test as internal and revert changes to it

TestRenderer is built with strict mode doublerender off.

We could change that but I'm not sure we want to. So I'll just flip the flag off for this test.

Author: gaearon

packages/react-test-renderer/src/__tests__/ReactTestRendererAsync-test.js renamed to packages/react-test-renderer/src/__tests__/ReactTestRendererAsync-test.internal.js

@@ -11,11 +11,14 @@
 'use strict';
 
 let React;
+let ReactFeatureFlags;
 let ReactTestRenderer;
 
 describe('ReactTestRendererAsync', () => {
   beforeEach(() => {
     jest.resetModules();
+    ReactFeatureFlags = require('shared/ReactFeatureFlags');
+    ReactFeatureFlags.debugRenderPhaseSideEffectsForStrictMode = false;
     React = require('react');
     ReactTestRenderer = require('react-test-renderer');
   });
@@ -62,19 +65,11 @@ describe('ReactTestRendererAsync', () => {
       unstable_isConcurrent: true,
     });
 
-    expect(renderer).toFlushAndYield(
-      __DEV__
-        ? ['A:1', 'A:1', 'B:1', 'B:1', 'C:1', 'C:1']
-        : ['A:1', 'B:1', 'C:1'],
-    );
+    expect(renderer).toFlushAndYield(['A:1', 'B:1', 'C:1']);
     expect(renderer.toJSON()).toEqual(['A:1', 'B:1', 'C:1']);
 
     renderer.update(<Parent step={2} />);
-    expect(renderer).toFlushAndYield(
-      __DEV__
-        ? ['A:2', 'A:2', 'B:2', 'B:2', 'C:2', 'C:2']
-        : ['A:2', 'B:2', 'C:2'],
-    );
+    expect(renderer).toFlushAndYield(['A:2', 'B:2', 'C:2']);
     expect(renderer.toJSON()).toEqual(['A:2', 'B:2', 'C:2']);
   });
 
@@ -97,14 +92,12 @@ describe('ReactTestRendererAsync', () => {
     });
 
     // Flush the first two siblings
-    expect(renderer).toFlushAndYieldThrough(
-      __DEV__ ? ['A:1', 'A:1', 'B:1', 'B:1'] : ['A:1', 'B:1'],
-    );
+    expect(renderer).toFlushAndYieldThrough(['A:1', 'B:1']);
     // Did not commit yet.
     expect(renderer.toJSON()).toEqual(null);
 
     // Flush the remaining work
-    expect(renderer).toFlushAndYield(__DEV__ ? ['C:1', 'C:1'] : ['C:1']);
+    expect(renderer).toFlushAndYield(['C:1']);
     expect(renderer.toJSON()).toEqual(['A:1', 'B:1', 'C:1']);
   });
 
@@ -136,7 +129,7 @@ describe('ReactTestRendererAsync', () => {
     });
 
     // Flush the some of the changes, but don't commit
-    expect(renderer).toFlushAndYieldThrough(__DEV__ ? ['A:1', 'A:1'] : ['A:1']);
+    expect(renderer).toFlushAndYieldThrough(['A:1']);
     expect(renderer.toJSON()).toEqual(null);
 
     // Interrupt with higher priority properties
@@ -232,54 +225,30 @@ describe('ReactTestRendererAsync', () => {
       });
 
       expect(renderer).toFlushAndThrow('Oh no!');
-      expect(ReactTestRenderer).toHaveYielded(
-        __DEV__
-          ? [
-              'A',
-              'A',
-              'B',
-              'B',
-              'C',
-              'C',
-              'D',
-              'D',
-              'A',
-              'A',
-              'B',
-              'B',
-              'C',
-              'C',
-              'D',
-              'D',
-            ]
-          : ['A', 'B', 'C', 'D', 'A', 'B', 'C', 'D'],
-      );
+      expect(ReactTestRenderer).toHaveYielded([
+        'A',
+        'B',
+        'C',
+        'D',
+        'A',
+        'B',
+        'C',
+        'D',
+      ]);
 
       renderer.update(<App />);
 
       expect(renderer).toFlushAndThrow('Oh no!');
-      expect(ReactTestRenderer).toHaveYielded(
-        __DEV__
-          ? [
-              'A',
-              'A',
-              'B',
-              'B',
-              'C',
-              'C',
-              'D',
-              'D',
-              'A',
-              'A',
-              'B',
-              'B',
-              'C',
-              'C',
-              'D',
-              'D',
-            ]
-          : ['A', 'B', 'C', 'D', 'A', 'B', 'C', 'D'],
-      );
+      expect(ReactTestRenderer).toHaveYielded([
+        'A',
+        'B',
+        'C',
+        'D',
+        'A',
+        'B',
+        'C',
+        'D',
+      ]);
 
       renderer.update(<App />);
       expect(renderer).toFlushAndThrow('Oh no!');
@@ -317,4 +286,4 @@ describe('ReactTestRendererAsync', () => {
       'Log of yielded values is not empty.',
     );
   });
-});
+});