Skip to content

Commit 4b905c8

Browse files
authored
Rollup merge of rust-lang#134423 - jieyouxu:bootstrap-test-valid, r=onur-ozkan
bootstrap: use specific-purpose ui test path for `test_valid` self-test I wanted to move some ui tests around in rust-lang#134418, which broke `test_valid` since it was referencing two non-specific-purpose ui tests. This PR instead adds two dummy tests under `tests/ui/bootstrap/self-test/`, for that purpose specifically. r? bootstrap
2 parents 53635e5 + 13e8313 commit 4b905c8

File tree

3 files changed

+5
-1
lines changed

3 files changed

+5
-1
lines changed

src/bootstrap/src/core/builder/tests.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ macro_rules! rustc {
9191
#[test]
9292
fn test_valid() {
9393
// make sure multi suite paths are accepted
94-
check_cli(["test", "tests/ui/attr-start.rs", "tests/ui/attr-shebang.rs"]);
94+
check_cli(["test", "tests/ui/bootstrap/self-test/a.rs", "tests/ui/bootstrap/self-test/b.rs"]);
9595
}
9696

9797
#[test]

tests/ui/bootstrap/self-test/a.rs

+2
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
//! Not used by compiler, this is used by bootstrap cli self-test.
2+
//@ ignore-test

tests/ui/bootstrap/self-test/b.rs

+2
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
//! Not used by compiler, used by bootstrap cli self-test.
2+
//@ ignore-test

0 commit comments

Comments
 (0)