chore: fail fast AFTER processing traces (#6628)

foundry-rs · Dec 20, 2023 · b4b691b · b4b691b
1 parent 70195e7
commit b4b691b
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/crates/forge/bin/cmd/test/mod.rs b/crates/forge/bin/cmd/test/mod.rs
@@ -407,11 +407,6 @@ impl TestArgs {
             for (name, result) in &mut tests {
                 short_test_result(name, result);
 
-                // If the test failed, we want to stop processing the rest of the tests
-                if self.fail_fast && result.status == TestStatus::Failure {
-                    break 'outer
-                }
-
                 // We only display logs at level 2 and above
                 if verbosity >= 2 {
                     // We only decode logs from Hardhat and DS-style console events
@@ -482,6 +477,11 @@ impl TestArgs {
                 if self.gas_report {
                     gas_report.analyze(&result.traces);
                 }
+
+                // If the test failed, we want to stop processing the rest of the tests
+                if self.fail_fast && result.status == TestStatus::Failure {
+                    break 'outer
+                }
             }
             let block_outcome = TestOutcome::new(
                 [(contract_name.clone(), suite_result)].into(),