You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Feb 1, 2018. It is now read-only.
Run bwoken with multiple .js test files. One of those test files must fail. Observed:
When the failing test runs, the test run stops and the remainder of tests are not run.
When running multiple tests, if one test fails the test run stops. In practical terms, that means if one component turns buggy, tests for non-buggy components will never run (and we'll never know for sure that they're non-buggy).
It could be argued that it is preferred that as many tests are run as possible. One failing test should not stop the run. Log the failure and move on.
Perhaps it's just a difference in philosophy between me and the bwoken authors. For now I've modified script.rb so that the run method does not raise if a test fails.
The text was updated successfully, but these errors were encountered:
Regarding your last paragraph, I would tend to agree: this isn't a bug in my eyes, but a feature request. This could get added later as a command-line flag, but I'm fairly certain it won't be the default behavior.
I'd be happy to accept a PR for this if someone writes it. Otherwise, it won't be at the top of my priority list.
Steps to repro:
Observed:
When the failing test runs, the test run stops and the remainder of tests are not run.
When running multiple tests, if one test fails the test run stops. In practical terms, that means if one component turns buggy, tests for non-buggy components will never run (and we'll never know for sure that they're non-buggy).
It could be argued that it is preferred that as many tests are run as possible. One failing test should not stop the run. Log the failure and move on.
Perhaps it's just a difference in philosophy between me and the bwoken authors. For now I've modified script.rb so that the run method does not raise if a test fails.
The text was updated successfully, but these errors were encountered: