From fb6f3d78ec7039c51f0fa4dc2d3d9170d6936ba9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pascal=20Eng=C3=A9libert?= <tuxmain@zettascript.org>
Date: Wed, 15 Jan 2025 14:21:04 +0100
Subject: [PATCH] certification/benchmark: fix add_certs number

---
 pallets/certification/src/benchmarking.rs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/pallets/certification/src/benchmarking.rs b/pallets/certification/src/benchmarking.rs
index 6f9d2ee32..668d5cdfa 100644
--- a/pallets/certification/src/benchmarking.rs
+++ b/pallets/certification/src/benchmarking.rs
@@ -37,11 +37,11 @@ mod benchmarks {
 
     fn add_certs<T: Config>(i: u32, receiver: T::IdtyIndex) -> Result<(), &'static str> {
         Pallet::<T>::remove_all_certs_received_by(RawOrigin::Root.into(), receiver)?;
-        for j in 1..i {
+        for j in 1..i + 1 {
             Pallet::<T>::do_add_cert_checked(j.into(), receiver, false)?;
         }
         assert!(
-            CertsByReceiver::<T>::get(receiver).len() as u32 == i - 1,
+            CertsByReceiver::<T>::get(receiver).len() as u32 == i,
             "Certs not added",
         );
         Ok(())
@@ -167,7 +167,7 @@ mod benchmarks {
             Pallet::<T>::do_remove_all_certs_received_by(receiver);
         }
 
-        for issuer in 1..i {
+        for issuer in 1..i + 1 {
             assert_has_event::<T>(
                 Event::<T>::CertRemoved {
                     issuer: issuer.into(),
-- 
GitLab