Skip to content
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

x.py test does not always support --test-args #72536

Closed
RalfJung opened this issue May 24, 2020 · 3 comments
Closed

x.py test does not always support --test-args #72536

RalfJung opened this issue May 24, 2020 · 3 comments
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc C-enhancement Category: An issue proposing an enhancement or a PR with one. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@RalfJung
Copy link
Member

#72527 added support for --test-args for miri and clippy. But some other tools still do not support this (I am not sure if it would make sense for all of them):

  • Linkcheck
  • Cargotest
  • Cargo
  • Rustfmt
  • CompiletestTest
  • All the Rustdoc ones
  • Tidy
  • ExpandYamlAnchors
  • the book tests
  • RustcGuide
  • Distcheck

Cc @Mark-Simulacrum

@jonas-schievink jonas-schievink added T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) A-testsuite Area: The testsuite used to check the correctness of rustc C-enhancement Category: An issue proposing an enhancement or a PR with one. labels May 24, 2020
@JohnTitor JohnTitor added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label May 28, 2020
@jyn514
Copy link
Member

jyn514 commented Jul 11, 2020

All the Rustdoc ones

I don't think this is true? x.py test --stage 1 src/test/rustdoc --test-args intra-links works fine for me locally.

@RalfJung
Copy link
Member Author

Maybe I missed the code that implements that.

@jyn514
Copy link
Member

jyn514 commented Nov 9, 2022

I'm going to close this issue, it seems too broad. If you run into a specific tool that isn't respecting --test-args we can open an issue for that.

I do want to refactor all the test code to be consistent at some point; right now we copy paste a bunch of code in various places and it ends up with not all test suites supporting the same arguments. But that's a little different from what you have here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc C-enhancement Category: An issue proposing an enhancement or a PR with one. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

4 participants