Don't consider panic fixes as "new failures"

boa-dev · May 29, 2021 · 9152734 · 9152734
1 parent e7ffe7f
commit 9152734
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/boa_tester/src/results.rs b/boa_tester/src/results.rs
@@ -433,10 +433,14 @@ fn compute_result_diff(
                 | (TestOutcomeResult::Failed, TestOutcomeResult::Failed)
                 | (TestOutcomeResult::Panic, TestOutcomeResult::Panic) => {}
 
+                (TestOutcomeResult::Panic, TestOutcomeResult::Failed) => {
+                    final_diff.panic_fixes.push(test_name)
+                }
+
                 (_, TestOutcomeResult::Passed) => final_diff.fixed.push(test_name),
                 (_, TestOutcomeResult::Failed) => final_diff.broken.push(test_name),
                 (_, TestOutcomeResult::Panic) => final_diff.new_panics.push(test_name),
-                (TestOutcomeResult::Panic, _) => final_diff.panic_fixes.push(test_name),
+
                 _ => {}
             }
         }