bootstrap: Explicitly mark the end of a failed test's captured output

[Zalathar]

While working on some compiletest stuff, I noticed that when bootstrap prints a failed test's captured output, there's no indication of where that output actually ends.

In addition to indicating where the captured output ends, this end marker also makes it easier to see the relevant test name when scrolling upwards in terminal output.

[rustbot]

r? @jieyouxu

rustbot has assigned @jieyouxu.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[Zalathar]

Relevant output snippet before and after the change:

BEFORE

failures:

---- [coverage-run] tests/coverage/trivial.rs stdout ----
Saved the actual coverage to `/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/test/coverage/trivial.coverage-run/trivial.coverage`
diff of coverage:

1	   LL|       |//@ edition: 2021
-	   LL|       |
+	   LL|       |// (fake change)
3	   LL|      1|fn main() {}
4	
5	


The actual coverage differed from the expected coverage

error: an error occurred comparing coverage output.
status: exit status: 0
command: "/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/llvm/bin/llvm-cov" "show" "--format=text" "--show-line-counts-or-regions" "--Xdemangler" "/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/stage1-tools-bin/coverage-dump" "--Xdemangler" "--demangle" "--instr-profile" "/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/test/coverage/trivial.coverage-run/default.profdata" "--object" "/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/test/coverage/trivial.coverage-run/a"
--- stdout -------------------------------
    1|       |//@ edition: 2021
    2|       |// (fake change)
    3|      1|fn main() {}
------------------------------------------
stderr: none



failures:
    [coverage-run] tests/coverage/trivial.rs

AFTER

failures:

---- [coverage-run] tests/coverage/trivial.rs stdout ----
Saved the actual coverage to `/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/test/coverage/trivial.coverage-run/trivial.coverage`
diff of coverage:

1	   LL|       |//@ edition: 2021
-	   LL|       |
+	   LL|       |// (fake change)
3	   LL|      1|fn main() {}
4	
5	


The actual coverage differed from the expected coverage

error: an error occurred comparing coverage output.
status: exit status: 0
command: "/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/llvm/bin/llvm-cov" "show" "--format=text" "--show-line-counts-or-regions" "--Xdemangler" "/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/stage1-tools-bin/coverage-dump" "--Xdemangler" "--demangle" "--instr-profile" "/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/test/coverage/trivial.coverage-run/default.profdata" "--object" "/Users/stuart/Dev/rust/rust/build/aarch64-apple-darwin/test/coverage/trivial.coverage-run/a"
--- stdout -------------------------------
    1|       |//@ edition: 2021
    2|       |// (fake change)
    3|      1|fn main() {}
------------------------------------------
stderr: none

---- [coverage-run] tests/coverage/trivial.rs stdout end ----

failures:
    [coverage-run] tests/coverage/trivial.rs

[hkBst]

lgtm

View changes since this review

[jieyouxu]

Thanks

View changes since this review

[jieyouxu]

@bors r+ rollup

[bors]

📌 Commit 21de27c has been approved by jieyouxu

It is now in the queue for this repository.