fix: fix tracepoint tests; comment out for now rr_test; add a note for Box<Db>

tracepoint tests: `jump_to` with replay to sync stepping; add a note for `Box<Db>`

debugged together with Nedy

Co-authored-by: pxor <nedislav@gmail.com>

Author: alehander92

src/db-backend/src/db.rs

@@ -785,6 +785,11 @@ pub enum EndOfProgram {
 
 #[derive(Debug)]
 pub struct DbReplay {
+    // to optimize and prevents problems with too many `.clone()`
+    //   currently mostly in flow;
+    //   we can try to share a readonly version: `Rc`? `RefCell`?
+    //   or we can leave it like this for now and expect that the new format
+    //   will deal with that?
     pub db: Box<Db>,
     pub step_id: StepId,
     pub call_key: CallKey,

src/db-backend/src/tracepoint_interpreter/tests.rs

@@ -14,6 +14,7 @@ use runtime_tracing::{StepId, TypeKind};
 use crate::{
     db::{Db, DbReplay},
     lang::Lang,
+    replay::Replay,
     task::StringAndValueTuple,
     trace_processor::{load_trace_data, load_trace_metadata, TraceProcessor},
     value::Value,
@@ -102,6 +103,7 @@ fn check_tracepoint_evaluate(
     let mut db_replay = DbReplay::new(Box::new(db.clone()));
     for step in db.step_from(StepId(0), true) {
         let curr_line = step.line.0 as usize;
+        db_replay.jump_to(step.step_id)?;
 
         if line == curr_line {
             let actual = interpreter.evaluate(0, step.step_id, &mut db_replay, lang);