-
-
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
test.concurrent.only
does not work as expected
#10790
Comments
Is it possible it is not skipping when |
Same problem here. Note that if you use "it.concurrent.skip" then test marked with "skip" really does not run. However, if you use "it.concurrent.only" then other tests still run despite being marked as skipped. Since this issue is marked as "Needs Repro" I have added reproduction repository: |
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
When using
test.concurrent.only
andtest.concurrent
with two different tests, thetest.concurrent
test runs but reports as skipped. This behavior does not happen withtest
andtest.only
.To Reproduce
Steps to reproduce the behavior:
In a test file with all
test.concurrent
, change one of them totest.concurrent.only
and rerun test for that file.Expected behavior
When run with regular
test.only
, the execution of the others are not performed and the test is marked as skipped. However, withtest.concurrent.only
, the test does run (only apparent via side effects likeconsole.log
) but the test is correctly marked as skipped. The test should not actually run like it does withtest.only
.Link to repl or repo (highly encouraged)
Using
test.only
:https://repl.it/repls/NativeHuskySweepsoftware#sum.test.js
In this repl you can see the console.log line does not appear for the skipped test.
Using
test.concurrent.only
:https://repl.it/repls/WillingDazzlingCategory#sum.test.js
But in this version the console.log line does appear for the skipped test.
envinfo
N/A (used repl above for all testing)
Notes
I did not try this mixing
test
andtest.concurrent.only
but there could be an issue there as well.The text was updated successfully, but these errors were encountered: