From d2cd68b2d52a0d76a96f71e60c77c6031cba440b Mon Sep 17 00:00:00 2001 From: Dori Medini Date: Fri, 11 Oct 2024 12:58:36 +0300 Subject: [PATCH] test(blockifier): parametrize test_invoke_tx by resource bounds types --- crates/blockifier/src/transaction/transactions_test.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/crates/blockifier/src/transaction/transactions_test.rs b/crates/blockifier/src/transaction/transactions_test.rs index f3e0892ab0..e10fdf06f1 100644 --- a/crates/blockifier/src/transaction/transactions_test.rs +++ b/crates/blockifier/src/transaction/transactions_test.rs @@ -128,6 +128,7 @@ use crate::transaction::test_utils::{ calculate_class_info_for_testing, create_account_tx_for_validate_test, create_account_tx_for_validate_test_nonce_0, + default_all_resource_bounds, default_l1_resource_bounds, l1_resource_bounds, FaultyAccountTxCreatorArgs, @@ -406,7 +407,8 @@ fn add_kzg_da_resources_to_resources_mapping( }, CairoVersion::Cairo1)] fn test_invoke_tx( - default_l1_resource_bounds: ValidResourceBounds, + #[values(default_l1_resource_bounds(), default_all_resource_bounds())] + resource_bounds: ValidResourceBounds, #[case] expected_arguments: ExpectedResultTestInvokeTx, #[case] account_cairo_version: CairoVersion, #[values(false, true)] use_kzg_da: bool, @@ -422,7 +424,7 @@ fn test_invoke_tx( let invoke_tx = invoke_tx(invoke_tx_args! { sender_address: account_contract_address, calldata: create_trivial_calldata(test_contract_address), - resource_bounds: default_l1_resource_bounds, + resource_bounds, }); // Extract invoke transaction fields for testing, as it is consumed when creating an account @@ -539,7 +541,7 @@ fn test_invoke_tx( let total_gas = expected_actual_resources.to_gas_vector( &block_context.versioned_constants, block_context.block_info.use_kzg_da, - &GasVectorComputationMode::NoL2Gas, + &resource_bounds.get_gas_vector_computation_mode(), ); let expected_execution_info = TransactionExecutionInfo {