Closed mildmojo closed 8 months ago
Thanks for the great bug report @mildmojo!
See #3297, the counters were never cleared correctly, so on every run they would increase.
Previous reports were likely closed because the watch mode was rewritten, and that should have resolved any remaining issues, but for a path inconsistency.
Thanks for the fast fix! :raised_hands:
Actual
When AVA v6.1.0 runs in watch mode, it prints something like
4 previous failures in test files that were not rerun
on every rerun, counting all previously-failed tests from all previous runs, even after re-running all tests with 100% passing results.Expected
When AVA runs in watch mode, it should not report previous test failures. When all tests are passing, it should report that all tests are passing, full stop.
Reproduction
Minimal reproduction steps below, with annotations about observed behavior:
Touch
test.js
to let AVA re-run the test:Change
t.fail()
tot.pass()
and save:Type 'r' in the test console and hit enter to re-run tests:
The "previous failures" message never seems to clear. I can't figure out what it means or how it would be useful; all test files were rerun, and none had failures.
There are some previous reports of similar issues (#2821, #2069, #2040), but they've all been closed, and this still seems to happen in the current release.