feat: fix bugs in the tests and remove debug prints

starkware-libs · Oct 26, 2024 · b38c400 · b38c400
1 parent 4fe754e
commit b38c400
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/crates/blockifier/src/execution/execution_utils.rs b/crates/blockifier/src/execution/execution_utils.rs
@@ -85,9 +85,6 @@ pub fn execute_entry_point_call_wrapper(
     let current_tracked_resource =
         context.tracked_resource_stack.pop().expect("Unexpected empty tracked resource.");
 
-    println!("enable_reverts: {:?}", context.versioned_constants().enable_reverts);
-    println!("res: {:?}", res);
-
     match res {
         Ok(call_info) => {
             if call_info.execution.failed && !context.versioned_constants().enable_reverts {

diff --git a/crates/blockifier/src/execution/native/syscall_handler.rs b/crates/blockifier/src/execution/native/syscall_handler.rs
@@ -174,7 +174,11 @@ impl<'state> StarknetSyscallHandler for &mut NativeSyscallHandler<'state> {
         deploy_from_zero: bool,
         remaining_gas: &mut u128,
     ) -> SyscallResult<(Felt, Vec<Felt>)> {
-        self.substract_syscall_gas_cost(remaining_gas, self.context.gas_costs().deploy_gas_cost)?;
+        self.substract_syscall_gas_cost(
+            remaining_gas,
+            SyscallSelector::Deploy,
+            self.context.gas_costs().deploy_gas_cost,
+        )?;
 
         let deployer_address = self.contract_address;
         let deployer_address_for_calculation =

diff --git a/crates/blockifier/src/execution/syscalls/syscall_tests/deploy.rs b/crates/blockifier/src/execution/syscalls/syscall_tests/deploy.rs
@@ -15,6 +15,7 @@ use crate::test_utils::initial_test_state::test_state;
 use crate::test_utils::{calldata_for_deploy_test, trivial_external_entry_point_new, CairoVersion};
 
 #[test_case(FeatureContract::TestContract(CairoVersion::Cairo1), 206800;"VM")]
+#[test_case(FeatureContract::TestContract(CairoVersion::Native), 216800;"Native")]
 fn no_constructor(deployer_contract: FeatureContract, expected_gas: u64) {
     // TODO(Yoni): share the init code of the tests in this file.
 
@@ -88,7 +89,7 @@ fn no_constructor_nonempty_calldata(deployer_contract: FeatureContract) {
          constructor."
     ));
 }
-#[test_case(FeatureContract::TestContract(CairoVersion::Native),226750, 5210;"Native")]
+#[test_case(FeatureContract::TestContract(CairoVersion::Native),236750, 15210;"Native")]
 #[test_case(FeatureContract::TestContract(CairoVersion::Cairo1),216750, 5210;"VM")]
 fn with_constructor(
     deployer_contract: FeatureContract,