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

Fix test names generated by compiletest #107039

Closed
wants to merge 1 commit into from

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