chore: set tx_hash as default in create declare tx for testing

crates/blockifier/src/test_utils/declare.rs

@@ -1,12 +1,15 @@
 use starknet_api::test_utils::declare::DeclareTxArgs;
+use starknet_api::transaction::TransactionHash;
 
 use crate::execution::contract_class::ClassInfo;
 use crate::transaction::account_transaction::AccountTransaction;
 use crate::transaction::transactions::DeclareTransaction;
 
 pub fn declare_tx(declare_tx_args: DeclareTxArgs, class_info: ClassInfo) -> AccountTransaction {
-    let tx_hash = declare_tx_args.tx_hash;
+    let default_tx_hash = TransactionHash::default();
     let declare_tx = starknet_api::test_utils::declare::declare_tx(declare_tx_args);
 
-    AccountTransaction::Declare(DeclareTransaction::new(declare_tx, tx_hash, class_info).unwrap())
+    AccountTransaction::Declare(
+        DeclareTransaction::new(declare_tx, default_tx_hash, class_info).unwrap(),
+    )
 }

crates/starknet_api/src/test_utils/declare.rs

@@ -32,8 +32,6 @@ pub struct DeclareTxArgs {
     pub nonce: Nonce,
     pub class_hash: ClassHash,
     pub compiled_class_hash: CompiledClassHash,
-    // TODO(Arni): Consider removing this field.
-    pub tx_hash: TransactionHash,
 }
 
 impl Default for DeclareTxArgs {
@@ -52,7 +50,6 @@ impl Default for DeclareTxArgs {
             nonce: Nonce::default(),
             class_hash: ClassHash::default(),
             compiled_class_hash: CompiledClassHash::default(),
-            tx_hash: TransactionHash::default(),
         }
     }
 }
@@ -124,7 +121,8 @@ pub fn executable_declare_tx(
     declare_tx_args: DeclareTxArgs,
     class_info: ClassInfo,
 ) -> ExecutableDeclareTransaction {
-    let tx_hash = declare_tx_args.tx_hash;
+    let default_tx_hash = TransactionHash::default();
     let tx = declare_tx(declare_tx_args);
-    ExecutableDeclareTransaction { tx, tx_hash, class_info }
+
+    ExecutableDeclareTransaction { tx, tx_hash: default_tx_hash, class_info }
 }