Update tests to reflect that we're now including unmount times for deleted subtrees

Author: sebmarkbage

packages/react/src/__tests__/ReactProfiler-test.internal.js

@@ -1643,7 +1643,7 @@ describe(`onCommit`, () => {
     expect(call).toHaveLength(4);
     expect(call[0]).toBe('root-update');
     expect(call[1]).toBe('update');
-    expect(call[2]).toBe(1100); // durations
+    expect(call[2]).toBe(11100); // durations
     expect(call[3]).toBe(1124); // commit start time (before mutations or effects)
   });
 
@@ -1952,11 +1952,7 @@ describe(`onPostCommit`, () => {
     expect(call).toHaveLength(4);
     expect(call[0]).toBe('unmount-test');
     expect(call[1]).toBe('update');
-    // TODO (bvaughn) The duration reported below should be 10100, but is 0
-    // by the time the passive effect is flushed its parent Fiber pointer is gone.
-    // If we refactor to preserve the unmounted Fiber tree we could fix this.
-    // The current implementation would require too much extra overhead to track this.
-    expect(call[2]).toBe(0); // durations
+    expect(call[2]).toBe(10100); // durations
     expect(call[3]).toBe(12030); // commit start time (before mutations or effects)
   });
 
@@ -2085,7 +2081,7 @@ describe(`onPostCommit`, () => {
     expect(call).toHaveLength(4);
     expect(call[0]).toBe('root-update');
     expect(call[1]).toBe('update');
-    expect(call[2]).toBe(1100); // durations
+    expect(call[2]).toBe(11100); // durations
     expect(call[3]).toBe(1124); // commit start time (before mutations or effects)
   });
 
@@ -2300,7 +2296,7 @@ describe(`onPostCommit`, () => {
     expect(call).toHaveLength(4);
     expect(call[0]).toBe('root');
     expect(call[1]).toBe('update');
-    expect(call[2]).toBe(100000000); // durations
+    expect(call[2]).toBe(100001000); // durations
     // The commit time varies because the above duration time varies
     expect(call[3]).toBe(11221221); // commit start time (before mutations or effects)
   });