-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
test_runner: detect only tests when --test is not used #54881
Conversation
Review requested:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, can we also add a test with isolation-none and only?
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #54881 +/- ##
==========================================
- Coverage 88.07% 88.06% -0.01%
==========================================
Files 651 651
Lines 183409 183409
Branches 35826 35825 -1
==========================================
- Hits 161529 161514 -15
- Misses 15142 15151 +9
- Partials 6738 6744 +6
|
Should the documentation be updated accordingly? (I'm not sure how this is currently documented) |
This commit updates the way the test runner processes 'only' tests when node:test files are run without the --test CLI. This is a breaking change.
CI: https://ci.nodejs.org/job/node-test-pull-request/62370/ 🟢 A rare all green CI that was not resumed. |
@MoLow (and another TSC member since this is semver-major) PTAL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
amazing progress
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Landed in dbaef33 |
This commit updates the way the test runner processes 'only' tests when node:test files are run without the - test CLI.
The first two commits here are from #54832. Please ignore them while reviewing. The third commit is a breaking change.I was going to wait to open this PR until #54832 landed, but it's taking longer than expected because of CI flakiness, so I figured I would get the 48 hour clock started on this change.