Skip to content
This repository was archived by the owner on Apr 5, 2024. It is now read-only.

Commit 11f4dee

Browse files
committed
Always show output if "version bump" line not found
This avoids us hiding some errors, like we did with the failing rmvp testcase.
1 parent 40325ee commit 11f4dee

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

tests/full.rs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,14 +94,18 @@ mod full {
9494
.expect("could not read line from rust-semverver output")
9595
.trim_end();
9696

97+
let has_expected_line = stdout.lines().any(|l| l.starts_with("version bump"));
98+
9799
stdout
98100
.lines()
99101
.chain(stderr.lines())
100102
.map(|l| l.trim_end())
101103
.skip_while(|line|
102104
// skip everything before the first important bit of info
103105
!line.starts_with("version bump") &&
104-
// ...unless debugging is enabled
106+
// ...if we know it's in there at all
107+
has_expected_line &&
108+
// ...unless debugging is enabled, then always show
105109
!log_enabled!(Level::Debug))
106110
.map(|line| {
107111
// sanitize paths for reproducibility

0 commit comments

Comments
 (0)