-
Notifications
You must be signed in to change notification settings - Fork 47.3k
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
Upgrade to Jest 23 #12894
Upgrade to Jest 23 #12894
Conversation
3a71609
to
67991b4
Compare
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", |
There was a problem hiding this comment.
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
Sorry we missed this, do you mind fixing up merge conflicts? |
@gaearon rebased and upgraded to 23.0.1 |
There was a problem hiding this 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 :)
|
Thanks! |
* Upgrade to Jest 23 beta * prefer `.toHaveBeenCalledTimes` * 23 stable
👋
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!