Skip to content
Snippets Groups Projects
Commit fb6f3d78 authored by Pascal Engélibert's avatar Pascal Engélibert :bicyclist: Committed by Hugo Trentesaux
Browse files

certification/benchmark: fix add_certs number

parent 91acf6cd
No related branches found
No related tags found
1 merge request!310Optimize do_remove_all_certs_received_by
Pipeline #39554 skipped
...@@ -37,11 +37,11 @@ mod benchmarks { ...@@ -37,11 +37,11 @@ mod benchmarks {
fn add_certs<T: Config>(i: u32, receiver: T::IdtyIndex) -> Result<(), &'static str> { fn add_certs<T: Config>(i: u32, receiver: T::IdtyIndex) -> Result<(), &'static str> {
Pallet::<T>::remove_all_certs_received_by(RawOrigin::Root.into(), receiver)?; 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)?; Pallet::<T>::do_add_cert_checked(j.into(), receiver, false)?;
} }
assert!( assert!(
CertsByReceiver::<T>::get(receiver).len() as u32 == i - 1, CertsByReceiver::<T>::get(receiver).len() as u32 == i,
"Certs not added", "Certs not added",
); );
Ok(()) Ok(())
...@@ -167,7 +167,7 @@ mod benchmarks { ...@@ -167,7 +167,7 @@ mod benchmarks {
Pallet::<T>::do_remove_all_certs_received_by(receiver); Pallet::<T>::do_remove_all_certs_received_by(receiver);
} }
for issuer in 1..i { for issuer in 1..i + 1 {
assert_has_event::<T>( assert_has_event::<T>(
Event::<T>::CertRemoved { Event::<T>::CertRemoved {
issuer: issuer.into(), issuer: issuer.into(),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment