-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
cargo doesn't include test = false tests in --tests or --all-targets #13828
Comments
For myself, I think the |
Yeah, Wdyt about having The full context is here using cargo-fixture. |
Hmm… I can't see it. Changing that will include Cargo targets not supposed to build, and is a breaking change for some existing scripts relying on the current behavior. |
Yes, there is a risk that people would have |
Problem
A test marked as
test = false
is supposed to not be run when invokingcargo test
.This works, however, not only is the test not run, it's not built or checked (when using
cargo check
) either, not even with--tests
or--all-targets
passed.As a workaround, I'm currently using
--tests \*
instead of--tests
to actually include all tests includingtest = false
ones.Steps
test = false
inCargo.toml
cargo check --tests
cargo check --test \*
Possible Solution(s)
No response
Notes
No response
Version
The text was updated successfully, but these errors were encountered: