Improve stack traces test error when transaction reverts

NomicFoundation · Sep 12, 2024 · ddadd4c · ddadd4c
1 parent af56e77
commit ddadd4c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/hardhat-core/test/internal/hardhat-network/stack-traces/test.ts b/packages/hardhat-core/test/internal/hardhat-network/stack-traces/test.ts
@@ -544,16 +544,14 @@ async function runTest(
       );
     }
 
-    compareConsoleLogs(logger.lines, tx.consoleLogs);
-
     const vmTraceDecoder = (provider as any)._vmTraceDecoder as VmTraceDecoder;
     const decodedTrace = vmTraceDecoder.tryToDecodeMessageTrace(trace);
 
     try {
       if (tx.stackTrace === undefined) {
         assert.isFalse(
           trace.exit.isError(),
-          `Transaction ${txIndex} shouldn't have failed`
+          `Transaction ${txIndex} shouldn't have failed (${trace.exit.getReason()})`
         );
       } else {
         assert.isDefined(
@@ -588,6 +586,8 @@ async function runTest(
         throw err;
       }
     }
+
+    compareConsoleLogs(logger.lines, tx.consoleLogs);
   }
 }