-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Notification pops up even when no tests are run #6363
Comments
PR welcome! I think a notification might make sense? But it should say "no tests ran" or something. Code is here: https://github.com/facebook/jest/blob/master/packages/jest-cli/src/reporters/notify_reporter.js |
Happy to pick this one up. What's the proposed format as currently the notifier expects a message and a title? |
Awesome! Title maybe "No tests were run", and an empty message? |
My thinking was that there should be no notification unless some tests are run. What’s the point of a notification that says nothing ran? |
Yeah this makes sense to me as well as the expected behaviour. Could simply check result of the test run and exit early from the notifier logic if |
@thymikee @rickhanlonii thoughts? |
I think such "empty" notifications may be annoying, especially when fiddling with git a lot. I'm OK with not displaying it whenever there were no tests run. |
Please, add an options to notify in case of faled test only |
Failed only is already an option: https://facebook.github.io/jest/docs/en/configuration.html#notifymode-string |
@SimenB sorry, not noticed (
shows notification if test is passed successfully
|
Closed via #6495 |
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. |
🐛 Bug Report
Jest notifies me as follows that 100% of 0 tests passed when I save any file in my watched directories, even if there are no tests associated with it. What is the utility of this?
To Reproduce
Steps to reproduce the behavior:
--watch --notify
No tests found related to files changed since last commit.
Expected behavior
Notifications are only shown when tests are run
The text was updated successfully, but these errors were encountered: