[nextest-runner] update display of Windows aborts slightly

nextest-runner/src/helpers.rs

@@ -320,7 +320,11 @@ pub(crate) fn display_abort_status(abort_status: AbortStatus) -> String {
         },
         #[cfg(windows)]
         AbortStatus::WindowsNtStatus(nt_status) => {
-            format!("code {}", crate::helpers::display_nt_status(nt_status))
+            format!(
+                "code {}",
+                // TODO: pass down a style here
+                crate::helpers::display_nt_status(nt_status, Style::new())
+            )
         }
     }
 }
@@ -351,18 +355,21 @@ pub(crate) fn signal_str(signal: i32) -> Option<&'static str> {
 }
 
 #[cfg(windows)]
-pub(crate) fn display_nt_status(nt_status: windows_sys::Win32::Foundation::NTSTATUS) -> String {
+pub(crate) fn display_nt_status(
+    nt_status: windows_sys::Win32::Foundation::NTSTATUS,
+    bold_style: Style,
+) -> String {
+    let bolded_status = format!("{:#x}", nt_status.style(bold_style));
     // Convert the NTSTATUS to a Win32 error code.
     let win32_code = unsafe { windows_sys::Win32::Foundation::RtlNtStatusToDosError(nt_status) };
 
     if win32_code == windows_sys::Win32::Foundation::ERROR_MR_MID_NOT_FOUND {
         // The Win32 code was not found.
-        return format!("{nt_status:#x} ({nt_status})");
+        return format!("{bolded_status} ({nt_status})");
     }
 
     format!(
-        "{:#x}: {}",
-        nt_status,
+        "{bolded_status}: {}",
         io::Error::from_raw_os_error(win32_code as i32)
     )
 }

nextest-runner/src/reporter/displayer.rs

@@ -1323,7 +1323,7 @@ impl<'a> TestReporterImpl<'a> {
             leaked: _,
         } = last_status.result
         {
-            self.write_windows_message_line(nt_status, writer)?;
+            write_windows_message_line(nt_status, &self.styles, writer)?;
         }
 
         Ok(())
@@ -1397,7 +1397,7 @@ impl<'a> TestReporterImpl<'a> {
             leaked: _,
         } = last_status.result
         {
-            self.write_windows_message_line(nt_status, writer)?;
+            write_windows_message_line(nt_status, &self.styles, writer)?;
         }
 
         Ok(())
@@ -1815,23 +1815,6 @@ impl<'a> TestReporterImpl<'a> {
         write!(writer, "[>{:>7.3?}s] ", duration.as_secs_f64())
     }
 
-    #[cfg(windows)]
-    fn write_windows_message_line(
-        &self,
-        nt_status: windows_sys::Win32::Foundation::NTSTATUS,
-        writer: &mut dyn Write,
-    ) -> io::Result<()> {
-        write!(writer, "{:>12} ", "Message".style(self.styles.fail))?;
-        write!(writer, "[         ] ")?;
-        writeln!(
-            writer,
-            "code {}",
-            crate::helpers::display_nt_status(nt_status)
-        )?;
-
-        Ok(())
-    }
-
     fn write_setup_script_execute_status(
         &self,
         script_id: &ScriptId,
@@ -2523,6 +2506,25 @@ fn short_status_str(result: ExecutionResult) -> Cow<'static, str> {
     }
 }
 
+#[cfg(windows)]
+fn write_windows_message_line(
+    nt_status: windows_sys::Win32::Foundation::NTSTATUS,
+    styles: &Styles,
+    writer: &mut dyn Write,
+) -> io::Result<()> {
+    // The code is typically of the form 0xC0000XXX, i.e. 10 characters long.
+    // (The high bit is always set in this code path.)
+    //
+    // In general we want to align to 24 characters, so "with code" is 24 - 10 =
+    // 14 characters.
+    writeln!(
+        writer,
+        "{:>14} {}",
+        "with code".style(styles.fail),
+        crate::helpers::display_nt_status(nt_status, styles.count)
+    )
+}
+
 fn write_final_warnings(
     final_stats: FinalRunStats,
     cancel_status: Option<CancelReason>,
@@ -3488,6 +3490,20 @@ mod tests {
         );
     }
 
+    #[cfg(windows)]
+    #[test]
+    fn test_write_windows_message_line() {
+        unsafe {
+            // Set the thread UI language to US English for consistent output.
+            windows_sys::Win32::Globalization::SetThreadUILanguage(0x0409);
+        }
+
+        let mut buf = Vec::new();
+        write_windows_message_line(0xc000013a_u32 as i32, &Styles::default(), &mut buf).unwrap();
+        let buf = String::from_utf8(buf).unwrap();
+        insta::assert_snapshot!(buf, @"");
+    }
+
     fn make_split_output(
         result: Option<ExecutionResult>,
         stdout: &str,

nextest-runner/src/reporter/error_description.rs

@@ -166,7 +166,9 @@ impl fmt::Display for UnitAbortDescription {
                 write!(
                     f,
                     " with code {}",
-                    crate::helpers::display_nt_status(exception)
+                    // TODO: pass in bold style (probably need to not use
+                    // fmt::Display)
+                    crate::helpers::display_nt_status(exception, owo_colors::Style::new())
                 )?;
             }
         }