Always return error from dry-run with highest allowed gas limit

m-Peter · m-Peter · commit 227f57ea17b3 · 2024-11-27T10:38:55.000+02:00
diff --git a/services/requester/requester.go b/services/requester/requester.go
@@ -404,6 +404,7 @@ func (e *EVM) EstimateGas(
 		if resultSummary.ErrorCode == evmTypes.ExecutionErrCodeExecutionReverted {
 			return 0, errs.NewRevertError(resultSummary.ReturnedData)
 		}
+		return 0, errs.NewFailedTransactionError(resultSummary.ErrorMessage)
 	}
 
 	// For almost any transaction, the gas consumed by the unconstrained execution

Original file line number	Diff line number	Diff line change
`@@ -404,6 +404,7 @@ func (e *EVM) EstimateGas(`
`404`	`404`	`if resultSummary.ErrorCode == evmTypes.ExecutionErrCodeExecutionReverted {`
`405`	`405`	`return 0, errs.NewRevertError(resultSummary.ReturnedData)`
`406`	`406`	`}`
	`407`	`+ return 0, errs.NewFailedTransactionError(resultSummary.ErrorMessage)`
`407`	`408`	`}`
`408`	`409`
`409`	`410`	`// For almost any transaction, the gas consumed by the unconstrained execution`