diff --git a/polkadot/runtime/parachains/src/disputes.rs b/polkadot/runtime/parachains/src/disputes.rs index 7acfc0d9c088..353c2b0cf87f 100644 --- a/polkadot/runtime/parachains/src/disputes.rs +++ b/polkadot/runtime/parachains/src/disputes.rs @@ -702,7 +702,6 @@ impl DisputeStateImporter { let mut undo = ImportUndo { validator_index: validator, vote_kind: kind, new_participant: false }; - log::debug!(target: LOG_TARGET, "Setting dispute bit: {}", validator.0); bits.set(validator.0 as usize, true); if kind.is_backing() { let is_new = self.backers.insert(validator); @@ -992,7 +991,6 @@ impl Pallet { let backers = BackersOnDisputes::::get(&set.session, &set.candidate_hash).unwrap_or_default(); - log::debug!(target: LOG_TARGET, "Dispute import with {} backers", backers.len()); // Check and import all votes. let summary = { @@ -1010,7 +1008,6 @@ impl Pallet { }; let kind = VoteKind::from(statement); - log::debug!(target: LOG_TARGET, "Importing {:?} vote from {}", kind, validator_index.0); let undo = match importer.import(*validator_index, kind) { Ok(u) => u, diff --git a/polkadot/runtime/parachains/src/paras_inherent/mod.rs b/polkadot/runtime/parachains/src/paras_inherent/mod.rs index aa13fd6bf079..e5164f5b1f31 100644 --- a/polkadot/runtime/parachains/src/paras_inherent/mod.rs +++ b/polkadot/runtime/parachains/src/paras_inherent/mod.rs @@ -402,14 +402,12 @@ impl Pallet { // Limit the disputes first, since the following statements depend on the votes include // here. - log::debug!(target: LOG_TARGET, "Disputes before sanitization: {}", disputes.len()); let (checked_disputes_sets, checked_disputes_sets_consumed_weight) = limit_and_sanitize_disputes::( disputes, dispute_statement_set_valid, max_block_weight, ); - log::debug!(target: LOG_TARGET, "Disputes after sanitization: {}", checked_disputes_sets.len()); let all_weight_after = if context == ProcessInherentDataContext::ProvideInherent { // Assure the maximum block weight is adhered, by limiting bitfields and backed diff --git a/polkadot/runtime/parachains/src/paras_inherent/tests.rs b/polkadot/runtime/parachains/src/paras_inherent/tests.rs index c71aa531c3bf..078a6985f26c 100644 --- a/polkadot/runtime/parachains/src/paras_inherent/tests.rs +++ b/polkadot/runtime/parachains/src/paras_inherent/tests.rs @@ -3319,11 +3319,7 @@ mod sanitizers { 3, 10, ); - let allowed_parents = shared::AllowedRelayParents::::get(); - println!("before enter: {allowed_parents:?}"); let _ = Pallet::::enter(frame_system::RawOrigin::None.into(), data).unwrap(); - let allowed_parents = shared::AllowedRelayParents::::get(); - println!("right after: {allowed_parents:?}"); assert_matches!(pallet::OnChainVotes::::get(), Some(ScrapedOnChainVotes { disputes, @@ -3341,14 +3337,6 @@ mod sanitizers { parent_header.hash(), // `parent_hash`, Default::default(), // digest, ); - // let data = ParachainsInherentData { - // disputes: Vec::new(), - // parent_header: parent_header.clone(), - // bitfields: Vec::new(), - // backed_candidates: Vec::new(), - // }; - // let _ = Pallet::::enter(frame_system::RawOrigin::None.into(), - // data).unwrap(); frame_system::Pallet::::reset_events(); frame_system::Pallet::::initialize( @@ -3364,17 +3352,6 @@ mod sanitizers { backed_candidates, }; - // shared::Pallet::::clear_allowed_relay_parents(); - // let header = default_header(); - // shared::Pallet::::add_allowed_relay_parent( - // header.hash(), - // *header.state_root(), - // 3, - // 10, - // ); - let allowed_parents = shared::AllowedRelayParents::::get(); - println!("{allowed_parents:?}"); - let _ = Pallet::::enter(frame_system::RawOrigin::None.into(), data).unwrap(); assert_eq!( // The length of this vec is equal to the number of candidates, so we know diff --git a/polkadot/runtime/parachains/src/shared.rs b/polkadot/runtime/parachains/src/shared.rs index d22c72253cac..319b22515889 100644 --- a/polkadot/runtime/parachains/src/shared.rs +++ b/polkadot/runtime/parachains/src/shared.rs @@ -44,7 +44,7 @@ pub(crate) const SESSION_DELAY: SessionIndex = 2; mod tests; /// Information about past relay-parents. -#[derive(Encode, Decode, Default, TypeInfo, Debug)] +#[derive(Encode, Decode, Default, TypeInfo)] pub struct AllowedRelayParentsTracker { // The past relay parents, paired with state roots, that are viable to build upon. // @@ -270,9 +270,4 @@ impl Pallet { tracker.update(relay_parent, state_root, number, max_ancestry_len) }) } - - #[cfg(test)] - pub(crate) fn clear_allowed_relay_parents() { - AllowedRelayParents::::kill(); - } }