internal: output failure in json format

moonbitlang · Aug 26, 2024 · 840ce7d · 840ce7d
1 parent a29ee09
commit 840ce7d
Show file tree

Hide file tree

Showing 4 changed files with 34 additions and 15 deletions.
diff --git a/crates/moon/src/cli/generate_test_driver.rs b/crates/moon/src/cli/generate_test_driver.rs
@@ -125,6 +125,7 @@ pub fn generate_test_driver(
             filter_file: filter_file.clone(),
             filter_index,
             limit: 256,
+            test_failure_json: false,
         }),
         fmt_opt: None,
         sort_input,

diff --git a/crates/moon/src/cli/test.rs b/crates/moon/src/cli/test.rs
@@ -80,6 +80,9 @@ pub struct TestSubcommand {
 
     #[clap(long)]
     pub no_parallelize: bool,
+
+    #[clap(long)]
+    pub test_failure_json: bool,
 }
 
 pub fn run_test(cli: UniversalFlags, cmd: TestSubcommand) -> anyhow::Result<i32> {
@@ -185,6 +188,7 @@ fn run_test_internal(
             filter_file: filter_file.clone(),
             filter_index,
             limit,
+            test_failure_json: cmd.test_failure_json,
         }),
         sort_input,
         run_mode,

diff --git a/crates/moonbuild/src/entry.rs b/crates/moonbuild/src/entry.rs
@@ -546,6 +546,11 @@ async fn handle_test_result(
     target_dir: &Path,
     printed: Arc<AtomicBool>,
 ) -> anyhow::Result<()> {
+    let output_failure_in_json = moonbuild_opt
+        .test_opt
+        .as_ref()
+        .map(|it| it.test_failure_json)
+        .unwrap_or(false);
     for item in test_res_for_cur_pkg {
         match item {
             Ok(ok_ts) => {
@@ -626,27 +631,35 @@ async fn handle_test_result(
                 );
             }
             Err(TestFailedStatus::RuntimeError(err_ts) | TestFailedStatus::Failed(err_ts)) => {
-                println!(
-                    "test {}/{}::{} {}: {}",
-                    err_ts.package,
-                    err_ts.filename,
-                    err_ts.test_name,
-                    "failed".bold().red(),
-                    err_ts.message,
-                );
+                if output_failure_in_json {
+                    println!("{}", serde_json_lenient::to_string(err_ts)?);
+                } else {
+                    println!(
+                        "test {}/{}::{} {}: {}",
+                        err_ts.package,
+                        err_ts.filename,
+                        err_ts.test_name,
+                        "failed".bold().red(),
+                        err_ts.message,
+                    );
+                }
             }
             Err(TestFailedStatus::Others(e)) => {
                 eprintln!("{}: {}", "failed".red(), e);
             }
             Err(TestFailedStatus::ExpectTestFailed(origin_err)) => {
                 if !auto_update {
-                    println!(
-                        "test {}/{}::{} {}",
-                        origin_err.package,
-                        origin_err.filename,
-                        origin_err.test_name,
-                        "failed".bold().red(),
-                    );
+                    if output_failure_in_json {
+                        println!("{}", serde_json_lenient::to_string(&origin_err)?);
+                    } else {
+                        println!(
+                            "test {}/{}::{} {}",
+                            origin_err.package,
+                            origin_err.filename,
+                            origin_err.test_name,
+                            "failed".bold().red(),
+                        );
+                    }
                     let _ = crate::expect::render_expect_fail(&origin_err.message);
                 }
                 if auto_update {

diff --git a/crates/moonutil/src/common.rs b/crates/moonutil/src/common.rs
@@ -361,6 +361,7 @@ pub struct TestOpt {
     pub filter_file: Option<String>,
     pub filter_index: Option<u32>,
     pub limit: u32,
+    pub test_failure_json: bool,
 }
 
 impl TestOpt {