Skip to content

Commit a55163d

Browse files
authored
Merge pull request #874 from onflow/mpeter/tx-with-evm-error-code
Include `errorCode` from EVM `txResult` in Cadence assert message
2 parents 742085b + 7cda9d4 commit a55163d

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

services/requester/cadence/batch_run.cdc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ transaction(hexEncodedTxs: [String], coinbase: String) {
2828
for txResult in txResults {
2929
assert(
3030
txResult.status == EVM.Status.failed || txResult.status == EVM.Status.successful,
31-
message: "evm_error=".concat(txResult.errorMessage).concat("\n")
31+
message: "evm_error=\(txResult.errorMessage);evm_error_code=\(txResult.errorCode)"
3232
)
3333
}
3434
}

services/requester/cadence/run.cdc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ transaction(hexEncodedTx: String, coinbase: String) {
88
)
99
assert(
1010
txResult.status == EVM.Status.failed || txResult.status == EVM.Status.successful,
11-
message: "evm_error=".concat(txResult.errorMessage).concat("\n")
11+
message: "evm_error=\(txResult.errorMessage);evm_error_code=\(txResult.errorCode)"
1212
)
1313
}
1414
}

services/requester/tx_pool.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,7 @@ import (
99
errs "github.com/onflow/flow-evm-gateway/models/errors"
1010
)
1111

12-
const (
13-
evmErrorRegex = `evm_error=(.*)\n`
14-
)
12+
const evmErrorRegex = `evm_error=(.*);`
1513

1614
// TxPool is the minimum interface that needs to be implemented by
1715
// the various transaction pool strategies.

0 commit comments

Comments
 (0)