Skip to content

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)