starknet_committer: reorganize facts db files

Author: ArielElp

crates/starknet_committer/src/db.rs

@@ -1,6 +1,4 @@
-pub mod create_facts_tree;
 #[cfg(any(feature = "testing", test))]
 pub mod external_test_utils;
 pub mod facts_db;
 pub mod forest_trait;
-pub mod traversal;

crates/starknet_committer/src/db/external_test_utils.rs

@@ -17,7 +17,7 @@ use starknet_patricia::patricia_merkle_tree::updated_skeleton_tree::tree::{
 };
 use starknet_patricia_storage::map_storage::MapStorage;
 
-use crate::db::create_facts_tree::create_original_skeleton_tree;
+use crate::db::facts_db::create_facts_tree::create_original_skeleton_tree;
 
 pub async fn tree_computation_flow<L, TH>(
     leaf_modifications: LeafModifications<L>,

…et_committer/src/db/create_facts_tree.rs …ter/src/db/facts_db/create_facts_tree.rscrates/starknet_committer/src/db/create_facts_tree.rs renamed to crates/starknet_committer/src/db/facts_db/create_facts_tree.rs crates/starknet_committer/src/db/create_facts_tree.rs renamed to crates/starknet_committer/src/db/facts_db/create_facts_tree.rs

@@ -23,7 +23,7 @@ use starknet_patricia::patricia_merkle_tree::types::{NodeIndex, SortedLeafIndice
 use starknet_patricia_storage::storage_trait::Storage;
 use tracing::warn;
 
-use crate::db::traversal::calculate_subtrees_roots;
+use crate::db::facts_db::traversal::calculate_subtrees_roots;
 
 #[cfg(test)]
 #[path = "create_facts_tree_test.rs"]

…mmitter/src/db/create_facts_tree_test.rs …rc/db/facts_db/create_facts_tree_test.rscrates/starknet_committer/src/db/create_facts_tree_test.rs renamed to crates/starknet_committer/src/db/facts_db/create_facts_tree_test.rs crates/starknet_committer/src/db/create_facts_tree_test.rs renamed to crates/starknet_committer/src/db/facts_db/create_facts_tree_test.rs

@@ -23,7 +23,7 @@ use starknet_patricia_storage::map_storage::MapStorage;
 use starknet_patricia_storage::storage_trait::{DbHashMap, DbKey, DbValue};
 use starknet_types_core::felt::Felt;
 
-use crate::db::create_facts_tree::create_original_skeleton_tree;
+use crate::db::facts_db::create_facts_tree::create_original_skeleton_tree;
 
 #[tokio::test]
 #[rstest]

…es/starknet_committer/src/db/facts_db.rs …starknet_committer/src/db/facts_db/db.rscrates/starknet_committer/src/db/facts_db.rs renamed to crates/starknet_committer/src/db/facts_db/db.rs crates/starknet_committer/src/db/facts_db.rs renamed to crates/starknet_committer/src/db/facts_db/db.rs

@@ -14,7 +14,7 @@ use crate::block_committer::input::{
     ConfigImpl,
     StarknetStorageValue,
 };
-use crate::db::create_facts_tree::{
+use crate::db::facts_db::create_facts_tree::{
     create_original_skeleton_tree,
     create_original_skeleton_tree_and_get_previous_leaves,
 };

crates/starknet_committer/src/db/facts_db/mod.rs

@@ -0,0 +1,4 @@
+pub mod create_facts_tree;
+// TODO(Ariel): Move db.rs to this module and avoid db::fact_db::db path..
+pub mod db;
+pub mod traversal;

…s/starknet_committer/src/db/traversal.rs …t_committer/src/db/facts_db/traversal.rscrates/starknet_committer/src/db/traversal.rs renamed to crates/starknet_committer/src/db/facts_db/traversal.rs crates/starknet_committer/src/db/traversal.rs renamed to crates/starknet_committer/src/db/facts_db/traversal.rs

@@ -32,7 +32,7 @@ use starknet_patricia_storage::storage_trait::{DbHashMap, DbKey, DbValue};
 use starknet_types_core::felt::Felt;
 use starknet_types_core::hash::Pedersen;
 
-use super::fetch_patricia_paths_inner;
+use crate::db::facts_db::traversal::fetch_patricia_paths_inner;
 
 fn to_preimage_map(raw_preimages: HashMap<u32, Vec<u32>>) -> PreimageMap {
     raw_preimages
@@ -514,10 +514,10 @@ struct TestPatriciaPathsInput {
 /// The files names indicate the tree height, number of initial leaves and number of modified
 /// leaves. The hash function used in the python tests is Pedersen.
 /// The leaves values are their NodeIndices.
-#[case(include_str!("../../resources/fetch_patricia_paths_test_10_200_50.json"))]
-#[case(include_str!("../../resources/fetch_patricia_paths_test_10_5_2.json"))]
-#[case(include_str!("../../resources/fetch_patricia_paths_test_10_100_30.json"))]
-#[case(include_str!("../../resources/fetch_patricia_paths_test_8_120_70.json"))]
+#[case(include_str!("../../../resources/fetch_patricia_paths_test_10_200_50.json"))]
+#[case(include_str!("../../../resources/fetch_patricia_paths_test_10_5_2.json"))]
+#[case(include_str!("../../../resources/fetch_patricia_paths_test_10_100_30.json"))]
+#[case(include_str!("../../../resources/fetch_patricia_paths_test_8_120_70.json"))]
 async fn test_fetch_patricia_paths_inner_from_json(#[case] input_data: &str) {
     let input: TestPatriciaPathsInput = serde_json::from_str(input_data)
         .unwrap_or_else(|error| panic!("JSON was not well-formatted: {error:?}"));

…rknet_committer/src/db/traversal_test.rs …mitter/src/db/facts_db/traversal_test.rscrates/starknet_committer/src/db/traversal_test.rs renamed to crates/starknet_committer/src/db/facts_db/traversal_test.rs crates/starknet_committer/src/db/traversal_test.rs renamed to crates/starknet_committer/src/db/facts_db/traversal_test.rs

@@ -32,7 +32,7 @@ use crate::block_committer::input::{
     StarknetStorageValue,
     StateDiff,
 };
-use crate::db::facts_db::FactsDb;
+use crate::db::facts_db::db::FactsDb;
 use crate::db::forest_trait::ForestReader;
 use crate::forest::original_skeleton_forest::{ForestSortedIndices, OriginalSkeletonForest};
 use crate::patricia_merkle_tree::leaf::leaf_impl::ContractState;

crates/starknet_committer/src/forest/skeleton_forest_test.rs

@@ -14,7 +14,7 @@ use crate::block_committer::input::{
     StarknetStorageKey,
     StarknetStorageValue,
 };
-use crate::db::traversal::fetch_patricia_paths;
+use crate::db::facts_db::traversal::fetch_patricia_paths;
 use crate::patricia_merkle_tree::leaf::leaf_impl::ContractState;
 use crate::patricia_merkle_tree::types::{
     class_hash_into_node_index,