add long_name for ResultName

Author: zeegomo

src/report/display.rs

@@ -4,6 +4,7 @@ use crate::results::{BrokenReason, FailureReason, TestResult};
 
 pub trait ResultName {
     fn name(&self) -> String;
+    fn long_name(&self) -> String;
 }
 
 impl ResultName for FailureReason {
@@ -17,6 +18,16 @@ impl ResultName for FailureReason {
             FailureReason::DependsOn(_) => "faulty deps".into(),
         }
     }
+
+    fn long_name(&self) -> String {
+        match self {
+            FailureReason::CompilerError(_) | FailureReason::DependsOn(_) => self.to_string(),
+            FailureReason::Unknown
+            | FailureReason::Timeout
+            | FailureReason::OOM
+            | FailureReason::ICE => self.name(),
+        }
+    }
 }
 
 impl ResultName for BrokenReason {
@@ -28,6 +39,10 @@ impl ResultName for BrokenReason {
             BrokenReason::MissingGitRepository => "missing repo".into(),
         }
     }
+
+    fn long_name(&self) -> String {
+        self.name()
+    }
 }
 
 impl ResultName for TestResult {
@@ -42,6 +57,18 @@ impl ResultName for TestResult {
             TestResult::Skipped => "skipped".into(),
         }
     }
+
+    fn long_name(&self) -> String {
+        match self {
+            TestResult::BuildFail(reason) => format!("build {}", reason.long_name()),
+            TestResult::TestFail(reason) => format!("test {}", reason.long_name()),
+            TestResult::BrokenCrate(reason) => reason.long_name(),
+            TestResult::TestSkipped
+            | TestResult::TestPass
+            | TestResult::Error
+            | TestResult::Skipped => self.name(),
+        }
+    }
 }
 
 #[derive(Serialize)]

src/report/html.rs

@@ -5,7 +5,7 @@ use crate::report::{
     analyzer::ReportCrates, archives::Archive, Color, Comparison, CrateResult, ReportWriter,
     ResultColor, ResultName, TestResults,
 };
-use crate::results::{EncodingType, FailureReason, TestResult};
+use crate::results::EncodingType;
 use indexmap::IndexMap;
 
 #[derive(Serialize)]
@@ -170,12 +170,7 @@ fn write_report<W: ReportWriter>(
                         .into_iter()
                         .map(|(res, krates)| {
                             (
-                                if let TestResult::BuildFail(FailureReason::CompilerError(_)) = res
-                                {
-                                    res.to_string()
-                                } else {
-                                    res.name()
-                                },
+                                res.long_name(),
                                 krates
                                     .into_iter()
                                     .map(|result| to_html_crate_result(result))