Use explicit transaction version in InternalInvokeFunction::new

bench/internals.rs

@@ -15,7 +15,7 @@ use starknet_rs::{
         },
     },
     core::contract_address::starknet_contract_address::compute_deprecated_class_hash,
-    definitions::general_config::StarknetChainId,
+    definitions::{constants::TRANSACTION_VERSION, general_config::StarknetChainId},
     public::abi::VALIDATE_ENTRY_POINT_SELECTOR,
     services::api::contract_classes::deprecated_contract_class::ContractClass,
     utils::Address,
@@ -209,6 +209,7 @@ fn invoke() {
                 address,
                 selector,
                 0,
+                TRANSACTION_VERSION,
                 calldata,
                 signature,
                 StarknetChainId::TestNet.to_felt(),

src/bin/invoke_with_cachedstate.rs

@@ -9,7 +9,10 @@ use starknet_rs::{
         state::{cached_state::CachedState, state_api_objects::BlockInfo},
         transaction::objects::internal_invoke_function::InternalInvokeFunction,
     },
-    definitions::general_config::{StarknetChainId, StarknetGeneralConfig, StarknetOsConfig},
+    definitions::{
+        constants::TRANSACTION_VERSION,
+        general_config::{StarknetChainId, StarknetGeneralConfig, StarknetOsConfig},
+    },
     services::api::contract_classes::deprecated_contract_class::ContractClass,
     utils::Address,
 };
@@ -56,6 +59,7 @@ fn main() {
             CONTRACT_ADDRESS.clone(),
             INCREASE_BALANCE_SELECTOR.clone(),
             2,
+            TRANSACTION_VERSION,
             vec![1000.into()],
             vec![],
             StarknetChainId::TestNet.to_felt(),
@@ -70,6 +74,7 @@ fn main() {
             CONTRACT_ADDRESS.clone(),
             GET_BALANCE_SELECTOR.clone(),
             2,
+            TRANSACTION_VERSION,
             vec![],
             vec![],
             StarknetChainId::TestNet.to_felt(),

src/business_logic/transaction/objects/internal_invoke_function.rs

@@ -17,8 +17,7 @@ use crate::{
         calculate_transaction_hash_common, TransactionHashPrefix,
     },
     definitions::{
-        constants::{EXECUTE_ENTRY_POINT_SELECTOR, TRANSACTION_VERSION},
-        general_config::StarknetGeneralConfig,
+        constants::EXECUTE_ENTRY_POINT_SELECTOR, general_config::StarknetGeneralConfig,
         transaction_type::TransactionType,
     },
     public::abi::VALIDATE_ENTRY_POINT_SELECTOR,
@@ -54,13 +53,13 @@ impl InternalInvokeFunction {
         contract_address: Address,
         entry_point_selector: Felt252,
         max_fee: u128,
+        version: u64,
         calldata: Vec<Felt252>,
         signature: Vec<Felt252>,
         chain_id: Felt252,
         nonce: Option<Felt252>,
         hash_value: Option<Felt252>,
     ) -> Result<Self, TransactionError> {
-        let version = TRANSACTION_VERSION;
         let (entry_point_selector_field, additional_data) = preprocess_invoke_function_fields(
             entry_point_selector.clone(),
             nonce.clone(),

src/core/syscalls/deprecated_syscall_handler.rs

@@ -191,7 +191,10 @@ mod tests {
             DeprecatedDeployRequest, DeprecatedSendMessageToL1SysCallRequest,
             DeprecatedSyscallRequest,
         },
-        definitions::{general_config::StarknetGeneralConfig, transaction_type::TransactionType},
+        definitions::{
+            constants::TRANSACTION_VERSION, general_config::StarknetGeneralConfig,
+            transaction_type::TransactionType,
+        },
         memory_insert,
         services::api::contract_classes::deprecated_contract_class::{
             ContractClass, EntryPointType,
@@ -1090,6 +1093,7 @@ mod tests {
             )
             .unwrap(),
             0,
+            TRANSACTION_VERSION,
             vec![10.into()],
             Vec::new(),
             0.into(),

src/main.rs

@@ -188,6 +188,7 @@ fn invoke_parser(
         contract_address.clone(),
         entrypoint_selector.clone(),
         0,
+        TRANSACTION_VERSION,
         calldata.clone(),
         vec![],
         Felt252::zero(),

src/testing/starknet_state.rs

@@ -284,6 +284,7 @@ impl StarknetState {
             contract_address,
             entry_point_selector,
             max_fee,
+            TRANSACTION_VERSION,
             calldata,
             signature,
             self.chain_id(),

tests/internals.rs

@@ -527,6 +527,7 @@ fn invoke_tx(calldata: Vec<Felt252>) -> InternalInvokeFunction {
         TEST_ACCOUNT_CONTRACT_ADDRESS.clone(),
         EXECUTE_ENTRY_POINT_SELECTOR.clone(),
         2,
+        TRANSACTION_VERSION,
         calldata,
         vec![],
         StarknetChainId::TestNet.to_felt(),
@@ -1317,6 +1318,7 @@ fn test_invoke_tx_wrong_entrypoint() {
         // Entrypoiont that doesnt exits in the contract
         Felt252::from_bytes_be(&calculate_sn_keccak(b"none_function")),
         1,
+        TRANSACTION_VERSION,
         vec![
             test_contract_address, // CONTRACT_ADDRESS
             Felt252::from_bytes_be(&calculate_sn_keccak(b"return_result")), // CONTRACT FUNCTION SELECTOR