diff --git a/consensus/state_processing/src/per_block_processing/process_operations.rs b/consensus/state_processing/src/per_block_processing/process_operations.rs index 09c7c540ed..8047f67784 100644 --- a/consensus/state_processing/src/per_block_processing/process_operations.rs +++ b/consensus/state_processing/src/per_block_processing/process_operations.rs @@ -476,7 +476,10 @@ pub fn apply_deposit( // The signature should be checked for new validators. Return early for a bad // signature. if is_valid_deposit_signature(&deposit_data, spec).is_err() { + dbg!("invalid sig"); return Ok(()); + } else { + dbg!("valid sig"); } let new_validator_index = state.validators().len(); diff --git a/consensus/types/src/beacon_state.rs b/consensus/types/src/beacon_state.rs index 6b4bfd29cb..f2fdbc2780 100644 --- a/consensus/types/src/beacon_state.rs +++ b/consensus/types/src/beacon_state.rs @@ -371,6 +371,7 @@ where pub eth1_deposit_index: u64, // Registry + #[compare_fields(as_iter)] #[test_random(default)] pub validators: List, #[serde(with = "ssz_types::serde_utils::quoted_u64_var_list")] @@ -396,8 +397,10 @@ where pub current_epoch_attestations: List, E::MaxPendingAttestations>, // Participation (Altair and later) + #[compare_fields(as_iter)] #[superstruct(only(Altair, Bellatrix, Capella, Deneb, Electra))] #[test_random(default)] + #[compare_fields(as_iter)] pub previous_epoch_participation: List, #[superstruct(only(Altair, Bellatrix, Capella, Deneb, Electra))] #[test_random(default)] @@ -494,13 +497,16 @@ where #[superstruct(only(Electra), partial_getter(copy))] #[metastruct(exclude_from(tree_lists))] pub earliest_consolidation_epoch: Epoch, + #[compare_fields(as_iter)] #[test_random(default)] #[superstruct(only(Electra))] pub pending_balance_deposits: List, + #[compare_fields(as_iter)] #[test_random(default)] #[superstruct(only(Electra))] pub pending_partial_withdrawals: List, + #[compare_fields(as_iter)] #[test_random(default)] #[superstruct(only(Electra))] pub pending_consolidations: List,