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

watchAll not running tests on save. #4448

Closed
truckingsim opened this issue Sep 7, 2017 · 6 comments · Fixed by #4550
Closed

watchAll not running tests on save. #4448

truckingsim opened this issue Sep 7, 2017 · 6 comments · Fixed by #4550

Comments

@truckingsim
Copy link
Contributor

Do you want to request a feature or report a bug?

Bug

What is the current behavior?

When running --watchAll the test don't run on file save. They do if i use --watch.

If the current behavior is a bug, please provide the steps to reproduce and either a repl.it demo through https://repl.it/languages/jest or a minimal repository on GitHub that we can yarn install and yarn test.

https://github.com/truckingsim/jest_bug_example

Install the packages and then in separate tabs side by side ( or one at a time this happens either way) run npm test in one and npm run test-changes in the other. Edit add.js to fix the ReferenceError on line 2, and save. Thetest-changes command is the expected behavior with --watch. The tests run on file save. The test command is watchAll the tests don't re-run on save.

What is the expected behavior?

Jest should run the tests on file save.

Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.

No jest config, versions are:

Jest - 21.0.1,
Node - 8.4.0
npm - 5.3.0
OS - MacOS Sierra 10.12.6

This happens with or without watchman, but I also have watchman version 4.9.0 installed.

@truckingsim
Copy link
Contributor Author

This was working on Jest 20.0.4, this only happened after upgrading to 21.0.1.

@rogeliog
Copy link
Contributor

rogeliog commented Sep 8, 2017

I'm also able to reproduce it, seems like a regression in 21.0.1

@mts
Copy link

mts commented Sep 21, 2017

@rogeliog Do I still need to create a jest bug example like @truckingsim asked for or is it not necessary at this point ?

@GiancarlosIO
Copy link

Hi i'm getting the same bug with "jest": "^21.1.0", :(

@truckingsim
Copy link
Contributor Author

@GiancarlosIO if you update to the latest version of jest it should fix the watchAll for you.

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants