starknet_committer: move fetch_patricia_paths tests

[ArielElp]

No description provided.

[reviewable-StarkWare]

This change is 

[ArielElp]

• 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
• starknet_committer: db keys for proofs and accessed keys digest #13997
• apollo_committer_types: add patricia proofs request and response types #13996
• 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 👈 (View in Graphite)
• 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 module path and include_str fixes; trie traversal production code is unchanged.

Overview
Reorganizes Patricia path-fetch tests so they live in fetch_patricia_paths_tests.rs under src/db/ instead of facts_db/traversal_test.rs, and wires them into trie_traversal via an updated #[path = ...] test module.

Updates include_str! paths for the Python-generated JSON fixtures from ../../../resources/ to ../../resources/, matching the test file’s location so the embedded cases still resolve correctly.

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

[yoavGrs]

@yoavGrs partially reviewed 2 files and made 2 comments.
Reviewable status: 2 of 3 files reviewed, 2 unresolved discussions (waiting on ArielElp).

---

crates/starknet_committer/src/db.rs line 13 at r1 (raw file):

#[cfg(test)]
mod fetch_patricia_paths_tests;

Define it in trie_traversal.rs.

Code quote:

#[cfg(test)]
mod fetch_patricia_paths_tests;

---

crates/starknet_committer/src/db/trie_traversal.rs line 57 at r1 (raw file):

#[cfg(test)]
#[path = "facts_db/traversal_test.rs"]

According to code quality, it should stay here.

Suggestion:

#[path = "traversal_test.rs"]

[ArielElp]

@ArielElp partially reviewed 2 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on yoavGrs).

---

crates/starknet_committer/src/db.rs line 13 at r1 (raw file):

Previously, yoavGrs wrote…

Define it in trie_traversal.rs.

Done.

---

crates/starknet_committer/src/db/trie_traversal.rs line 57 at r1 (raw file):

Previously, yoavGrs wrote…

According to code quality, it should stay here.

You mean under facts_db, it's weird now that it's generic right?
In any case, I moved the definition to trie_traversal.

[yoavGrs]

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

[dorimedini-starkware]

@dorimedini-starkware partially reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on ArielElp).

[dorimedini-starkware]

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

[dorimedini-starkware]

@dorimedini-starkware partially reviewed 2 files and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on ArielElp).

[dorimedini-starkware]

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

[graphite-app]

Merge activity

• May 26, 8:37 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

[dorimedini-starkware]

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