Skip to content

Commit 75c06bb

Browse files
authored
Merge pull request rust-lang#2361 from senekor/senekor/mxstspkxoyyp
Improve error messages if tests fail
2 parents 7e5793b + c163bfe commit 75c06bb

1 file changed

Lines changed: 12 additions & 10 deletions

File tree

tests/integration_tests.rs

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -61,30 +61,32 @@ impl<'a> Cmd<'a> {
6161

6262
cmd.args(self.args).stdin(Stdio::null());
6363

64-
let status = match self.output {
65-
None => cmd
66-
.stdout(Stdio::null())
67-
.stderr(Stdio::null())
68-
.status()
69-
.unwrap(),
64+
let output = match self.output {
65+
None => cmd.output().unwrap(),
7066
Some(FullStdout(stdout)) => {
7167
let output = cmd.stderr(Stdio::null()).output().unwrap();
7268
assert_eq!(from_utf8(&output.stdout).unwrap(), stdout);
73-
output.status
69+
output
7470
}
7571
Some(PartialStdout(stdout)) => {
7672
let output = cmd.stderr(Stdio::null()).output().unwrap();
7773
assert!(from_utf8(&output.stdout).unwrap().contains(stdout));
78-
output.status
74+
output
7975
}
8076
Some(PartialStderr(stderr)) => {
8177
let output = cmd.stdout(Stdio::null()).output().unwrap();
8278
assert!(from_utf8(&output.stderr).unwrap().contains(stderr));
83-
output.status
79+
output
8480
}
8581
};
8682

87-
assert_eq!(status.success(), success, "{cmd:?}");
83+
assert_eq!(
84+
output.status.success(),
85+
success,
86+
"{cmd:?}\n\nstdout:\n{}\nstderr:\n{}",
87+
from_utf8(&output.stdout).unwrap(),
88+
from_utf8(&output.stderr).unwrap(),
89+
);
8890
}
8991

9092
#[inline]

0 commit comments

Comments
 (0)