Fix setState callback error test

Author: acdlite

scripts/fiber/tests-passing.txt

@@ -453,6 +453,7 @@ src/renderers/dom/__tests__/ReactDOMProduction-test.js
 * should use prod React
 * should handle a simple flow
 * should call lifecycle methods
+* should throw with an error code in production
 
 src/renderers/dom/fiber/__tests__/ReactDOMFiber-test.js
 * should render strings as children
@@ -816,6 +817,7 @@ src/renderers/shared/fiber/__tests__/ReactIncrementalErrorHandling-test.js
 * continues work on other roots despite caught errors
 * continues work on other roots despite uncaught errors
 * force unmounts failed subtree before rerendering
+* force unmounts failed subtree before rerendering (fragment)
 * force unmounts failed root
 
 src/renderers/shared/fiber/__tests__/ReactIncrementalReflection-test.js

src/renderers/shared/fiber/__tests__/ReactIncremental-test.js

@@ -1482,30 +1482,26 @@ describe('ReactIncremental', () => {
     ReactNoop.flush();
     ops = [];
 
-    expect(instance.state.n).toEqual(0);
-
-    // first good callback
-    instance.setState({ n: 1 }, () => ops.push('first good callback'));
-    ReactNoop.flush();
+    function updater({ n }) {
+      return { n: n + 1 };
+    }
 
-    // callback throws
-    instance.setState({ n: 2 }, () => {
-      throw new Error('Bail');
+    instance.setState(updater, () => ops.push('first callback'));
+    instance.setState(updater, () => {
+      ops.push('second callback');
+      throw new Error('callback error');
     });
+    instance.setState(updater, () => ops.push('third callback'));
+
     expect(() => {
       ReactNoop.flush();
-    }).toThrow('Bail');
-    
-    // should set state to 2 even if callback throws up
-    expect(instance.state.n).toEqual(2);
-
-    // another good callback
-    instance.setState({ n: 3 }, () => ops.push('second good callback'));
-    ReactNoop.flush();
+    }).toThrow('callback error');
 
+    // Should call all callbacks, even though the second one throws
     expect(ops).toEqual([
-      'first good callback',
-      'second good callback',
+      'first callback',
+      'second callback',
+      'third callback',
     ]);
     expect(instance.state.n).toEqual(3);
   });