pr feedback: add test impl for ErasureConfig

solana-labs · Oct 25, 2023 · 9d9252a · 9d9252a
1 parent 3196f26
commit 9d9252a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/ledger/src/blockstore.rs b/ledger/src/blockstore.rs
@@ -7511,10 +7511,7 @@ pub mod tests {
         let blockstore = Blockstore::open(ledger_path.path()).unwrap();
         let erasure_meta_cf = &blockstore.erasure_meta_cf;
 
-        let config = ErasureConfig {
-            num_data: 1,
-            num_coding: 17,
-        };
+        let config = ErasureConfig::new(1, 17);
         let erasure_meta_old = ErasureMetaLegacy {
             set_index: 5,
             first_coding_index: 8,

diff --git a/ledger/src/blockstore_meta.rs b/ledger/src/blockstore_meta.rs
@@ -150,8 +150,18 @@ pub struct ErasureMeta {
 
 #[derive(Clone, Copy, Debug, Eq, PartialEq, Serialize, Deserialize)]
 pub(crate) struct ErasureConfig {
-    pub(crate) num_data: usize,
-    pub(crate) num_coding: usize,
+    num_data: usize,
+    num_coding: usize,
+}
+
+#[cfg(test)]
+impl ErasureConfig {
+    pub(crate) fn new(num_data: usize, num_coding: usize) -> Self {
+        ErasureConfig {
+            num_data,
+            num_coding,
+        }
+    }
 }
 
 #[derive(Deserialize, Serialize)]