fix flow, fix creteReactClassIntegration test

facebook · Dec 3, 2024 · 25b3193 · 25b3193
1 parent 7879dbc
commit 25b3193
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/packages/react/index.fb.js b/packages/react/index.fb.js
@@ -72,7 +72,7 @@ export {useMemoCache as unstable_useMemoCache} from './src/ReactHooks';
 export {useMemoCache as c} from './src/ReactHooks';
 
 // Only export captureOwnerStack if the flag is on, to support feature detection.
-let captureOwnerStack;
+let captureOwnerStack: ?() => null | string;
 if (enableOwnerStacks) {
   captureOwnerStack = captureOwnerStackImpl;
 }

diff --git a/packages/react/src/__tests__/createReactClassIntegration-test.js b/packages/react/src/__tests__/createReactClassIntegration-test.js
@@ -338,7 +338,14 @@ describe('create-react-class-integration', () => {
       root.render(<Outer />);
     });
     assertConsoleErrorDev([
-      'Component uses the legacy childContextTypes API which will soon be removed. Use React.createContext() instead.',
+      gate(flags =>
+        flags.enableOwnerStacks
+          ? [
+              'Component uses the legacy childContextTypes API which will soon be removed. Use React.createContext() instead.',
+              {withoutStack: true},
+            ]
+          : 'Component uses the legacy childContextTypes API which will soon be removed. Use React.createContext() instead.',
+      ),
       'Component uses the legacy contextTypes API which will soon be removed. Use React.createContext() with static contextType instead.',
     ]);
     expect(container.firstChild.className).toBe('foo');