Skip to content

Conversation

@bjorn3
Copy link
Member

@bjorn3 bjorn3 commented Jan 18, 2023

This is fallout from moving tests from src/test/ to tests/. Before this commit compiletest would give a test at tests/ui/foo.rs if your rust checkout is called my_rust_checkout the name my_rust_checkout/tests/ui/foo.rs rather than the proper tests/ui/foo.rs

This is fallout from moving tests from src/test/ to tests/. Before this
commit compiletest would give a test at tests/ui/foo.rs if your rust
checkout is called my_rust_checkout the name
my_rust_checkout/tests/ui/foo.rs rather than the proper tests/ui/foo.rs
@rustbot
Copy link
Collaborator

rustbot commented Jan 18, 2023

r? @Mark-Simulacrum

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 18, 2023
@bjorn3
Copy link
Member Author

bjorn3 commented Jan 18, 2023

Didn't check the existing PRs. Closing in favor of #107027.

@bjorn3 bjorn3 closed this Jan 18, 2023
@bjorn3 bjorn3 deleted the fix_compiletest_test_names branch January 18, 2023 18:18
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 S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants