tests: Improve integration test output on assertion failure

The child process output was only being printed on process failure, not later test assertions. Printing the output unconditionally allows it to be seen on any failure without noisy --nocapture. Signed-off-by: Lann Martin <[email protected]>
fermyon · Mar 7, 2023 · 1decdd2 · 1decdd2
1 parent 276e570
commit 1decdd2
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/tests/integration.rs b/tests/integration.rs
@@ -845,17 +845,20 @@ mod integration_tests {
  .collect::<Vec<String>>()
  .join(" "),
  );
+
+ cmd.env("RUST_LOG", "spin_cli=warn");
  if let Some(envs) = envs {
  for (k, v) in envs {
  cmd.env(k, v);
  }
  }
 
  let output = cmd.output()?;
+ println!("STDOUT:\n{}", String::from_utf8_lossy(&output.stdout));
+ println!("STDERR:\n{}", String::from_utf8_lossy(&output.stderr));
+
  let code = output.status.code().expect("should have status code");
  if code != 0 {
- println!("{:#?}", std::str::from_utf8(&output.stderr)?);
- println!("{:#?}", std::str::from_utf8(&output.stdout)?);
  panic!("command `{:?}` exited with code {}", cmd, code);
  }