print path for diff test --keep-input option

diff test framework has a --keep-input option to keep the generated
trace, however it is often in some difficult to discover file in /tmp
like

Print the path to make the location obvious to the user

Author: cphlipot1

python/generators/diff_tests/runner.py

@@ -183,9 +183,10 @@ def _run_test(self, test: TestCase,
         gen_trace_file.close()
         os.remove(trace_path)
 
+    print_trace_path = gen_trace_file and self.config.keep_input
     run_str = self._process_test_result(result, trace_path,
                                         extension_descriptor_paths,
-                                        trace_descriptor_path)
+                                        trace_descriptor_path, print_trace_path)
     return test.name, run_str, result
 
   def _process_test_result(
@@ -194,6 +195,7 @@ def _process_test_result(
       trace_path: str,
       extension_descriptor_paths: List[str],
       trace_descriptor_path: str,
+      print_trace_path: bool,
   ) -> str:
     colors = ColorFormatter(self.config.no_colors)
 
@@ -219,6 +221,7 @@ def write_cmdlines() -> str:
       return res
 
     run_str = f"{colors.yellow('[ RUN      ]')} {result.test.name}\n"
+    run_diagnostics = [f"trace_path: {trace_path}"] if print_trace_path else []
     if result.exit_code != 0 or not result.passed:
       result.passed = False
       run_str += result.stderr
@@ -230,13 +233,15 @@ def write_cmdlines() -> str:
       else:
         run_str += write_cmdlines()
 
-      run_str += (f"{colors.red('[  FAILED  ]')} {result.test.name}\n")
+      run_str += (f"{colors.red('[  FAILED  ]')} {result.test.name}")
     else:
       assert result.perf_result
-      run_str += (
-          f"{colors.green('[       OK ]')} {result.test.name} "
-          f"(ingest: {result.perf_result.ingest_time_ns / 1000000:.2f} ms "
-          f"query: {result.perf_result.real_time_ns / 1000000:.2f} ms)\n")
+      run_str += f"{colors.green('[       OK ]')} {result.test.name}"
+      run_diagnostics.append(
+          f"ingest: {result.perf_result.ingest_time_ns / 1000000:.2f} ms")
+      run_diagnostics.append(
+          f"query: {result.perf_result.real_time_ns / 1000000:.2f} ms")
+    run_str += f" ({' '.join(run_diagnostics)})\n" if run_diagnostics else "\n"
     return run_str
 
   def _get_build_config(self) -> Set[str]: