blockifier: add CallInfo::new (#14007)

Author: dorimedini-starkware

crates/apollo_consensus_orchestrator/src/cende/central_objects_test.rs

@@ -532,8 +532,8 @@ fn build_call_info(include_opcodes: bool) -> CallInfo {
     let extended_execution_resources = extended_execution_resources(include_opcodes);
     let builtin_counters = extended_execution_resources.prover_cairo_primitives();
 
-    CallInfo {
-        call: CallEntryPoint {
+    CallInfo::new(
+        CallEntryPoint {
             class_hash: Some(ClassHash(felt!("0x80020000"))),
             code_address: Some(contract_address!("0x40070000")),
             entry_point_type: EntryPointType::External,
@@ -551,7 +551,7 @@ fn build_call_info(include_opcodes: bool) -> CallInfo {
             call_type: CallType::Call,
             initial_gas: 100_000_000,
         },
-        execution: CallExecution {
+        CallExecution {
             retdata: Retdata(vec![felt!("0x56414c4944")]),
             events: vec![OrderedEvent {
                 order: 2,
@@ -571,10 +571,10 @@ fn build_call_info(include_opcodes: bool) -> CallInfo {
             gas_consumed: 11_690,
             cairo_native: false,
         },
-        inner_calls: Vec::new(),
-        resources: extended_execution_resources,
-        tracked_resource: TrackedResource::SierraGas,
-        storage_access_tracker: StorageAccessTracker {
+        Vec::new(),
+        extended_execution_resources,
+        TrackedResource::SierraGas,
+        StorageAccessTracker {
             storage_read_values: felt_vector(),
             accessed_storage_keys: HashSet::from([StorageKey::from(1_u128)]),
             read_class_hash_values: vec![ClassHash(felt!("0x80020000"))],
@@ -583,14 +583,13 @@ fn build_call_info(include_opcodes: bool) -> CallInfo {
             accessed_blocks: HashSet::from([BlockNumber(100)]),
         },
         builtin_counters,
-        syscalls_usage: HashMap::from([
+        HashMap::from([
             (SyscallSelector::CallContract, SyscallUsage { call_count: 7, linear_factor: 0 }),
             (SyscallSelector::StorageRead, SyscallUsage { call_count: 4, linear_factor: 0 }),
             (SyscallSelector::StorageWrite, SyscallUsage { call_count: 4, linear_factor: 0 }),
             (SyscallSelector::EmitEvent, SyscallUsage { call_count: 2, linear_factor: 0 }),
         ]),
-        ..Default::default()
-    }
+    )
 }
 
 fn transaction_execution_info() -> TransactionExecutionInfo {

crates/blockifier/src/execution/call_info.rs

@@ -462,6 +462,32 @@ pub struct CallInfo {
 }
 
 impl CallInfo {
+    /// Utility method to create an instance with a default Self::time field, if it exists.
+    #[expect(clippy::too_many_arguments)]
+    pub fn new(
+        call: CallEntryPoint,
+        execution: CallExecution,
+        inner_calls: Vec<CallInfo>,
+        resources: ExtendedExecutionResources,
+        tracked_resource: TrackedResource,
+        storage_access_tracker: StorageAccessTracker,
+        builtin_counters: CairoPrimitiveCounterMap,
+        syscalls_usage: SyscallUsageMap,
+    ) -> Self {
+        Self {
+            call,
+            execution,
+            inner_calls,
+            resources,
+            tracked_resource,
+            storage_access_tracker,
+            builtin_counters,
+            syscalls_usage,
+            #[cfg(feature = "benchmarking")]
+            time: Default::default(),
+        }
+    }
+
     pub fn iter(&self) -> CallInfoIter<'_> {
         let call_infos = vec![self];
         CallInfoIter { call_infos }

crates/starknet_committer_and_os_cli/src/block_hash_cli/tests/objects.rs

@@ -213,20 +213,19 @@ fn create_call_info(
     execution: CallExecution,
     inner_calls: Vec<CallInfo>,
 ) -> CallInfo {
-    CallInfo {
+    CallInfo::new(
         call,
         execution,
         inner_calls,
-        resources: create_execution_resources(1000, 0, 10, 5, 3),
-        tracked_resource: blockifier::execution::contract_class::TrackedResource::CairoSteps,
-        storage_access_tracker: Default::default(),
-        builtin_counters: cairo_primitive_counter_map([
+        create_execution_resources(1000, 0, 10, 5, 3),
+        blockifier::execution::contract_class::TrackedResource::CairoSteps,
+        Default::default(),
+        cairo_primitive_counter_map([
             (cairo_vm::types::builtin_name::BuiltinName::range_check, 10),
             (cairo_vm::types::builtin_name::BuiltinName::pedersen, 5),
         ]),
-        syscalls_usage: HashMap::new(),
-        ..Default::default()
-    }
+        HashMap::new(),
+    )
 }
 
 /// Helper function to create [TransactionResources].