failing tests aren't that noisy by default

Orion Gonzalez · Orion Gonzalez · commit 86861f311874 · 2024-10-24T14:51:23.000+02:00
diff --git a/src/tools/compiletest/src/runtest.rs b/src/tools/compiletest/src/runtest.rs
@@ -763,7 +763,9 @@ impl<'test> TestCx<'test> {
                 unexpected.len(),
                 not_found.len()
             ));
-            println!("status: {}\ncommand: {}\n", proc_res.status, proc_res.cmdline);
+            if env::var("COMPILETEST_DEBUG_INFO").is_ok() {
+                println!("status: {}\ncommand: {}\n", proc_res.status, proc_res.cmdline);
+            }
             if !unexpected.is_empty() {
                 println!("{}", "--- unexpected errors (from JSON output) ---".green());
                 for error in &unexpected {
@@ -2696,6 +2698,9 @@ impl ProcRes {
                 )
             }
         }
+        if env::var("COMPILETEST_DEBUG_INFO").is_err() {
+            return;
+        }
 
         println!(
             "status: {}\ncommand: {}\n{}\n{}\n",

Original file line number	Diff line number	Diff line change
`@@ -763,7 +763,9 @@ impl<'test> TestCx<'test> {`
`763`	`763`	`unexpected.len(),`
`764`	`764`	`not_found.len()`
`765`	`765`	`));`
`766`		`- println!("status: {}\ncommand: {}\n", proc_res.status, proc_res.cmdline);`
	`766`	`+ if env::var("COMPILETEST_DEBUG_INFO").is_ok() {`
	`767`	`+ println!("status: {}\ncommand: {}\n", proc_res.status, proc_res.cmdline);`
	`768`	`+ }`
`767`	`769`	`if !unexpected.is_empty() {`
`768`	`770`	`println!("{}", "--- unexpected errors (from JSON output) ---".green());`
`769`	`771`	`for error in &unexpected {`
`@@ -2696,6 +2698,9 @@ impl ProcRes {`
`2696`	`2698`	`)`
`2697`	`2699`	`}`
`2698`	`2700`	`}`
	`2701`	`+ if env::var("COMPILETEST_DEBUG_INFO").is_err() {`
	`2702`	`+ return;`
	`2703`	`+ }`
`2699`	`2704`
`2700`	`2705`	`println!(`
`2701`	`2706`	`"status: {}\ncommand: {}\n{}\n{}\n",`