Merge pull request onflow#178 from m-Peter/properly-handle-evm-tx-result

Properly handle tx results from `EVM.run`
m-Peter · Mar 26, 2024 · cbfa7b4 · cbfa7b4
2 parents b97076e + 36eb1cb
commit cbfa7b4
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/services/requester/cadence/run.cdc b/services/requester/cadence/run.cdc
@@ -10,10 +10,12 @@ transaction(hexEncodedTx: String) {
     }
 
     execute {
-        let txResult = EVM.run(tx: hexEncodedTx.decodeHex(), coinbase: self.coa.address())
-        // todo only temporary until we correctly handle failure events
+        let txResult = EVM.run(
+            tx: hexEncodedTx.decodeHex(),
+            coinbase: self.coa.address()
+        )
         assert(
-            txResult.status == EVM.Status.successful,
+            txResult.status == EVM.Status.failed || txResult.status == EVM.Status.successful,
             message: "failed to execute evm transaction: ".concat(txResult.errorCode.toString())
         )
     }