From af021cbaf68e58ac808573166d57af8f92c7e049 Mon Sep 17 00:00:00 2001
From: Hugo Trentesaux <hugo.trentesaux@lilo.org>
Date: Mon, 22 May 2023 09:46:30 +0200
Subject: [PATCH] fix build benchmark (nodes/rust/duniter-v2s!169)
* Revert "run benchmarks"
This reverts commit 2c7751318c17054f4451e011531a920acf381935.
* run benchmarks
commit only section which changed
* fix build benchmark
---
pallets/certification/src/benchmarking.rs | 4 ++--
pallets/membership/src/benchmarking.rs | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/pallets/certification/src/benchmarking.rs b/pallets/certification/src/benchmarking.rs
index 25f12579e..47fda2c72 100644
--- a/pallets/certification/src/benchmarking.rs
+++ b/pallets/certification/src/benchmarking.rs
@@ -34,7 +34,7 @@ fn assert_has_event<T: Config<I>, I: 'static>(generic_event: <T as Config<I>>::R
fn add_certs<T: Config<I>, I: 'static>(i: u32, receiver: T::IdtyIndex) -> Result<(), &'static str> {
Pallet::<T, I>::remove_all_certs_received_by(RawOrigin::Root.into(), receiver)?;
for j in 1..i {
- Pallet::<T, I>::do_add_cert_checked(RawOrigin::Root.into(), j.into(), receiver, false)?;
+ Pallet::<T, I>::do_add_cert_checked(j.into(), receiver, false)?;
}
assert!(
CertsByReceiver::<T, I>::get(receiver).len() as u32 == i - 1,
@@ -63,7 +63,7 @@ benchmarks_instance_pallet! {
del_cert {
let issuer: T::IdtyIndex = 1.into();
let receiver: T::IdtyIndex = 0.into();
- Pallet::<T, I>::do_add_cert_checked(RawOrigin::Root.into(), issuer, receiver, false)?;
+ Pallet::<T, I>::do_add_cert_checked(issuer, receiver, false)?;
let receiver_cert: u32 = StorageIdtyCertMeta::<T, I>::get(receiver).received_count;
let issuer_cert: u32 = StorageIdtyCertMeta::<T, I>::get(issuer).issued_count;
}: _<T::RuntimeOrigin>(RawOrigin::Root.into(), issuer, receiver)
diff --git a/pallets/membership/src/benchmarking.rs b/pallets/membership/src/benchmarking.rs
index 3196d10c5..9b0ad18d0 100644
--- a/pallets/membership/src/benchmarking.rs
+++ b/pallets/membership/src/benchmarking.rs
@@ -61,7 +61,7 @@ benchmarks_instance_pallet! {
PendingMembership::<T, I>::insert(idty.clone(), T::MetaData::default());
let caller: T::AccountId = T::AccountIdOf::convert(idty.clone()).unwrap();
let caller_origin: <T as frame_system::Config>::RuntimeOrigin = RawOrigin::Signed(caller.clone()).into();
- }: _<T::RuntimeOrigin>(caller_origin, Some(idty))
+ }: _<T::RuntimeOrigin>(caller_origin)
verify {
assert_has_event::<T, I>(Event::<T, I>::MembershipAcquired(idty).into());
}
@@ -69,7 +69,7 @@ benchmarks_instance_pallet! {
let idty: T::IdtyId = 3.into();
let caller: T::AccountId = T::AccountIdOf::convert(idty.clone()).unwrap();
let caller_origin: <T as frame_system::Config>::RuntimeOrigin = RawOrigin::Signed(caller.clone()).into();
- }: _<T::RuntimeOrigin>(caller_origin, Some(idty))
+ }: _<T::RuntimeOrigin>(caller_origin)
verify {
assert_has_event::<T, I>(Event::<T, I>::MembershipRenewed(idty).into());
}
@@ -77,7 +77,7 @@ benchmarks_instance_pallet! {
let idty: T::IdtyId = 3.into();
let caller: T::AccountId = T::AccountIdOf::convert(idty.clone()).unwrap();
let caller_origin: <T as frame_system::Config>::RuntimeOrigin = RawOrigin::Signed(caller.clone()).into();
- }: _<T::RuntimeOrigin>(caller_origin, Some(idty))
+ }: _<T::RuntimeOrigin>(caller_origin)
verify {
assert_has_event::<T, I>(Event::<T, I>::MembershipRevoked(idty).into());
}
--
GitLab