forked from rust-lang/rust
-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Rollup merge of rust-lang#113518 - jyn514:streaming-failures, r=cuviper bootstrap/libtest: print test name eagerly on failure even with `verbose-tests=false` / `--quiet` Previously, libtest would wait until all tests finished running to print the progress, which made it annoying to run many tests at once (since you don't know which have failed). Change it to print the names as soon as they fail. This makes it much easier to know which test failed without having to wait for compiletest to completely finish running. Before: ``` Testing stage0 compiletest suite=ui mode=ui (x86_64-unknown-linux-gnu) running 15274 tests iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 88/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 176/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 264/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 352/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 440/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 528/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiFFiiiiiii ... ``` After: ``` Testing stage0 compiletest suite=ui mode=ui (x86_64-unknown-linux-gnu) running 15274 tests iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 88/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 176/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 264/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 352/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 440/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii 528/15274 iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii [ui] tests/ui/associated-type-bounds/implied-in-supertrait.rs ... F [ui] tests/ui/associated-type-bounds/return-type-notation/basic.rs#next_with ... F iiiiiiiiiiiii ... ``` This serves a similar use case to the existing RUSTC_TEST_FAIL_FAST, but is on by default and as a result much more discoverable. We should consider unifying RUSTC_TEST_FAIL_FAST with the `--no-fail-fast` flag in the future for consistency and discoverability.
- Loading branch information
Showing
4 changed files
with
191 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,125 @@ | ||
| //@ compile-flags: --test | ||
| //@ run-fail | ||
| //@ run-flags: --test-threads=1 --quiet | ||
| //@ check-run-results | ||
| //@ exec-env:RUST_BACKTRACE=0 | ||
| //@ normalize-stdout-test "finished in \d+\.\d+s" -> "finished in $$TIME" | ||
| //@ ignore-emscripten no threads support | ||
| //@ needs-unwind | ||
|
|
||
| #[test] | ||
| fn abc() { | ||
| panic!(); | ||
| } | ||
|
|
||
| #[test] | ||
| fn foo() { | ||
| panic!(); | ||
| } | ||
|
|
||
| #[test] | ||
| fn foo2() { | ||
| panic!(); | ||
| } | ||
|
|
||
| // run a whole bunch of tests so we can see what happens when we go over 88 columns | ||
| #[test] fn f0() {} | ||
| #[test] fn f1() {} | ||
| #[test] fn f2() {} | ||
| #[test] fn f3() {} | ||
| #[test] fn f4() {} | ||
| #[test] fn f5() {} | ||
| #[test] fn f6() {} | ||
| #[test] fn f7() {} | ||
| #[test] fn f8() {} | ||
| #[test] fn f9() {} | ||
| #[test] fn f10() {} | ||
| #[test] fn f11() {} | ||
| #[test] fn f12() {} | ||
| #[test] fn f13() {} | ||
| #[test] fn f14() {} | ||
| #[test] fn f15() {} | ||
| #[test] fn f16() {} | ||
| #[test] fn f17() {} | ||
| #[test] fn f18() {} | ||
| #[test] fn f19() {} | ||
| #[test] fn f20() {} | ||
| #[test] fn f21() {} | ||
| #[test] fn f22() {} | ||
| #[test] fn f23() {} | ||
| #[test] fn f24() {} | ||
| #[test] fn f25() {} | ||
| #[test] fn f26() {} | ||
| #[test] fn f27() {} | ||
| #[test] fn f28() {} | ||
| #[test] fn f29() {} | ||
| #[test] fn f30() {} | ||
| #[test] fn f31() {} | ||
| #[test] fn f32() {} | ||
| #[test] fn f33() {} | ||
| #[test] fn f34() {} | ||
| #[test] fn f35() {} | ||
| #[test] fn f36() {} | ||
| #[test] fn f37() {} | ||
| #[test] fn f38() {} | ||
| #[test] fn f39() {} | ||
| #[test] fn f40() {} | ||
| #[test] fn f41() {} | ||
| #[test] fn f42() {} | ||
| #[test] fn f43() {} | ||
| #[test] fn f44() {} | ||
| #[test] fn f45() {} | ||
| #[test] fn f46() {} | ||
| #[test] fn f47() {} | ||
| #[test] fn f48() {} | ||
| #[test] fn f49() {} | ||
| #[test] fn f50() {} | ||
| #[test] fn f51() {} | ||
| #[test] fn f52() {} | ||
| #[test] fn f53() {} | ||
| #[test] fn f54() {} | ||
| #[test] fn f55() {} | ||
| #[test] fn f56() {} | ||
| #[test] fn f57() {} | ||
| #[test] fn f58() {} | ||
| #[test] fn f59() {} | ||
| #[test] fn f60() {} | ||
| #[test] fn f61() {} | ||
| #[test] fn f62() {} | ||
| #[test] fn f63() {} | ||
| #[test] fn f64() {} | ||
| #[test] fn f65() {} | ||
| #[test] fn f66() {} | ||
| #[test] fn f67() {} | ||
| #[test] fn f68() {} | ||
| #[test] fn f69() {} | ||
| #[test] fn f70() {} | ||
| #[test] fn f71() {} | ||
| #[test] fn f72() {} | ||
| #[test] fn f73() {} | ||
| #[test] fn f74() {} | ||
| #[test] fn f75() {} | ||
| #[test] fn f76() {} | ||
| #[test] fn f77() {} | ||
| #[test] fn f78() {} | ||
| #[test] fn f79() {} | ||
| #[test] fn f80() {} | ||
| #[test] fn f81() {} | ||
| #[test] fn f82() {} | ||
| #[test] fn f83() {} | ||
| #[test] fn f84() {} | ||
| #[test] fn f85() {} | ||
| #[test] fn f86() {} | ||
| #[test] fn f87() {} | ||
| #[test] fn f88() {} | ||
| #[test] fn f89() {} | ||
| #[test] fn f90() {} | ||
| #[test] fn f91() {} | ||
| #[test] fn f92() {} | ||
| #[test] fn f93() {} | ||
| #[test] fn f94() {} | ||
| #[test] fn f95() {} | ||
| #[test] fn f96() {} | ||
| #[test] fn f97() {} | ||
| #[test] fn f98() {} | ||
| #[test] fn f99() {} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
|
|
||
| running 103 tests | ||
| abc --- FAILED | ||
| ....................................................................................... 88/103 | ||
| ............. 101/103 | ||
| foo --- FAILED | ||
| foo2 --- FAILED | ||
|
|
||
| failures: | ||
|
|
||
| ---- abc stdout ---- | ||
| thread 'abc' panicked at $DIR/terse.rs:12:5: | ||
| explicit panic | ||
| note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace | ||
|
|
||
| ---- foo stdout ---- | ||
| thread 'foo' panicked at $DIR/terse.rs:17:5: | ||
| explicit panic | ||
|
|
||
| ---- foo2 stdout ---- | ||
| thread 'foo2' panicked at $DIR/terse.rs:22:5: | ||
| explicit panic | ||
|
|
||
|
|
||
| failures: | ||
| abc | ||
| foo | ||
| foo2 | ||
|
|
||
| test result: FAILED. 100 passed; 3 failed; 0 ignored; 0 measured; 0 filtered out; finished in $TIME | ||
|
|