Added chenges from PR review

aurora-is-near · Aug 13, 2024 · 9a74881 · 9a74881
1 parent 4945df0
commit 9a74881
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/engine-tests/src/tests/standalone/call_tracer.rs b/engine-tests/src/tests/standalone/call_tracer.rs
@@ -2,6 +2,7 @@ use crate::prelude::{H160, H256};
 use crate::utils::solidity::erc20::{ERC20Constructor, ERC20};
 use crate::utils::{self, standalone, Signer};
 use aurora_engine_modexp::AuroraModExp;
+use aurora_engine_types::parameters::engine::{EvmErrorKind, TransactionStatus};
 use aurora_engine_types::{
     parameters::{CrossContractCallArgs, PromiseArgs, PromiseCreateArgs},
     storage,
@@ -296,9 +297,10 @@ fn test_contract_create_too_large() {
     let standalone_result = sputnik::traced_call(&mut listener, || {
         runner.submit_transaction(&signer.secret_key, tx)
     });
-    // As we charge users for the EVM errors, we should charge them for
-    // the gas used to deploy the large contract.
-    assert!(standalone_result.is_ok());
+    assert!(matches!(
+        standalone_result.unwrap().status,
+        TransactionStatus::Error(EvmErrorKind::CreateContractLimit)
+    ));
 }
 
 #[allow(clippy::too_many_lines)]

diff --git a/engine-tests/src/utils/solidity/exit_precompile.rs b/engine-tests/src/utils/solidity/exit_precompile.rs
@@ -95,7 +95,7 @@ impl Tester {
             aurora_engine::parameters::TransactionStatus::Succeed(_) => Ok(result),
             aurora_engine::parameters::TransactionStatus::Revert(_bytes) => Err(Revert),
             aurora_engine::parameters::TransactionStatus::Error(err) => {
-                panic!("Unexpected status {err:?}")
+                panic!("Unexpected status {:?}", err.as_ref())
             }
         }
     }