pezkuwichain/pezkuwi-subxt
1
0
Fork 0
mirror of https://github.com/pezkuwichain/pezkuwi-subxt.git synced 2026-05-30 07:01:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix panic in pallet-collator-selection benchmarking (#1061)

Browse Source 
* register validators in set_invulnerables benchmarking

* Apply suggestions from code review

Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
This commit is contained in:
Kai
2022-03-02 10:24:36 +01:00
committed by GitHub
parent 62ccc85392
commit 1f91a3068f
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cumulus/pallets/collator-selection/src/benchmarking.rs
+5 -3
View File
@@ -84,12 +84,14 @@ fn validator<T: Config + session::Config>(c: u32) -> (T::AccountId, <T as sessio
(create_funded_user::<T>("candidate", c, 1000), keys::<T>(c)) (create_funded_user::<T>("candidate", c, 1000), keys::<T>(c))
} }
fn register_validators<T: Config + session::Config>(count: u32) { fn register_validators<T: Config + session::Config>(count: u32) -> Vec<T::AccountId> {
let validators = (0..count).map(|c| validator::<T>(c)).collect::<Vec<_>>(); let validators = (0..count).map(|c| validator::<T>(c)).collect::<Vec<_>>();
for (who, keys) in validators { for (who, keys) in validators.clone() {
<session::Pallet<T>>::set_keys(RawOrigin::Signed(who).into(), keys, Vec::new()).unwrap(); <session::Pallet<T>>::set_keys(RawOrigin::Signed(who).into(), keys, Vec::new()).unwrap();
} }
validators.into_iter().map(|(who, _)| who).collect()
} }
fn register_candidates<T: Config>(count: u32) { fn register_candidates<T: Config>(count: u32) {
@@ -107,7 +109,7 @@ benchmarks! {
set_invulnerables { set_invulnerables {
let b in 1 .. T::MaxInvulnerables::get(); let b in 1 .. T::MaxInvulnerables::get();
let new_invulnerables = (0..b).map(|c| account("candidate", c, SEED)).collect::<Vec<_>>(); let new_invulnerables = register_validators::<T>(b);
let origin = T::UpdateOrigin::successful_origin(); let origin = T::UpdateOrigin::successful_origin();
}: { }: {
assert_ok!( assert_ok!(