ninja-xtask: show stdout and exit status for failed commands

ninja-xtask/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog ninja-xtask
 
+## v0.2.6
+
+- show stdout and exit status for failed commands
+
 ## v0.2.4 & 0.2.5
 
 - fix build workflow for publishing binaries

ninja-xtask/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "ninja-xtask"
-version = "0.2.5"
+version = "0.2.6"
 edition = "2024"
 readme = "README.md"
 description = "xtask utilities that I use in most projects"

ninja-xtask/src/lib.rs

@@ -93,8 +93,12 @@ impl From<Cmd> for Exit<()> {
                     println!("{}: OK", cmd.name);
                     Self::Ok(())
                 } else {
+                    let stdout = String::from_utf8_lossy(&output.stdout);
                     let stderr = String::from_utf8_lossy(&output.stderr);
-                    Self::Error(stderr.to_string())
+                    Self::Error(format!(
+                        "====== {} exited with {} ======\n-- stdout: --\n{}\n\n-- stderr: --\n{}",
+                        cmd.name, output.status, stdout, stderr
+                    ))
                 }
             }
             Err(e) => {

ninja-xtask/tests/fixture/src/lib.rs

@@ -1 +1,11 @@
-enum foo { bar }
+enum foo { bar }
+
+#[cfg(test)]
+mod tests {
+    #[test]
+    fn print_and_panic() {
+        println!("test printed to stdout");
+        dbg!("test dbg");
+        assert_eq!(5,7);
+    }
+}

ninja-xtask/tests/test_fmt.rs → ninja-xtask/tests/test_commands.rs

@@ -1,7 +1,10 @@
-use std::fs;
+use std::{fs, path::PathBuf};
 
 use dircpy::copy_dir;
-use ninja_xtask::commands::fmt;
+use ninja_xtask::{
+    Exit,
+    commands::{fmt, test},
+};
 use tempfile::tempdir;
 
 #[test]
@@ -23,3 +26,18 @@ fn fmt_fixture() {
     assert_ne!(original, formatted);
     dbg!(tmp.path());
 }
+
+#[test]
+fn fail_output() {
+    let fixture = PathBuf::from("tests/fixture");
+    let run_tests = test(&fixture);
+    let exit = Exit::from(run_tests);
+    let Exit::Error(output) = exit else {
+        panic!("test didn't fail")
+    };
+    assert!(output.contains("====== tests exited with"));
+    assert!(output.contains("test printed to stdout"));
+    assert!(output.contains("test dbg"));
+    assert!(output.contains("left: 5"));
+    println!("{output}");
+}