-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Move ui if tests from top-level into expr/if
#79395
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
@bors r+ I'm going to approve this, since you've done the work, but before you continue I think it would be good to have some broader discussion about the goals here -- in particular, I usually run either all UI tests, or a particular one -- I don't think I've ever wanted to run "if" tests only. But maybe that's more common in other kinds of compiler work? I mostly want to avoid churn and long-term needing to worry about this on every addition if there's not too much advantage. |
📌 Commit 6919a77 has been approved by |
Rollup of 11 pull requests Successful merges: - rust-lang#79327 (Require allocator to be static for boxed `Pin`-API) - rust-lang#79340 (Rename "stability" CSS class to "item-info" and combine `document_stability` with `document_short`) - rust-lang#79363 (BTreeMap: try to enhance various comments) - rust-lang#79395 (Move ui if tests from top-level into `expr/if`) - rust-lang#79443 (Improve rustdoc JS tests error output) - rust-lang#79464 (Extend doc keyword feature by allowing any ident) - rust-lang#79484 (add enable-full-tools to freebsd builds to prevent occasional link er…) - rust-lang#79505 (Cleanup: shorter and faster code) - rust-lang#79514 (Add test for issue rust-lang#54121: order dependent trait bounds) - rust-lang#79516 (Remove unnecessary `mut` binding) - rust-lang#79528 (Fix a bootstrap comment) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Rustc_codegen_cranelift doesn't yet support certain features, so I remove the tests for those features. Some features have their tests in a single dir, so removing them is easy. Others have them spread out in the root dir, so I have to remove each one individually: https://github.com/bjorn3/rustc_codegen_cranelift/blob/5f60b36a715eef37d8851ca81e62f5a919b8b54e/test.sh#L102-L169 |
This lowers the number of top-level files in src/test/ui from 1612 to 1604.