[nextest-runner] handle SIGINFO and SIGUSR1 to dump info to screen

Author: sunshowers

nextest-runner/src/errors.rs

@@ -6,10 +6,9 @@
 use crate::{
     cargo_config::{TargetTriple, TargetTripleSource},
     config::{ConfigExperimental, CustomTestGroup, ScriptId, TestGroup},
-    helpers::{dylib_path_envvar, extract_abort_status},
+    helpers::{display_exit_status, dylib_path_envvar},
     redact::Redactor,
     reuse_build::{ArchiveFormat, ArchiveStep},
-    runner::AbortStatus,
     target_runner::PlatformRunnerSource,
 };
 use camino::{FromPathBufError, Utf8Path, Utf8PathBuf};
@@ -958,30 +957,6 @@ impl CreateTestListError {
     }
 }
 
-fn display_exit_status(exit_status: ExitStatus) -> String {
-    match extract_abort_status(exit_status) {
-        #[cfg(unix)]
-        Some(AbortStatus::UnixSignal(sig)) => match crate::helpers::signal_str(sig) {
-            Some(s) => {
-                format!("signal {sig} (SIG{s})")
-            }
-            None => {
-                format!("signal {sig}")
-            }
-        },
-        #[cfg(windows)]
-        Some(AbortStatus::WindowsNtStatus(nt_status)) => {
-            format!("code {}", crate::helpers::display_nt_status(nt_status))
-        }
-        None => match exit_status.code() {
-            Some(code) => {
-                format!("code {code}")
-            }
-            None => "an unknown error".to_owned(),
-        },
-    }
-}
-
 /// An error that occurs while writing list output.
 #[derive(Debug, Error)]
 #[non_exhaustive]

nextest-runner/src/helpers.rs

@@ -313,6 +313,35 @@ pub(crate) fn extract_abort_status(exit_status: ExitStatus) -> Option<AbortStatu
     }
 }
 
+pub(crate) fn display_exit_status(exit_status: ExitStatus) -> String {
+    match extract_abort_status(exit_status) {
+        Some(abort_status) => display_abort_status(abort_status),
+        None => match exit_status.code() {
+            Some(code) => format!("exit code {}", code),
+            None => "an unknown error".to_owned(),
+        },
+    }
+}
+
+/// Display the abort status.
+pub(crate) fn display_abort_status(abort_status: AbortStatus) -> String {
+    match abort_status {
+        #[cfg(unix)]
+        AbortStatus::UnixSignal(sig) => match crate::helpers::signal_str(sig) {
+            Some(s) => {
+                format!("signal {sig} (SIG{s})")
+            }
+            None => {
+                format!("signal {sig}")
+            }
+        },
+        #[cfg(windows)]
+        AbortStatus::WindowsNtStatus(nt_status) => {
+            format!("code {}", crate::helpers::display_nt_status(nt_status))
+        }
+    }
+}
+
 #[cfg(unix)]
 pub(crate) fn signal_str(signal: i32) -> Option<&'static str> {
     // These signal numbers are the same on at least Linux, macOS, FreeBSD and illumos.

nextest-runner/src/list/test_list.rs

@@ -990,7 +990,7 @@ impl<'a> TestInstance<'a> {
     /// Return an identifier for test instances, including being able to sort
     /// them.
     #[inline]
-    pub(crate) fn id(&self) -> TestInstanceId<'a> {
+    pub fn id(&self) -> TestInstanceId<'a> {
         TestInstanceId {
             binary_id: &self.suite_info.binary_id,
             test_name: self.name,
@@ -1060,7 +1060,7 @@ impl<'a> TestInstance<'a> {
 ///
 /// Returned by [`TestInstance::id`].
 #[derive(Clone, Copy, Debug, Eq, PartialEq, Ord, PartialOrd)]
-pub(crate) struct TestInstanceId<'a> {
+pub struct TestInstanceId<'a> {
     /// The binary ID.
     pub binary_id: &'a RustBinaryId,
 

nextest-runner/src/reporter/aggregator.rs

@@ -65,7 +65,10 @@ impl<'cfg> MetadataJunit<'cfg> {
             | TestEventKind::RunContinued { .. } => {}
             TestEventKind::SetupScriptStarted { .. }
             | TestEventKind::SetupScriptSlow { .. }
-            | TestEventKind::SetupScriptFinished { .. } => {}
+            | TestEventKind::SetupScriptFinished { .. }
+            | TestEventKind::InfoStarted { .. }
+            | TestEventKind::InfoResponse { .. }
+            | TestEventKind::InfoFinished { .. } => {}
             TestEventKind::TestStarted { .. } => {}
             TestEventKind::TestSlow { .. } => {}
             TestEventKind::TestAttemptFailedWillRetry { .. }