--- a/pallets/nonfungible/src/benchmarking.rs +++ b/pallets/nonfungible/src/benchmarking.rs @@ -228,7 +228,7 @@ owner: sub; collection: collection(owner); owner: cross_from_sub; operator: cross_sub; }; - }: {>::set_allowance_for_all(&collection, &owner, &operator, true)} + }: {>::set_allowance_for_all(&collection, &owner, &operator, true)?} allowance_for_all { bench_init!{ --- a/pallets/refungible/src/benchmarking.rs +++ b/pallets/refungible/src/benchmarking.rs @@ -22,21 +22,17 @@ use frame_benchmarking::{benchmarks, account}; use pallet_common::{ bench_init, - benchmarking::{create_collection_raw, property_key, property_value, create_data}, + benchmarking::{create_collection_raw, property_key, property_value}, }; -use sp_core::H160; use sp_std::prelude::*; -use up_data_structs::{ - CollectionMode, MAX_ITEMS_PER_BATCH, MAX_PROPERTIES_PER_ITEM, CUSTOM_DATA_LIMIT, - budget::Unlimited, -}; +use up_data_structs::{CollectionMode, MAX_ITEMS_PER_BATCH, MAX_PROPERTIES_PER_ITEM, budget::Unlimited}; const SEED: u32 = 1; -fn create_max_item_data( - users: impl IntoIterator, -) -> CreateItemData { - CreateItemData { +fn create_max_item_data( + users: impl IntoIterator, +) -> CreateItemData { + CreateItemData:: { users: users .into_iter() .collect::>() @@ -51,7 +47,7 @@ sender: &T::CrossAccountId, users: impl IntoIterator, ) -> Result { - let data: CreateItemData = create_max_item_data(users); + let data: CreateItemData = create_max_item_data::(users); >::create_item(&collection, sender, data, &Unlimited)?; Ok(TokenId(>::get(&collection.id))) } @@ -83,7 +79,7 @@ owner: sub; collection: collection(owner); sender: cross_from_sub(owner); to: cross_sub; }; - let data = (0..b).map(|_| create_max_item_data([(to.clone(), 200)])).collect(); + let data = (0..b).map(|_| create_max_item_data::([(to.clone(), 200)])).collect(); }: {>::create_multiple_items(&collection, &sender, data, &Unlimited)?} create_multiple_items_ex_multiple_items { @@ -94,7 +90,7 @@ }; let data = (0..b).map(|t| { bench_init!(to: cross_sub(t);); - create_max_item_data([(to, 200)]) + create_max_item_data::([(to, 200)]) }).collect(); }: {>::create_multiple_items(&collection, &sender, data, &Unlimited)?} @@ -104,7 +100,7 @@ owner: sub; collection: collection(owner); sender: cross_from_sub(owner); }; - let data = vec![create_max_item_data((0..b).map(|u| { + let data = vec![create_max_item_data::((0..b).map(|u| { bench_init!(to: cross_sub(u);); (to, 200) }))].try_into().unwrap(); @@ -296,7 +292,7 @@ owner: sub; collection: collection(owner); owner: cross_from_sub; operator: cross_sub; }; - }: {>::set_allowance_for_all(&collection, &owner, &operator, true)} + }: {>::set_allowance_for_all(&collection, &owner, &operator, true)?} allowance_for_all { bench_init!{