core/storage/epochs: Change pred Epochs default to be empty

apps/src/lib/node/ledger/storage/mod.rs

@@ -510,6 +510,7 @@ mod tests {
             .expect("write failed");
 
         storage.block.epoch = storage.block.epoch.next();
+        storage.block.pred_epochs.new_epoch(new_epoch_start);
         let batch = PersistentStorage::batch();
         storage.commit_block(batch).expect("commit failed");
 

core/src/ledger/storage/mod.rs

@@ -963,8 +963,6 @@ where
         } = parameters.epoch_duration;
         self.next_epoch_min_start_height = initial_height + min_num_of_blocks;
         self.next_epoch_min_start_time = genesis_time + min_duration;
-        // The default start height in `Epochs` is set to 1, we override it
-        // here in case it differs at genesis
         self.block.pred_epochs = Epochs {
             first_block_heights: vec![initial_height],
         };
@@ -1302,6 +1300,12 @@ mod tests {
                 minimum_gas_price: BTreeMap::default(),
             };
             parameters.init_storage(&mut wl_storage).unwrap();
+            // Initialize pred_epochs to the current height
+            wl_storage
+                .storage
+                .block
+                .pred_epochs
+                .new_epoch(wl_storage.storage.block.height);
 
             let epoch_before = wl_storage.storage.last_epoch;
             assert_eq!(epoch_before, wl_storage.storage.block.epoch);

core/src/types/storage.rs

@@ -1165,6 +1165,7 @@ impl Mul for Epoch {
 #[derive(
     Clone,
     Debug,
+    Default,
     PartialEq,
     Eq,
     PartialOrd,
@@ -1178,15 +1179,6 @@ pub struct Epochs {
     /// Invariant: the values must be sorted in ascending order.
     pub first_block_heights: Vec<BlockHeight>,
 }
-impl Default for Epochs {
-    /// Initialize predecessor epochs, assuming start of the first epoch at
-    /// block height 1.
-    fn default() -> Self {
-        Self {
-            first_block_heights: vec![BlockHeight(1)],
-        }
-    }
-}
 
 impl Epochs {
     /// Record start of a new epoch at the given block height
@@ -1630,7 +1622,9 @@ mod tests {
 
     #[test]
     fn test_predecessor_epochs_and_heights() {
-        let mut epochs = Epochs::default();
+        let mut epochs = Epochs {
+            first_block_heights: vec![BlockHeight::first()],
+        };
         println!("epochs {:#?}", epochs);
         assert_eq!(
             epochs.get_start_height_of_epoch(Epoch(0)),

ethereum_bridge/src/test_utils.rs

@@ -227,6 +227,12 @@ pub fn init_storage_with_validators(
             .write(&protocol_pk_key(validator), protocol_key)
             .expect("Test failed");
     }
+    // Initialize pred_epochs to the current height
+    wl_storage
+        .storage
+        .block
+        .pred_epochs
+        .new_epoch(wl_storage.storage.block.height);
     wl_storage.commit_block().expect("Test failed");
 
     all_keys