Skip to content

Commit c600b63

Browse files
jackpopenikeee
authored andcommitted
Clean up disableDOMTestUtils (facebook#29610)
`disableDOMTestUtils` and the FB build `ReactTestUtilsFB` allowed us to finish migrating internal callsites off of ReactTestUtils. Now that usage is cleaned up, we can remove the flag, build artifact, and test coverage for the deprecated utility methods.
1 parent fe95af9 commit c600b63

11 files changed

+1
-1640
lines changed

packages/react-dom/src/__tests__/ReactTestUtils-test.js

Lines changed: 0 additions & 735 deletions
This file was deleted.

packages/react-dom/src/test-utils/ReactTestUtilsFB.js

Lines changed: 0 additions & 884 deletions
This file was deleted.

packages/react-dom/test-utils.fb.js

Lines changed: 0 additions & 10 deletions
This file was deleted.

packages/shared/ReactFeatureFlags.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -192,8 +192,6 @@ export const enableReactTestRendererWarning = true;
192192
// before removing them in stable in the next Major
193193
export const disableLegacyMode = true;
194194

195-
export const disableDOMTestUtils = true;
196-
197195
// Make <Context> equivalent to <Context.Provider> instead of <Context.Consumer>
198196
export const enableRenderableContext = true;
199197

packages/shared/forks/ReactFeatureFlags.native-fb.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,6 @@ export const disableStringRefs = true;
9898

9999
export const enableReactTestRendererWarning = false;
100100
export const disableLegacyMode = false;
101-
export const disableDOMTestUtils = false;
102-
103101
export const enableOwnerStacks = false;
104102

105103
// Flow magic to verify the exports of this file match the original version.

packages/shared/forks/ReactFeatureFlags.native-oss.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ export const enableRefAsProp = __TODO_NEXT_RN_MAJOR__;
2222
export const disableStringRefs = __TODO_NEXT_RN_MAJOR__;
2323
export const enableFastJSX = __TODO_NEXT_RN_MAJOR__;
2424
export const disableLegacyMode = __TODO_NEXT_RN_MAJOR__;
25-
export const disableDOMTestUtils = __TODO_NEXT_RN_MAJOR__;
2625
export const useModernStrictMode = __TODO_NEXT_RN_MAJOR__;
2726
export const enableReactTestRendererWarning = __TODO_NEXT_RN_MAJOR__;
2827
export const enableAsyncActions = __TODO_NEXT_RN_MAJOR__;

packages/shared/forks/ReactFeatureFlags.test-renderer.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,6 @@ export const disableStringRefs = true;
9292
export const enableFastJSX = true;
9393
export const disableLegacyMode = true;
9494
export const disableLegacyContext = true;
95-
export const disableDOMTestUtils = true;
9695
export const enableRenderableContext = true;
9796
export const enableReactTestRendererWarning = true;
9897
export const disableDefaultPropsExceptForClasses = true;

packages/shared/forks/ReactFeatureFlags.test-renderer.native-fb.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ export const enableFastJSX = true;
8585

8686
export const enableReactTestRendererWarning = false;
8787
export const disableLegacyMode = false;
88-
export const disableDOMTestUtils = false;
8988

9089
export const disableDefaultPropsExceptForClasses = false;
9190
export const enableAddPropertiesFastPath = false;

packages/shared/forks/ReactFeatureFlags.test-renderer.www.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ export const enableFastJSX = false;
8585

8686
export const enableReactTestRendererWarning = false;
8787
export const disableLegacyMode = false;
88-
export const disableDOMTestUtils = false;
8988

9089
export const disableDefaultPropsExceptForClasses = false;
9190
export const enableAddPropertiesFastPath = false;

packages/shared/forks/ReactFeatureFlags.www.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,6 @@ export const disableStringRefs = false;
120120

121121
export const disableLegacyMode = __EXPERIMENTAL__;
122122

123-
export const disableDOMTestUtils = false;
124-
125123
export const enableOwnerStacks = false;
126124

127125
// Flow magic to verify the exports of this file match the original version.

0 commit comments

Comments
 (0)