apollo_committer: reduce tests boilerplate

[ArielElp]

No description provided.

[reviewable-StarkWare]

This change is 

[ArielElp]

• apollo_committer: reduce tests boilerplate #14425 👈 (View in Graphite)
• apollo_committer: test new edge bottom in witnesses #14388
• apollo_committer: add request_paths_and_commit_block tests #14208
• starknet_patricia: add storage proof verification #14296
• apollo_committer: add new request handler #14003
• starknet_committer: delete patricia paths on reverts #14002
• starknet_committer,apollo_committer: add timers for fetch patricia paths #14189
• starknet_committer: underlying logic of the new read witnesses and commit endpoint #14001
• starknet_committer: extract metadata construction to a function #14000
• starknet_committer: extract commit plan to commit_or_load #13999
• starknet_committer: read and write patricia paths to index db #13998
• apollo_committer_types: add patricia proofs request and response types #13996 : 1 other dependent PR (#13997 )
• starknet_committer: add patricia paths forest reader/writer traits #13995
• starknet_committer: define StarknetForestProofs serde #13994
• starknet_committer: compute and store accessed keys digest #13993
• starknet_committer: add utility that returns sorted leaf indices #13992
• starknet_committer: abstract fetch_all_patricia_paths #13991
• starknet_patricia_storage: two layer storage #13990
• starknet_committer: move fetch_patricia_paths tests #13960
• starknet_committer: fetch patricia paths tests #13959
• starknet_committer: support both layouts in fetch_patricia_paths #13958
• main

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

[cursor]

PR Summary

Low Risk
Test-only refactor in one file; no production committer logic changes.

Overview
Introduces a shared setup_and_read_paths helper in request_paths_and_commit_block_tests.rs that commits block 0 with a setup state diff (no witnesses), then runs read_paths_and_commit_block at block 1 and returns the response.

revert_removes_witnesses_and_digest and the two Patricia edge proof tests now call this helper instead of duplicating the two-step commit/read setup. Those tests also rename the initial diff variable to setup_state_diff where it is custom. Imports ReadPathsAndCommitBlockResponse for the helper’s return type.

Test behavior is unchanged; this is structural deduplication only.

^{Reviewed by Cursor Bugbot for commit 46fd17c. Bugbot is set up for automated code reviews on this repo. Configure here.}

[dorimedini-starkware]

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

[yoavGrs]

@yoavGrs reviewed 1 file and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on ArielElp).

[yoavGrs]

@yoavGrs reviewed 1 file and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on ArielElp).