feat: add more information to errors

Signed-off-by: Gustavo Inacio <[email protected]>

crates/header-accumulator/src/epoch.rs

@@ -53,7 +53,10 @@ impl TryFrom<Vec<ExtHeaderRecord>> for Epoch {
             .map(|w| w[0].block_number + 1)
             .collect();
         if !blocks_missing.is_empty() {
-            return Err(EraValidateError::MissingBlock(blocks_missing));
+            return Err(EraValidateError::MissingBlock {
+                blocks: blocks_missing,
+                epoch: epoch_number,
+            });
         }
 
         // check if all blocks are in the same era

crates/header-accumulator/src/errors.rs

@@ -13,16 +13,16 @@ pub enum EraValidateError {
     #[error("Error validating inclusion proof")]
     ProofValidationFailure,
 
-    #[error("blocks in epoch must be exactly 8192 units, found {0}")]
+    #[error("Blocks in epoch must be exactly 8192 units, found {0}")]
     InvalidEpochLength(usize),
 
-    #[error("block was missing while creating epoch")]
-    MissingBlock(Vec<u64>),
+    #[error("Block was missing while creating epoch {epoch}. Missing blocks: {blocks:?}")]
+    MissingBlock { epoch: u64, blocks: Vec<u64> },
 
-    #[error("not all blocks are in the same epoch. epochs found: {0:?}")]
+    #[error("Not all blocks are in the same epoch. Epochs found: {0:?}")]
     InvalidBlockInEpoch(HashSet<u64>),
-    #[error("Error converting ExtHeaderRecord to header")]
-    ExtHeaderRecordError,
+    #[error("Error converting ExtHeaderRecord to header block number {0}")]
+    ExtHeaderRecordError(u64),
     #[error("Invalid block range: {0} - {1}")]
     InvalidBlockRange(u64, u64),
     #[error("Epoch is in post merge: {0}")]

crates/header-accumulator/src/types.rs

@@ -32,7 +32,7 @@ impl TryFrom<ExtHeaderRecord> for Header {
 
     fn try_from(ext: ExtHeaderRecord) -> Result<Self, Self::Error> {
         ext.full_header
-            .ok_or(EraValidateError::ExtHeaderRecordError)
+            .ok_or(EraValidateError::ExtHeaderRecordError(ext.block_number))
     }
 }