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

fix: Jest --watch fails to reload #182

Merged
merged 1 commit into from
Jan 13, 2019
Merged

Conversation

xiaoyuhen
Copy link
Contributor

@xiaoyuhen xiaoyuhen commented Jan 11, 2019

Summary

close #181

Test plan

  • npm run test:integrationWithWatch is running normally in cra example

@gregberge
Copy link
Member

@xiaoyuhen I don't understand why we don't launch the server in watchMode?

@xiaoyuhen
Copy link
Contributor Author

@neoziro
We are not starting the server when watchMode, but not starting the server when DisplayWatchUsage. When the user selects the corresponding watch usage, We started the service and executed the test.

@gregberge gregberge merged commit cc9bbfa into argos-ci:master Jan 13, 2019
@gregberge
Copy link
Member

OK Thanks!

@nicoalurralde
Copy link

I get "TypeError: Cannot read property 'watch' of undefined" with this fix. What am I missing?

@gregberge
Copy link
Member

It will be fixed in next release.

@nicoalurralde
Copy link

Thanks, just realised I needed to update Jets to 23.6.0 for a quick fix

JaKXz added a commit to JaKXz/jest-puppeteer that referenced this pull request Jul 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Jest --watch fails to reload
3 participants