Skip to content
Snippets Groups Projects
Commit 69f0c5a8 authored by Hugo Trentesaux's avatar Hugo Trentesaux
Browse files

rename force_add_cert to do_add_cert_checked

parent 4849a7b6
No related branches found
No related tags found
1 merge request!166refac membership renewal
This commit is part of merge request !166. Comments created here will be created in the context of that merge request.
......@@ -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>::force_add_cert(RawOrigin::Root.into(), j.into(), receiver, false)?;
Pallet::<T, I>::do_add_cert_checked(RawOrigin::Root.into(), 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>::force_add_cert(RawOrigin::Root.into(), issuer, receiver, false)?;
Pallet::<T, I>::do_add_cert_checked(RawOrigin::Root.into(), 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)
......
......@@ -20,7 +20,7 @@ use frame_support::weights::{constants::RocksDbWeight, Weight};
/// Weight functions needed for pallet_universal_dividend.
pub trait WeightInfo {
fn force_add_cert() -> Weight;
fn do_add_cert_checked() -> Weight;
fn add_cert() -> Weight;
fn del_cert() -> Weight;
fn remove_all_certs_received_by(i: u32) -> Weight;
......@@ -32,7 +32,7 @@ impl WeightInfo for () {
// Storage: Parameters ParametersStorage (r:1 w:0)
// Storage: Cert StorageCertsRemovableOn (r:1 w:1)
// Storage: Cert CertsByReceiver (r:1 w:1)
fn force_add_cert() -> Weight {
fn do_add_cert_checked() -> Weight {
// Minimum execution time: 221_467 nanoseconds.
Weight::from_ref_time(227_833_000 as u64)
.saturating_add(RocksDbWeight::get().reads(5 as u64))
......
......@@ -303,9 +303,9 @@ impl<T: Config<I>, I: 'static> pallet_identity::traits::OnIdtyChange<T> for Pall
fn on_idty_change(idty_index: IdtyIndex, idty_event: &IdtyEvent<T>) -> Weight {
match idty_event {
IdtyEvent::Created { creator } => {
if let Err(e) =
<pallet_certification::Pallet<T, I>>::force_add_cert(*creator, idty_index, true)
{
if let Err(e) = <pallet_certification::Pallet<T, I>>::do_add_cert_checked(
*creator, idty_index, true,
) {
sp_std::if_std! {
println!("fail to force add cert: {:?}", e)
}
......
......@@ -50,7 +50,7 @@ impl<T: frame_system::Config> pallet_certification::WeightInfo for WeightInfo<T>
// Storage: Parameters ParametersStorage (r:1 w:0)
// Storage: Cert StorageCertsRemovableOn (r:1 w:1)
// Storage: Cert CertsByReceiver (r:1 w:1)
fn force_add_cert() -> Weight {
fn do_add_cert_checked() -> Weight {
// Minimum execution time: 48_363 nanoseconds.
Weight::from_ref_time(59_125_000 as u64)
.saturating_add(T::DbWeight::get().reads(5 as u64))
......
......@@ -50,7 +50,7 @@ impl<T: frame_system::Config> pallet_certification::WeightInfo for WeightInfo<T>
// Storage: Parameters ParametersStorage (r:1 w:0)
// Storage: SmithCert StorageCertsRemovableOn (r:1 w:1)
// Storage: SmithCert CertsByReceiver (r:1 w:1)
fn force_add_cert() -> Weight {
fn do_add_cert_checked() -> Weight {
// Minimum execution time: 57_241 nanoseconds.
Weight::from_ref_time(69_631_000 as u64)
.saturating_add(T::DbWeight::get().reads(5 as u64))
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment