try to find a failure reason before falling back to error

Author: Skgland

src/runner/test.rs

@@ -15,7 +15,7 @@ use rustwide::{Build, PrepareError};
 use std::collections::{BTreeSet, HashMap, HashSet};
 use std::io::ErrorKind;
 
-fn failure_reason(err: &Error) -> FailureReason {
+pub(crate) fn failure_reason(err: &Error) -> FailureReason {
     if let Some(reason) = err.downcast_ref::<FailureReason>() {
         reason.clone()
     } else if let Some(command_error) = err.downcast_ref::<CommandError>() {

src/runner/worker.rs

@@ -4,7 +4,7 @@ use crate::experiments::{Experiment, Mode};
 use crate::prelude::*;
 use crate::results::{BrokenReason, TestResult};
 use crate::runner::tasks::{Task, TaskStep};
-use crate::runner::test::detect_broken;
+use crate::runner::test::{detect_broken, failure_reason};
 use crate::runner::OverrideResult;
 use crate::toolchain::Toolchain;
 use crate::utils;
@@ -244,7 +244,10 @@ impl<'a> Worker<'a> {
                 let mut result = if self.config.is_broken(&krate) {
                     TestResult::BrokenCrate(BrokenReason::Unknown)
                 } else {
-                    TestResult::Error
+                    match failure_reason(&err) {
+                        crate::results::FailureReason::Unknown => TestResult::Error,
+                        reason => TestResult::BuildFail(reason),
+                    }
                 };
 
                 if let Some(OverrideResult(res)) = err.downcast_ref() {