From a897bf2a13d5d8ff389c5d8c84485f1183565410 Mon Sep 17 00:00:00 2001 From: Jegor Sidorenko <5252494+jsidorenko@users.noreply.github.com> Date: Wed, 5 Jul 2023 12:04:31 +0200 Subject: [PATCH] Use set_balance() instead of write_balance() in benchmark (#14497) * Use set_balance instead of write_balance in benchmark * Another approach * Use BalanceOf::::max_value() / 1000 * use a regular div instead of a checked one --- frame/asset-conversion/src/benchmarking.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/frame/asset-conversion/src/benchmarking.rs b/frame/asset-conversion/src/benchmarking.rs index 0a212fb5ceeb4..795c75703e684 100644 --- a/frame/asset-conversion/src/benchmarking.rs +++ b/frame/asset-conversion/src/benchmarking.rs @@ -23,13 +23,13 @@ use frame_support::{ assert_ok, storage::bounded_vec::BoundedVec, traits::{ - fungible::{Inspect as InspectFungible, Unbalanced}, + fungible::{Inspect as InspectFungible, Mutate as MutateFungible, Unbalanced}, fungibles::{Create, Inspect, Mutate}, }, }; use frame_system::RawOrigin as SystemOrigin; use sp_runtime::traits::{Bounded, StaticLookup}; -use sp_std::prelude::*; +use sp_std::{ops::Div, prelude::*}; use crate::Pallet as AssetConversion; @@ -55,7 +55,7 @@ where let caller: T::AccountId = whitelisted_caller(); let caller_lookup = T::Lookup::unlookup(caller.clone()); if let Ok(asset_id) = T::MultiAssetIdConverter::try_convert(asset) { - assert_ok!(T::Currency::write_balance(&caller, BalanceOf::::max_value())); + T::Currency::set_balance(&caller, BalanceOf::::max_value().div(1000u32.into())); assert_ok!(T::Assets::create(asset_id.clone(), caller.clone(), true, 1.into())); assert_ok!(T::Assets::mint_into(asset_id, &caller, INITIAL_ASSET_BALANCE.into())); }