apollo_committer_types: extend mock committer client

[yoavGrs]

No description provided.

[reviewable-StarkWare]

This change is 

[github-actions]

Artifacts upload workflows:

• Started at 2026-01-28T15:51:27Z
• Started at 2026-01-29T12:27:10Z
• Started at 2026-01-29T12:27:10Z
• Started at 2026-01-29T12:27:06Z
• Started at 2026-02-01T07:45:31Z

[yoavGrs]

• apollo_batcher: use mock committer client #12116
• apollo_committer_types: extend mock committer client #12115 👈 (View in Graphite)
• apollo_batcher: full revert commitment flow #11496
• apollo_proc_macros: features visibility #12060
• apollo_batcher: add revert task #11451 : 1 other dependent PR (#11516 )
• apollo_batcher: define custom input task type #11595
• apollo_batcher: wait for revert result #11484
• apollo_batcher: perform commitment tasks #11442
• apollo_committer: use rocks db storage #11733
• apollo_integration_tests: path with node index #11817
• apollo_committer_config: add db path to config #11800
• main-v0.14.1-committer

This stack of pull requests is managed by Graphite. Learn more about stacking.

[dorimedini-starkware]

@dorimedini-starkware reviewed 4 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @amosStarkware and @yoavGrs).

---

crates/apollo_committer_types/src/test_utils.rs line 49 at r1 (raw file):

        Self { inner, offset: Arc::new(Mutex::new(offset)) }
    }
    pub async fn set_offset(&self, offset: BlockNumber) {

non blocking

Suggestion:

    }
    
    pub async fn set_offset(&self, offset: BlockNumber) {

[dorimedini-starkware]

@dorimedini-starkware made 1 comment.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @amosStarkware and @yoavGrs).

---

crates/apollo_committer_types/src/test_utils.rs line 55 at r1 (raw file):

    pub fn get_offset(&self) -> Arc<Mutex<BlockNumber>> {
        Arc::clone(&self.offset)
    }

why return a locked offset and not just a block number?

Suggestion:

    pub async fn get_offset(&self) -> BlockNumber {
        self.offset.lock().await
    }

[yoavGrs]

@yoavGrs made 2 comments and resolved 1 discussion.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @amosStarkware and @dorimedini-starkware).

---

crates/apollo_committer_types/src/test_utils.rs line 49 at r1 (raw file):

Previously, dorimedini-starkware wrote…

non blocking

Done.

---

crates/apollo_committer_types/src/test_utils.rs line 55 at r1 (raw file):

Previously, dorimedini-starkware wrote…

why return a locked offset and not just a block number?

In the test, I'm moving the committer client before checking the offset.

[dorimedini-starkware]

@dorimedini-starkware reviewed 1 file and all commit messages, and resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @amosStarkware).

[dorimedini-starkware]

@dorimedini-starkware reviewed 1 file and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @amosStarkware).