Skip to content

Fix HelloWorld Jest preset after react-native/jest-preset removal - #57491

Closed
cipolleschi wants to merge 1 commit into
mainfrom
cipolleschi/fix-helloworld-jest-preset
Closed

Fix HelloWorld Jest preset after react-native/jest-preset removal#57491
cipolleschi wants to merge 1 commit into
mainfrom
cipolleschi/fix-helloworld-jest-preset

Conversation

@cipolleschi

@cipolleschicipolleschi commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

#57481 removed the react-native/jest-preset.js redirect module (and the react-native/jest-preset public export), but the in-repo private/helloworld app still referenced preset: 'react-native' in its Jest config. This breaks yarn test in the test_ios_helloworld CI job (which cascade-cancels the rest of the iOS matrix and turns the e2e retry report jobs red):

● Validation Error:
Module react-native should have "jest-preset.js" or "jest-preset.json" file at the root.

This migrates HelloWorld to the standalone @react-native/jest-preset package — the documented migration path (see packages/jest-preset/README.md).

Changes

  • private/helloworld/jest.config.js: preset: 'react-native'preset: '@react-native/jest-preset'
  • private/helloworld/package.json: add @react-native/jest-preset devDependency (0.87.0-main, matching the sibling @react-native/* packages)

Changelog:

[Internal] [Fixed] - Fix HelloWorld Jest preset after react-native/jest-preset removal

Test Plan

  • cd private/helloworld && yarn test resolves the preset and runs (previously failed with the validation error above).
  • CI test_ios_helloworld job.

The react-native/jest-preset.js redirect was removed in #57481, but
private/helloworld still referenced 'preset: react-native' in its Jest
config, breaking 'yarn test' in the test_ios_helloworld CI job with:
Validation Error: Module react-native should have jest-preset.js ...
Migrate HelloWorld to the standalone @react-native/jest-preset package,
matching the documented migration path.
@meta-clameta-claBot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 9, 2026
@meta-codesync

Copy link
Copy Markdown

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this in D111215532.

@cortinicocortinico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review automatically exported from Phabricator review in Meta.

@meta-codesyncmeta-codesyncBot added the Merged This PR has been merged. label Jul 9, 2026
@meta-codesync

Copy link
Copy Markdown

@cipolleschi merged this pull request in 7d2ef78.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.MergedThis PR has been merged.p: FacebookPartner: FacebookPartner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@cipolleschi@cortinico