Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Jest and remove hacks #11372

Merged
merged 3 commits into from
Oct 26, 2017
Merged

Update Jest and remove hacks #11372

merged 3 commits into from
Oct 26, 2017

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Oct 26, 2017

I fixed a Jest + Yarn Workspaces integration bug in jestjs/jest#4761.
This lets us remove the hacky transform whitelist, and use relative requires in tests again.

I changed them to absolute to work around a Jest bug.
The bug has been fixed so I can revert my past changes now.
@@ -53,6 +53,7 @@
"eslint-plugin-flowtype": "^2.25.0",
"eslint-plugin-react": "^6.7.1",
"eslint-plugin-react-internal": "file:./scripts/eslint-rules",
"expect": "^21.3.0-beta.4",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

jest-matchers was renamed to expect.

Copy link
Contributor

@trueadm trueadm left a comment

Choose a reason for hiding this comment

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

LGTM

@gaearon gaearon merged commit 707ca7f into facebook:master Oct 26, 2017
@gaearon gaearon deleted the jest-up branch October 26, 2017 14:15
@gaearon gaearon mentioned this pull request Oct 26, 2017
19 tasks
@xjlim xjlim mentioned this pull request Oct 28, 2017
Ethan-Arrowood pushed a commit to Ethan-Arrowood/react that referenced this pull request Dec 8, 2017
* Update Jest

* Remove hacks for Jest + Workspace integration

They were fixed by jestjs/jest#4761.

* Use relative requires in tests relying on private APIs

I changed them to absolute to work around a Jest bug.
The bug has been fixed so I can revert my past changes now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants