diff --git a/substrate/frame/balances/src/benchmarking.rs b/substrate/frame/balances/src/benchmarking.rs index c825300218d462a8465df2e80afda4e02c951cf3..a8f92e9b52c422c4accc2731669701a374a54d06 100644 --- a/substrate/frame/balances/src/benchmarking.rs +++ b/substrate/frame/balances/src/benchmarking.rs @@ -65,7 +65,9 @@ mod benchmarks { #[extrinsic_call] _(RawOrigin::Signed(caller.clone()), recipient_lookup, transfer_amount); + #[cfg(not(feature = "insecure_zero_ed"))] assert_eq!(Balances::<T, I>::free_balance(&caller), Zero::zero()); + assert_eq!(Balances::<T, I>::free_balance(&recipient), transfer_amount); } @@ -173,7 +175,9 @@ mod benchmarks { #[extrinsic_call] _(RawOrigin::Root, source_lookup, recipient_lookup, transfer_amount); + #[cfg(not(feature = "insecure_zero_ed"))] assert_eq!(Balances::<T, I>::free_balance(&source), Zero::zero()); + assert_eq!(Balances::<T, I>::free_balance(&recipient), transfer_amount); } @@ -208,7 +212,9 @@ mod benchmarks { #[extrinsic_call] transfer_allow_death(RawOrigin::Signed(caller.clone()), recipient_lookup, transfer_amount); + #[cfg(not(feature = "insecure_zero_ed"))] assert_eq!(Balances::<T, I>::free_balance(&caller), Zero::zero()); + assert_eq!(Balances::<T, I>::free_balance(&recipient), transfer_amount); } diff --git a/substrate/frame/balances/src/tests/currency_tests.rs b/substrate/frame/balances/src/tests/currency_tests.rs index a6377c3ad72e8b5185422ebf2e2caefd045b9251..3525fb26193d424cbc500c13845c8e282e86e50a 100644 --- a/substrate/frame/balances/src/tests/currency_tests.rs +++ b/substrate/frame/balances/src/tests/currency_tests.rs @@ -1136,6 +1136,7 @@ fn operations_on_dead_account_should_not_change_state() { #[test] #[should_panic = "The existential deposit must be greater than zero!"] +#[cfg(not(feature = "insecure_zero_ed"))] fn zero_ed_is_prohibited() { // These functions all use `mutate_account` which may introduce a storage change when // the account never existed to begin with, and shouldn't exist in the end.