git.delta.rocks / unique-network / refs/commits / ee0e61035d6e

difftreelog

doc: utility benchmarking functions

Daniel Shiposha2022-11-09parent: #e3dab71.patch.diff
in: master

1 file changed

modifiedpallets/scheduler-v2/src/benchmarking.rsdiffbeforeafterboth
before · pallets/scheduler-v2/src/benchmarking.rs
1// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.2// This file is part of Unique Network.34// Unique Network is free software: you can redistribute it and/or modify5// it under the terms of the GNU General Public License as published by6// the Free Software Foundation, either version 3 of the License, or7// (at your option) any later version.89// Unique Network is distributed in the hope that it will be useful,10// but WITHOUT ANY WARRANTY; without even the implied warranty of11// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the12// GNU General Public License for more details.1314// You should have received a copy of the GNU General Public License15// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.1617// Original license:18// This file is part of Substrate.1920// Copyright (C) 2020-2022 Parity Technologies (UK) Ltd.21// SPDX-License-Identifier: Apache-2.02223// Licensed under the Apache License, Version 2.0 (the "License");24// you may not use this file except in compliance with the License.25// You may obtain a copy of the License at26//27// 	http://www.apache.org/licenses/LICENSE-2.028//29// Unless required by applicable law or agreed to in writing, software30// distributed under the License is distributed on an "AS IS" BASIS,31// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.32// See the License for the specific language governing permissions and33// limitations under the License.3435//! Scheduler pallet benchmarking.3637use super::*;38use frame_benchmarking::{account, benchmarks};39use frame_support::{40	ensure,41	traits::{schedule::Priority, PreimageRecipient},42};43use frame_system::RawOrigin;44use sp_std::{prelude::*, vec};45use sp_io::hashing::blake2_256;4647use crate::{Pallet as Scheduler, ScheduledCall, EncodedCall};48use frame_system::Call as SystemCall;4950const SEED: u32 = 0;5152const BLOCK_NUMBER: u32 = 2;5354/// Add `n` items to the schedule.55///56/// For `resolved`:57/// - `58/// - `None`: aborted (hash without preimage)59/// - `Some(true)`: hash resolves into call if possible, plain call otherwise60/// - `Some(false)`: plain call61fn fill_schedule<T: Config>(when: T::BlockNumber, n: u32) -> Result<(), &'static str> {62	let t = DispatchTime::At(when);63	let origin: <T as Config>::PalletsOrigin = frame_system::RawOrigin::Root.into();64	for i in 0..n {65		let call = make_call::<T>(None);66		let period = Some(((i + 100).into(), 100));67		let name = u32_to_name(i);68		Scheduler::<T>::do_schedule_named(name, t, period, 0, origin.clone(), call)?;69	}70	ensure!(71		Agenda::<T>::get(when).agenda.len() == n as usize,72		"didn't fill schedule"73	);74	Ok(())75}7677fn u32_to_name(i: u32) -> TaskName {78	i.using_encoded(blake2_256)79}8081fn make_task<T: Config>(82	periodic: bool,83	named: bool,84	signed: bool,85	maybe_lookup_len: Option<u32>,86	priority: Priority,87) -> ScheduledOf<T> {88	let call = make_call::<T>(maybe_lookup_len);89	let maybe_periodic = match periodic {90		true => Some((100u32.into(), 100)),91		false => None,92	};93	let maybe_id = match named {94		true => Some(u32_to_name(0)),95		false => None,96	};97	let origin = make_origin::<T>(signed);98	Scheduled {99		maybe_id,100		priority,101		call,102		maybe_periodic,103		origin,104		_phantom: PhantomData,105	}106}107108fn bounded<T: Config>(len: u32) -> Option<ScheduledCall<T>> {109	let call = <<T as Config>::RuntimeCall>::from(SystemCall::remark {110		remark: vec![0; len as usize],111	});112	ScheduledCall::new(call).ok()113}114115fn make_call<T: Config>(maybe_lookup_len: Option<u32>) -> ScheduledCall<T> {116	let bound = EncodedCall::bound() as u32;117	let mut len = match maybe_lookup_len {118		Some(len) => {119			len.min(<T::Preimages as PreimageRecipient<T::Hash>>::MaxSize::get() - 2)120				.max(bound) - 3121		}122		None => bound.saturating_sub(4),123	};124125	loop {126		let c = match bounded::<T>(len) {127			Some(x) => x,128			None => {129				len -= 1;130				continue;131			}132		};133		if c.lookup_needed() == maybe_lookup_len.is_some() {134			break c;135		}136		if maybe_lookup_len.is_some() {137			len += 1;138		} else {139			if len > 0 {140				len -= 1;141			} else {142				break c;143			}144		}145	}146}147148fn make_origin<T: Config>(signed: bool) -> <T as Config>::PalletsOrigin {149	match signed {150		true => frame_system::RawOrigin::Signed(account("origin", 0, SEED)).into(),151		false => frame_system::RawOrigin::Root.into(),152	}153}154155fn dummy_counter() -> WeightCounter {156	WeightCounter {157		used: Weight::zero(),158		limit: Weight::MAX,159	}160}161162benchmarks! {163	// `service_agendas` when no work is done.164	service_agendas_base {165		let now = T::BlockNumber::from(BLOCK_NUMBER);166		IncompleteSince::<T>::put(now - One::one());167	}: {168		Scheduler::<T>::service_agendas(&mut dummy_counter(), now, 0);169	} verify {170		assert_eq!(IncompleteSince::<T>::get(), Some(now - One::one()));171	}172173	// `service_agenda` when no work is done.174	service_agenda_base {175		let now = BLOCK_NUMBER.into();176		let s in 0 .. T::MaxScheduledPerBlock::get();177		fill_schedule::<T>(now, s)?;178		let mut executed = 0;179	}: {180		Scheduler::<T>::service_agenda(&mut dummy_counter(), &mut executed, now, now, 0);181	} verify {182		assert_eq!(executed, 0);183	}184185	// `service_task` when the task is a non-periodic, non-named, non-fetched call which is not186	// dispatched (e.g. due to being overweight).187	service_task_base {188		let now = BLOCK_NUMBER.into();189		let task = make_task::<T>(false, false, false, None, 0);190		// prevent any tasks from actually being executed as we only want the surrounding weight.191		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::zero() };192	}: {193		let result = Scheduler::<T>::service_task(&mut counter, now, now, 0, true, task);194	} verify {195		//assert_eq!(result, Ok(()));196	}197198	// TODO uncomment if we will use the Preimages199	// // `service_task` when the task is a non-periodic, non-named, fetched call (with a known200	// // preimage length) and which is not dispatched (e.g. due to being overweight).201	// service_task_fetched {202	// 	let s in (EncodedCall::bound() as u32) .. (<T::Preimages as PreimageRecipient<T::Hash>>::MaxSize::get());203	// 	let now = BLOCK_NUMBER.into();204	// 	let task = make_task::<T>(false, false, false, Some(s), 0);205	// 	// prevent any tasks from actually being executed as we only want the surrounding weight.206	// 	let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::zero() };207	// }: {208	// 	let result = Scheduler::<T>::service_task(&mut counter, now, now, 0, true, task);209	// } verify {210	// }211212	// `service_task` when the task is a non-periodic, named, non-fetched call which is not213	// dispatched (e.g. due to being overweight).214	service_task_named {215		let now = BLOCK_NUMBER.into();216		let task = make_task::<T>(false, true, false, None, 0);217		// prevent any tasks from actually being executed as we only want the surrounding weight.218		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::zero() };219	}: {220		let result = Scheduler::<T>::service_task(&mut counter, now, now, 0, true, task);221	} verify {222	}223224	// `service_task` when the task is a periodic, non-named, non-fetched call which is not225	// dispatched (e.g. due to being overweight).226	service_task_periodic {227		let now = BLOCK_NUMBER.into();228		let task = make_task::<T>(true, false, false, None, 0);229		// prevent any tasks from actually being executed as we only want the surrounding weight.230		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::zero() };231	}: {232		let result = Scheduler::<T>::service_task(&mut counter, now, now, 0, true, task);233	} verify {234	}235236	// `execute_dispatch` when the origin is `Signed`, not counting the dispatable's weight.237	execute_dispatch_signed {238		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::MAX };239		let origin = make_origin::<T>(true);240		let call = T::Preimages::realize(&make_call::<T>(None)).unwrap().0;241	}: {242		assert!(Scheduler::<T>::execute_dispatch(&mut counter, origin, call).is_ok());243	}244	verify {245	}246247	// `execute_dispatch` when the origin is not `Signed`, not counting the dispatable's weight.248	execute_dispatch_unsigned {249		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::MAX };250		let origin = make_origin::<T>(false);251		let call = T::Preimages::realize(&make_call::<T>(None)).unwrap().0;252	}: {253		assert!(Scheduler::<T>::execute_dispatch(&mut counter, origin, call).is_ok());254	}255	verify {256	}257258	schedule {259		let s in 0 .. (T::MaxScheduledPerBlock::get() - 1);260		let when = BLOCK_NUMBER.into();261		let periodic = Some((T::BlockNumber::one(), 100));262		let priority = Some(0);263		// Essentially a no-op call.264		let call = Box::new(SystemCall::set_storage { items: vec![] }.into());265266		fill_schedule::<T>(when, s)?;267	}: _(RawOrigin::Root, when, periodic, priority, call)268	verify {269		ensure!(270			Agenda::<T>::get(when).agenda.len() == (s + 1) as usize,271			"didn't add to schedule"272		);273	}274275	cancel {276		let s in 1 .. T::MaxScheduledPerBlock::get();277		let when = BLOCK_NUMBER.into();278279		fill_schedule::<T>(when, s)?;280		assert_eq!(Agenda::<T>::get(when).agenda.len(), s as usize);281	}: _(RawOrigin::Root, when, 0)282	verify {283		ensure!(284			Lookup::<T>::get(u32_to_name(0)).is_none(),285			"didn't remove from lookup"286		);287		// Removed schedule is NONE288		ensure!(289			Agenda::<T>::get(when).agenda[0].is_none(),290			"didn't remove from schedule"291		);292	}293294	schedule_named {295		let s in 0 .. (T::MaxScheduledPerBlock::get() - 1);296		let id = u32_to_name(s);297		let when = BLOCK_NUMBER.into();298		let periodic = Some((T::BlockNumber::one(), 100));299		let priority = Some(0);300		// Essentially a no-op call.301		let call = Box::new(SystemCall::set_storage { items: vec![] }.into());302303		fill_schedule::<T>(when, s)?;304	}: _(RawOrigin::Root, id, when, periodic, priority, call)305	verify {306		ensure!(307			Agenda::<T>::get(when).agenda.len() == (s + 1) as usize,308			"didn't add to schedule"309		);310	}311312	cancel_named {313		let s in 1 .. T::MaxScheduledPerBlock::get();314		let when = BLOCK_NUMBER.into();315316		fill_schedule::<T>(when, s)?;317	}: _(RawOrigin::Root, u32_to_name(0))318	verify {319		ensure!(320			Lookup::<T>::get(u32_to_name(0)).is_none(),321			"didn't remove from lookup"322		);323		// Removed schedule is NONE324		ensure!(325			Agenda::<T>::get(when).agenda[0].is_none(),326			"didn't remove from schedule"327		);328	}329330	change_named_priority {331		let origin: RawOrigin<T::AccountId> = frame_system::RawOrigin::Root;332		let s in 1 .. T::MaxScheduledPerBlock::get();333		let when = BLOCK_NUMBER.into();334		let idx = s - 1;335		let id = u32_to_name(idx);336		let priority = 42;337		fill_schedule::<T>(when, s)?;338	}: _(origin, id, priority)339	verify {340		ensure!(341			Agenda::<T>::get(when).agenda[idx as usize].clone().unwrap().priority == priority,342			"didn't change the priority"343		);344	}345346	impl_benchmark_test_suite!(Scheduler, crate::mock::new_test_ext(), crate::mock::Test);347}
after · pallets/scheduler-v2/src/benchmarking.rs
1// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.2// This file is part of Unique Network.34// Unique Network is free software: you can redistribute it and/or modify5// it under the terms of the GNU General Public License as published by6// the Free Software Foundation, either version 3 of the License, or7// (at your option) any later version.89// Unique Network is distributed in the hope that it will be useful,10// but WITHOUT ANY WARRANTY; without even the implied warranty of11// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the12// GNU General Public License for more details.1314// You should have received a copy of the GNU General Public License15// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.1617// Original license:18// This file is part of Substrate.1920// Copyright (C) 2020-2022 Parity Technologies (UK) Ltd.21// SPDX-License-Identifier: Apache-2.02223// Licensed under the Apache License, Version 2.0 (the "License");24// you may not use this file except in compliance with the License.25// You may obtain a copy of the License at26//27// 	http://www.apache.org/licenses/LICENSE-2.028//29// Unless required by applicable law or agreed to in writing, software30// distributed under the License is distributed on an "AS IS" BASIS,31// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.32// See the License for the specific language governing permissions and33// limitations under the License.3435//! Scheduler pallet benchmarking.3637use super::*;38use frame_benchmarking::{account, benchmarks};39use frame_support::{40	ensure,41	traits::{schedule::Priority, PreimageRecipient},42};43use frame_system::RawOrigin;44use sp_std::{prelude::*, vec};45use sp_io::hashing::blake2_256;4647use crate::{Pallet as Scheduler, ScheduledCall, EncodedCall};48use frame_system::Call as SystemCall;4950const SEED: u32 = 0;5152const BLOCK_NUMBER: u32 = 2;5354/// Add `n` items to the schedule.55///56/// For `resolved`:57/// - `58/// - `None`: aborted (hash without preimage)59/// - `Some(true)`: hash resolves into call if possible, plain call otherwise60/// - `Some(false)`: plain call61fn fill_schedule<T: Config>(when: T::BlockNumber, n: u32) -> Result<(), &'static str> {62	let t = DispatchTime::At(when);63	let origin: <T as Config>::PalletsOrigin = frame_system::RawOrigin::Root.into();64	for i in 0..n {65		let call = make_call::<T>(None);66		let period = Some(((i + 100).into(), 100));67		let name = u32_to_name(i);68		Scheduler::<T>::do_schedule_named(name, t, period, 0, origin.clone(), call)?;69	}70	ensure!(71		Agenda::<T>::get(when).agenda.len() == n as usize,72		"didn't fill schedule"73	);74	Ok(())75}7677/// Generate a name for a scheduled task from an unsigned integer.78fn u32_to_name(i: u32) -> TaskName {79	i.using_encoded(blake2_256)80}8182/// A utility for creating simple scheduled tasks.83///84/// # Arguments85/// * `periodic` - makes the task periodic.86/// 	Sets the task's period and repetition count to `100`.87/// * `named` - gives a name to the task: `u32_to_name(0)`.88/// * `signed` - determines the origin of the task.89/// 	If true, it will have the Signed origin. Otherwise it will have the Root origin.90/// 	See [`make_origin`] for details.91/// * maybe_lookup_len - sets optional lookup length. It is used to benchmark task fetching from the `Preimages` store.92/// * priority - the task's priority.93fn make_task<T: Config>(94	periodic: bool,95	named: bool,96	signed: bool,97	maybe_lookup_len: Option<u32>,98	priority: Priority,99) -> ScheduledOf<T> {100	let call = make_call::<T>(maybe_lookup_len);101	let maybe_periodic = match periodic {102		true => Some((100u32.into(), 100)),103		false => None,104	};105	let maybe_id = match named {106		true => Some(u32_to_name(0)),107		false => None,108	};109	let origin = make_origin::<T>(signed);110	Scheduled {111		maybe_id,112		priority,113		call,114		maybe_periodic,115		origin,116		_phantom: PhantomData,117	}118}119120/// Creates a `SystemCall::remark` scheduled call with a given `len` in bytes.121/// Returns `None` if the call is too large to encode.122fn bounded<T: Config>(len: u32) -> Option<ScheduledCall<T>> {123	let call = <<T as Config>::RuntimeCall>::from(SystemCall::remark {124		remark: vec![0; len as usize],125	});126	ScheduledCall::new(call).ok()127}128129/// Creates a scheduled call and maximizes its size.130///131/// If the `maybe_lookup_len` is not supplied, the task will create the maximal `Inline` scheduled call.132///133/// Otherwise, the function will take the length value from the `maybe_lookup_len`134/// and find a minimal length value that ensures that the scheduled call will require a Preimage lookup.135fn make_call<T: Config>(maybe_lookup_len: Option<u32>) -> ScheduledCall<T> {136	let bound = EncodedCall::bound() as u32;137	let mut len = match maybe_lookup_len {138		Some(len) => {139			len.min(<T::Preimages as PreimageRecipient<T::Hash>>::MaxSize::get() - 2)140				.max(bound) - 3141		}142		None => bound.saturating_sub(4),143	};144145	loop {146		let c = match bounded::<T>(len) {147			Some(x) => x,148			None => {149				len -= 1;150				continue;151			}152		};153		if c.lookup_needed() == maybe_lookup_len.is_some() {154			break c;155		}156		if maybe_lookup_len.is_some() {157			len += 1;158		} else {159			if len > 0 {160				len -= 1;161			} else {162				break c;163			}164		}165	}166}167168/// Creates an origin for a scheduled call.169///170/// If `signed` is true, it creates the Signed origin from a default account `account("origin", 0, SEED)`.171/// Otherwise, it creates the Root origin.172fn make_origin<T: Config>(signed: bool) -> <T as Config>::PalletsOrigin {173	match signed {174		true => frame_system::RawOrigin::Signed(account("origin", 0, SEED)).into(),175		false => frame_system::RawOrigin::Root.into(),176	}177}178179/// Creates a dummy `WeightCounter` with the maximum possible weight limit.180fn dummy_counter() -> WeightCounter {181	WeightCounter {182		used: Weight::zero(),183		limit: Weight::MAX,184	}185}186187benchmarks! {188	// `service_agendas` when no work is done.189	service_agendas_base {190		let now = T::BlockNumber::from(BLOCK_NUMBER);191		IncompleteSince::<T>::put(now - One::one());192	}: {193		Scheduler::<T>::service_agendas(&mut dummy_counter(), now, 0);194	} verify {195		assert_eq!(IncompleteSince::<T>::get(), Some(now - One::one()));196	}197198	// `service_agenda` when no work is done.199	service_agenda_base {200		let now = BLOCK_NUMBER.into();201		let s in 0 .. T::MaxScheduledPerBlock::get();202		fill_schedule::<T>(now, s)?;203		let mut executed = 0;204	}: {205		Scheduler::<T>::service_agenda(&mut dummy_counter(), &mut executed, now, now, 0);206	} verify {207		assert_eq!(executed, 0);208	}209210	// `service_task` when the task is a non-periodic, non-named, non-fetched call which is not211	// dispatched (e.g. due to being overweight).212	service_task_base {213		let now = BLOCK_NUMBER.into();214		let task = make_task::<T>(false, false, false, None, 0);215		// prevent any tasks from actually being executed as we only want the surrounding weight.216		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::zero() };217	}: {218		let result = Scheduler::<T>::service_task(&mut counter, now, now, 0, true, task);219	} verify {220		//assert_eq!(result, Ok(()));221	}222223	// TODO uncomment if we will use the Preimages224	// // `service_task` when the task is a non-periodic, non-named, fetched call (with a known225	// // preimage length) and which is not dispatched (e.g. due to being overweight).226	// service_task_fetched {227	// 	let s in (EncodedCall::bound() as u32) .. (<T::Preimages as PreimageRecipient<T::Hash>>::MaxSize::get());228	// 	let now = BLOCK_NUMBER.into();229	// 	let task = make_task::<T>(false, false, false, Some(s), 0);230	// 	// prevent any tasks from actually being executed as we only want the surrounding weight.231	// 	let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::zero() };232	// }: {233	// 	let result = Scheduler::<T>::service_task(&mut counter, now, now, 0, true, task);234	// } verify {235	// }236237	// `service_task` when the task is a non-periodic, named, non-fetched call which is not238	// dispatched (e.g. due to being overweight).239	service_task_named {240		let now = BLOCK_NUMBER.into();241		let task = make_task::<T>(false, true, false, None, 0);242		// prevent any tasks from actually being executed as we only want the surrounding weight.243		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::zero() };244	}: {245		let result = Scheduler::<T>::service_task(&mut counter, now, now, 0, true, task);246	} verify {247	}248249	// `service_task` when the task is a periodic, non-named, non-fetched call which is not250	// dispatched (e.g. due to being overweight).251	service_task_periodic {252		let now = BLOCK_NUMBER.into();253		let task = make_task::<T>(true, false, false, None, 0);254		// prevent any tasks from actually being executed as we only want the surrounding weight.255		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::zero() };256	}: {257		let result = Scheduler::<T>::service_task(&mut counter, now, now, 0, true, task);258	} verify {259	}260261	// `execute_dispatch` when the origin is `Signed`, not counting the dispatable's weight.262	execute_dispatch_signed {263		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::MAX };264		let origin = make_origin::<T>(true);265		let call = T::Preimages::realize(&make_call::<T>(None)).unwrap().0;266	}: {267		assert!(Scheduler::<T>::execute_dispatch(&mut counter, origin, call).is_ok());268	}269	verify {270	}271272	// `execute_dispatch` when the origin is not `Signed`, not counting the dispatable's weight.273	execute_dispatch_unsigned {274		let mut counter = WeightCounter { used: Weight::zero(), limit: Weight::MAX };275		let origin = make_origin::<T>(false);276		let call = T::Preimages::realize(&make_call::<T>(None)).unwrap().0;277	}: {278		assert!(Scheduler::<T>::execute_dispatch(&mut counter, origin, call).is_ok());279	}280	verify {281	}282283	schedule {284		let s in 0 .. (T::MaxScheduledPerBlock::get() - 1);285		let when = BLOCK_NUMBER.into();286		let periodic = Some((T::BlockNumber::one(), 100));287		let priority = Some(0);288		// Essentially a no-op call.289		let call = Box::new(SystemCall::set_storage { items: vec![] }.into());290291		fill_schedule::<T>(when, s)?;292	}: _(RawOrigin::Root, when, periodic, priority, call)293	verify {294		ensure!(295			Agenda::<T>::get(when).agenda.len() == (s + 1) as usize,296			"didn't add to schedule"297		);298	}299300	cancel {301		let s in 1 .. T::MaxScheduledPerBlock::get();302		let when = BLOCK_NUMBER.into();303304		fill_schedule::<T>(when, s)?;305		assert_eq!(Agenda::<T>::get(when).agenda.len(), s as usize);306	}: _(RawOrigin::Root, when, 0)307	verify {308		ensure!(309			Lookup::<T>::get(u32_to_name(0)).is_none(),310			"didn't remove from lookup"311		);312		// Removed schedule is NONE313		ensure!(314			Agenda::<T>::get(when).agenda[0].is_none(),315			"didn't remove from schedule"316		);317	}318319	schedule_named {320		let s in 0 .. (T::MaxScheduledPerBlock::get() - 1);321		let id = u32_to_name(s);322		let when = BLOCK_NUMBER.into();323		let periodic = Some((T::BlockNumber::one(), 100));324		let priority = Some(0);325		// Essentially a no-op call.326		let call = Box::new(SystemCall::set_storage { items: vec![] }.into());327328		fill_schedule::<T>(when, s)?;329	}: _(RawOrigin::Root, id, when, periodic, priority, call)330	verify {331		ensure!(332			Agenda::<T>::get(when).agenda.len() == (s + 1) as usize,333			"didn't add to schedule"334		);335	}336337	cancel_named {338		let s in 1 .. T::MaxScheduledPerBlock::get();339		let when = BLOCK_NUMBER.into();340341		fill_schedule::<T>(when, s)?;342	}: _(RawOrigin::Root, u32_to_name(0))343	verify {344		ensure!(345			Lookup::<T>::get(u32_to_name(0)).is_none(),346			"didn't remove from lookup"347		);348		// Removed schedule is NONE349		ensure!(350			Agenda::<T>::get(when).agenda[0].is_none(),351			"didn't remove from schedule"352		);353	}354355	change_named_priority {356		let origin: RawOrigin<T::AccountId> = frame_system::RawOrigin::Root;357		let s in 1 .. T::MaxScheduledPerBlock::get();358		let when = BLOCK_NUMBER.into();359		let idx = s - 1;360		let id = u32_to_name(idx);361		let priority = 42;362		fill_schedule::<T>(when, s)?;363	}: _(origin, id, priority)364	verify {365		ensure!(366			Agenda::<T>::get(when).agenda[idx as usize].clone().unwrap().priority == priority,367			"didn't change the priority"368		);369	}370371	impl_benchmark_test_suite!(Scheduler, crate::mock::new_test_ext(), crate::mock::Test);372}