diff --git a/xcm/pallet-xcm-benchmarks/src/generic/benchmarking.rs b/xcm/pallet-xcm-benchmarks/src/generic/benchmarking.rs index 62d0df265c42..b8104067fb97 100644 --- a/xcm/pallet-xcm-benchmarks/src/generic/benchmarking.rs +++ b/xcm/pallet-xcm-benchmarks/src/generic/benchmarking.rs @@ -20,10 +20,7 @@ use codec::Encode; use frame_benchmarking::{benchmarks, BenchmarkError}; use frame_support::dispatch::GetDispatchInfo; use sp_std::vec; -use xcm::{ - latest::{prelude::*, MultiAssets}, - DoubleEncoded, -}; +use xcm::{latest::prelude::*, DoubleEncoded}; benchmarks! { query_holding { @@ -70,26 +67,6 @@ benchmarks! { } - // Worst case scenario for this benchmark is a large number of assets to - // filter through the reserve. - reserve_asset_deposited { - const MAX_ASSETS: u32 = 100; // TODO when executor has a built in limit, use it here. #4426 - let mut executor = new_executor::(Default::default()); - let assets = (0..MAX_ASSETS).map(|i| MultiAsset { - id: Abstract(i.encode()), - fun: Fungible(i as u128), - - }).collect::>(); - let multiassets: MultiAssets = assets.into(); - - let instruction = Instruction::ReserveAssetDeposited(multiassets.clone()); - let xcm = Xcm(vec![instruction]); - }: { - executor.execute(xcm).map_err(|_| BenchmarkError::Skip)?; - } verify { - assert_eq!(executor.holding, multiassets.into()); - } - query_response { let mut executor = new_executor::(Default::default()); let (query_id, response) = T::worst_case_response();