fix: only identify traces if they should be included

Author: DaniPopes

crates/forge/bin/cmd/test/mod.rs

@@ -615,9 +615,6 @@ async fn test(
             // Decode the traces
             let mut decoded_traces = Vec::with_capacity(result.traces.len());
             for (kind, trace) in &mut result.traces {
-                decoder.identify(trace, &mut local_identifier);
-                decoder.identify(trace, &mut etherscan_identifier);
-
                 // Verbosity levels
                 // - 3: only display traces for failed tests
                 // - 4: also display the setup trace for failed tests
@@ -632,9 +629,11 @@ async fn test(
                     TraceKind::Deployment => false,
                 };
 
-                // We decode the trace if we either need to build a gas report or we need
-                // to print it
+                // We decode the trace if we either need to build a gas report or we need to print
+                // it
                 if should_include || gas_reporting {
+                    decoder.identify(trace, &mut local_identifier);
+                    decoder.identify(trace, &mut etherscan_identifier);
                     decoder.decode(trace).await;
                 }