forked from facebook/react
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit 325bb34
fix[devtools/ci]: split profiling cache test for different react versions and toEqual checker (facebook#28628)
This should fix the failing backwards-compatibility tests on CI:
-
https://app.circleci.com/pipelines/github/facebook/react/51347/workflows/9d319db5-7a29-4e9a-a3a0-8d49a24ee9bd/jobs/809381
-
https://app.circleci.com/pipelines/github/facebook/react/51347/workflows/9d319db5-7a29-4e9a-a3a0-8d49a24ee9bd/jobs/809386
Started failing after facebook#27991.
Brief summary:
1. Revert changes to `profilingCache-test` in
facebook#27991.
2. Scope previous test to react versions 16.9 - 18.2
3. Add a new test for react versions > 18.2 (includes testing react from
souce), which is also gated with `!disableLegacyContext`.
> [!IMPORTANT]
> `@gate` pragma expects the test to throw. Jest doesn't throw any
exceptions when snapshots are mismatched, this is why I've migrated test
from `toMatchInlineSnapshot` checker to `toEqual`.
> If the test doesn't throw, we will fail it manually:
>
https://github.com/facebook/react/blob/5a75f9e78544fa6d052aff7fe99607e48f35b979/scripts/jest/setupTests.js#L291-L2951 parent 4df4e53 commit 325bb34Copy full SHA for 325bb34
File tree
Expand file treeCollapse file tree
1 file changed
+386
-21
lines changedFilter options
- packages/react-devtools-shared/src/__tests__
Expand file treeCollapse file tree
1 file changed
+386
-21
lines changed
0 commit comments