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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Jest 23 #12894

Merged
merged 3 commits into from
May 28, 2018
Merged

Upgrade to Jest 23 #12894

merged 3 commits into from
May 28, 2018

Conversation

SimenB
Copy link
Contributor

@SimenB SimenB commented May 23, 2018

馃憢

I upgraded in order to check for regressions. And as I made the migration, I thought I might as well push it and make a PR.

This also includes some refatoring moving from jasmine based spies to the Jest ones.

I started migrating your spyOnDev stuff, but that was... a lot. So I pulled out. Sorry!

@SimenB SimenB force-pushed the upgrade-jest branch 2 times, most recently from 3a71609 to 67991b4 Compare May 23, 2018 22:54
@SimenB SimenB mentioned this pull request May 28, 2018
6 tasks
package.json Outdated
@@ -69,8 +69,8 @@
"gzip-js": "~0.3.2",
"gzip-size": "^3.0.0",
"jasmine-check": "^1.0.0-rc.0",
"jest": "^22.4.4",
"jest-diff": "^22.4.3",
"jest": "^23.0.0-charlie.4",
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's change to 23.0.1

@gaearon
Copy link
Collaborator

gaearon commented May 28, 2018

Sorry we missed this, do you mind fixing up merge conflicts?

@SimenB SimenB changed the title Upgrade to Jest 23 beta Upgrade to Jest 23 May 28, 2018
@SimenB
Copy link
Contributor Author

SimenB commented May 28, 2018

@gaearon rebased and upgraded to 23.0.1

Copy link
Contributor

@thymikee thymikee left a comment

Choose a reason for hiding this comment

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

LGTM. Would be great to also turn on the no-jasmine-globals rule of eslint plugin as a followup :)

@SimenB
Copy link
Contributor Author

SimenB commented May 28, 2018

spyOnDev and spyOnProd uses jasmine spies. Maybe @aaronabramov can codemod it?

@gaearon gaearon merged commit aa85b0f into facebook:master May 28, 2018
@gaearon
Copy link
Collaborator

gaearon commented May 28, 2018

Thanks!

@SimenB SimenB deleted the upgrade-jest branch May 29, 2018 05:42
NMinhNguyen referenced this pull request in enzymejs/react-shallow-renderer Jan 29, 2020
* Upgrade to Jest 23 beta

* prefer `.toHaveBeenCalledTimes`

* 23 stable
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.

None yet

4 participants