Use reference to ContractsDataStore instead clone

Author: ddoktorski

crates/debugging/src/trace/collect.rs

@@ -15,7 +15,7 @@ use starknet_api::execution_utils::format_panic_data;
 
 pub struct Collector<'a> {
     call_trace: &'a CallTrace,
-    context: &'a Context,
+    context: &'a Context<'a>,
 }
 
 impl<'a> Collector<'a> {

crates/debugging/src/trace/context.rs

@@ -2,15 +2,18 @@ use crate::Components;
 use crate::contracts_data_store::ContractsDataStore;
 
 /// Context is a structure that holds the necessary data for creating a [`Trace`](crate::Trace).
-pub struct Context {
-    contracts_data_store: ContractsDataStore,
+pub struct Context<'a> {
+    contracts_data_store: &'a ContractsDataStore,
     components: Components,
 }
 
-impl Context {
-    /// Creates a new instance of [`Context`] from a given `cheatnet` [`ContractsData`], [`ForkData`] and [`Components`].
+impl<'a> Context<'a> {
+    /// Creates a new instance of [`Context`] from a given [`ContractsDataStore`] and [`Components`].
     #[must_use]
-    pub fn new(contracts_data_store: ContractsDataStore, components: Components) -> Self {
+    pub fn new(
+        contracts_data_store: &'a ContractsDataStore,
+        components: Components,
+    ) -> Context<'a> {
         Self {
             contracts_data_store,
             components,
@@ -20,7 +23,7 @@ impl Context {
     /// Returns a reference to the [`ContractsDataStore`].
     #[must_use]
     pub fn contracts_data_store(&self) -> &ContractsDataStore {
-        &self.contracts_data_store
+        self.contracts_data_store
     }
 
     /// Returns a reference to the [`Components`].

crates/forge-runner/src/debugging/mod.rs

@@ -16,6 +16,6 @@ pub fn build_debugging_trace(
     test_name: String,
 ) -> Option<debugging::Trace> {
     let components = trace_args.to_components()?;
-    let context = debugging::Context::new(contracts_data_store.clone(), components);
+    let context = debugging::Context::new(contracts_data_store, components);
     Some(debugging::Trace::new(call_trace, &context, test_name))
 }