cargo test --help: clarify --tests and --benches#14675
cargo test --help: clarify --tests and --benches#14675bors merged 2 commits intorust-lang:masterfrom
Conversation
src/bin/cargo/commands/test.rs
Outdated
| "Test all examples", | ||
| "Test only the specified test target", | ||
| "Test all test targets", | ||
| "Test all targets that have the `test = true` manifest flag set", |
There was a problem hiding this comment.
Could we make them shorter, like
| "Test all targets that have the `test = true` manifest flag set", | |
| "Test all targets that have `test = true` set", |
and ditto --benches?
(I still find it confusing though, because it doesn't really test "all" targets. Anyway this PR is already an improvement)
src/bin/cargo/commands/test.rs
Outdated
| "Test all targets that have the `test = true` manifest flag set", | ||
| "Test only the specified bench target", | ||
| "Test all bench targets", | ||
| "Test all targets in benchmark mode that have the `bench = true` manifest flag set", |
There was a problem hiding this comment.
I am not sure what it means to "test a target in benchmark mode",
I don't really know, either. I think "in benchmark mode" should be removed from here and in man pages as well. Basically how I understand it is collecting Cargo targets that have bench = true and nothing else.
There was a problem hiding this comment.
I think "in benchmark mode" is left over from when cargo used to change modes/profiles based on which targets were being built. When we stabilized named profiles, we also changed cargo to stop doing that and use only a single profile.
I would delete it here and in the docs.
There was a problem hiding this comment.
Okay, I have done that. This was duplicated in a bunch of places so I hope this will work.
The term benchmark mode is now gone entirely; test mode still appears in a few places -- probably better for one of you to take a look since I am lacking the necessary context here.
|
I realized the |
There was a problem hiding this comment.
Here it was also talking about "benchmarking in test mode" vs "benchmarking in bench mode" -- I removed it for consistency, or is that still relevant somehow?
|
@bors r+ |
|
☀️ Test successful - checks-actions |
Update cargo 14 commits in 15fbd2f607d4defc87053b8b76bf5038f2483cf4..8c30ce53688e25f7e9d860b33cc914fb2957ca9a 2024-10-08 21:08:11 +0000 to 2024-10-15 16:43:16 +0000 - docs: More information on what is and isn't included by cargo package (rust-lang/cargo#14684) - fix(resolver): share conflict cache between activation retries (rust-lang/cargo#14692) - fix(git): dont fetch tags by default (rust-lang/cargo#14688) - Support package selection options like `--exclude` in `cargo publish` (rust-lang/cargo#14659) - docs: install options -> uninstall options (rust-lang/cargo#14682) - docs: tools should only interpret a line starting with `{` as JSON (rust-lang/cargo#14677) - cargo test --help: clarify --tests and --benches (rust-lang/cargo#14675) - docs(env): minor improvements in environment variables doc (rust-lang/cargo#14676) - docs: document official external commands (rust-lang/cargo#14669) - Fix panic when running cargo tree on a package with a cross compiled bindep (rust-lang/cargo#14593) - Remove the support for `Cargo.toml` of the cargo-script (rust-lang/cargo#14670) - docs(resolver): Lay groundwork for documenting MSRV-aware logic (rust-lang/cargo#14662) - chore(deps): update rust crate pulldown-cmark to 0.12.0 (rust-lang/cargo#14668) - Improve resolver speed (rust-lang/cargo#14663)
Update cargo 14 commits in 15fbd2f607d4defc87053b8b76bf5038f2483cf4..8c30ce53688e25f7e9d860b33cc914fb2957ca9a 2024-10-08 21:08:11 +0000 to 2024-10-15 16:43:16 +0000 - docs: More information on what is and isn't included by cargo package (rust-lang/cargo#14684) - fix(resolver): share conflict cache between activation retries (rust-lang/cargo#14692) - fix(git): dont fetch tags by default (rust-lang/cargo#14688) - Support package selection options like `--exclude` in `cargo publish` (rust-lang/cargo#14659) - docs: install options -> uninstall options (rust-lang/cargo#14682) - docs: tools should only interpret a line starting with `{` as JSON (rust-lang/cargo#14677) - cargo test --help: clarify --tests and --benches (rust-lang/cargo#14675) - docs(env): minor improvements in environment variables doc (rust-lang/cargo#14676) - docs: document official external commands (rust-lang/cargo#14669) - Fix panic when running cargo tree on a package with a cross compiled bindep (rust-lang/cargo#14593) - Remove the support for `Cargo.toml` of the cargo-script (rust-lang/cargo#14670) - docs(resolver): Lay groundwork for documenting MSRV-aware logic (rust-lang/cargo#14662) - chore(deps): update rust crate pulldown-cmark to 0.12.0 (rust-lang/cargo#14668) - Improve resolver speed (rust-lang/cargo#14663)
This tries to reduce the confusion expressed in #10936.
I am not sure what it means to "test a target in benchmark mode", this is copied verbatim from
cargo help test.