diff --git a/pallets/duniter-wot/src/tests.rs b/pallets/duniter-wot/src/tests.rs index 910991c656a19f181f186c2d0ffd31c12c29ec46..7b95a09c1b23d05be8b56480374726d4a4ea3dd3 100644 --- a/pallets/duniter-wot/src/tests.rs +++ b/pallets/duniter-wot/src/tests.rs @@ -73,7 +73,10 @@ fn test_create_idty_ok() { events[1], EventRecord { phase: Phase::Initialization, - event: Event::Identity(pallet_identity::Event::IdtyCreated(6, 6)), + event: Event::Identity(pallet_identity::Event::IdtyCreated { + idty_index: 6, + owner_key: 6, + }), topics: vec![], } ); @@ -145,7 +148,7 @@ fn test_new_idty_validation() { events[2], EventRecord { phase: Phase::Initialization, - event: Event::Identity(pallet_identity::Event::IdtyValidated(6,)), + event: Event::Identity(pallet_identity::Event::IdtyValidated { idty_index: 6 }), topics: vec![], } ); @@ -194,11 +197,11 @@ fn test_confirm_idty_ok() { events[1], EventRecord { phase: Phase::Initialization, - event: Event::Identity(pallet_identity::Event::IdtyConfirmed( - 6, - 6, - IdtyName::from("Ferdie"), - )), + event: Event::Identity(pallet_identity::Event::IdtyConfirmed { + idty_index: 6, + owner_key: 6, + name: IdtyName::from("Ferdie"), + }), topics: vec![], } ); diff --git a/pallets/identity/src/lib.rs b/pallets/identity/src/lib.rs index c35f8f4acc77dc1e2b91473b0271a052990460e6..b4d0c19a51f4563ae3db52a5873e05d56aae3d8e 100644 --- a/pallets/identity/src/lib.rs +++ b/pallets/identity/src/lib.rs @@ -200,13 +200,20 @@ pub mod pallet { pub enum Event<T: Config> { /// A new identity has been created /// [idty_index, owner_key] - IdtyCreated(T::IdtyIndex, T::AccountId), + IdtyCreated { + idty_index: T::IdtyIndex, + owner_key: T::AccountId, + }, /// An identity has been confirmed by it's owner /// [idty_index, owner_key, name] - IdtyConfirmed(T::IdtyIndex, T::AccountId, IdtyName), + IdtyConfirmed { + idty_index: T::IdtyIndex, + owner_key: T::AccountId, + name: IdtyName, + }, /// An identity has been validated /// [idty_index] - IdtyValidated(T::IdtyIndex), + IdtyValidated { idty_index: T::IdtyIndex }, } // CALLS // @@ -266,7 +273,10 @@ pub mod pallet { ); IdentitiesRemovableOn::<T>::append(removable_on, (idty_index, IdtyStatus::Created)); IdentityIndexOf::<T>::insert(owner_key.clone(), idty_index); - Self::deposit_event(Event::IdtyCreated(idty_index, owner_key)); + Self::deposit_event(Event::IdtyCreated { + idty_index, + owner_key, + }); T::OnIdtyChange::on_idty_change(idty_index, IdtyEvent::Created { creator }); Ok(().into()) } @@ -302,7 +312,11 @@ pub mod pallet { <Identities<T>>::insert(idty_index, idty_value); <IdentitiesNames<T>>::insert(idty_name.clone(), ()); - Self::deposit_event(Event::IdtyConfirmed(idty_index, who, idty_name)); + Self::deposit_event(Event::IdtyConfirmed { + idty_index, + owner_key: who, + name: idty_name, + }); T::OnIdtyChange::on_idty_change(idty_index, IdtyEvent::Confirmed); Ok(().into()) } @@ -332,7 +346,7 @@ pub mod pallet { idty_value.status = IdtyStatus::Validated; <Identities<T>>::insert(idty_index, idty_value); - Self::deposit_event(Event::IdtyValidated(idty_index)); + Self::deposit_event(Event::IdtyValidated { idty_index }); T::OnIdtyChange::on_idty_change(idty_index, IdtyEvent::Validated); Ok(().into()) diff --git a/pallets/identity/src/tests.rs b/pallets/identity/src/tests.rs index 313b7b5e288288adac1e44e68cc47a08de0ae17d..462c437ae0e9aebb61b8e654994669aca8d88847 100644 --- a/pallets/identity/src/tests.rs +++ b/pallets/identity/src/tests.rs @@ -62,7 +62,10 @@ fn test_create_identity_ok() { events[1], EventRecord { phase: Phase::Initialization, - event: Event::Identity(crate::Event::IdtyCreated(2, 2)), + event: Event::Identity(crate::Event::IdtyCreated { + idty_index: 2, + owner_key: 2, + }), topics: vec![], } ); @@ -86,7 +89,10 @@ fn test_idty_creation_period() { events[1], EventRecord { phase: Phase::Initialization, - event: Event::Identity(crate::Event::IdtyCreated(2, 2)), + event: Event::Identity(crate::Event::IdtyCreated { + idty_index: 2, + owner_key: 2, + }), topics: vec![], } ); @@ -108,7 +114,10 @@ fn test_idty_creation_period() { events[1], EventRecord { phase: Phase::Initialization, - event: Event::Identity(crate::Event::IdtyCreated(3, 3)), + event: Event::Identity(crate::Event::IdtyCreated { + idty_index: 3, + owner_key: 3, + }), topics: vec![], } ); diff --git a/pallets/universal-dividend/src/lib.rs b/pallets/universal-dividend/src/lib.rs index ce1ae01bf4f5a6d0da4eebd0b0e98c8e3b598579..ee04af70627433e9c9ec30de9f91ddff7d464273 100644 --- a/pallets/universal-dividend/src/lib.rs +++ b/pallets/universal-dividend/src/lib.rs @@ -177,11 +177,19 @@ pub mod pallet { #[pallet::generate_deposit(pub(super) fn deposit_event)] pub enum Event<T: Config> { /// A new universal dividend is created - /// [ud_amout, members_count] - NewUdCreated(BalanceOf<T>, BalanceOf<T>), + /// [amout, members_count] + NewUdCreated { + amount: BalanceOf<T>, + monetary_mass: BalanceOf<T>, + members_count: BalanceOf<T>, + }, /// The universal dividend has been re-evaluated /// [new_ud_amount, monetary_mass, members_count] - UdReevalued(BalanceOf<T>, BalanceOf<T>, BalanceOf<T>), + UdReevalued { + new_ud_amount: BalanceOf<T>, + monetary_mass: BalanceOf<T>, + members_count: BalanceOf<T>, + }, } // INTERNAL FUNCTIONS // @@ -197,10 +205,14 @@ pub mod pallet { Self::write_ud_history(n, account_id, ud_amount); } - <MonetaryMassStorage<T>>::put( - monetary_mass.saturating_add(ud_amount.saturating_mul(members_count)), - ); - Self::deposit_event(Event::NewUdCreated(ud_amount, members_count)); + let new_monetary_mass = + monetary_mass.saturating_add(ud_amount.saturating_mul(members_count)); + MonetaryMassStorage::<T>::put(new_monetary_mass); + Self::deposit_event(Event::NewUdCreated { + amount: ud_amount, + members_count, + monetary_mass: new_monetary_mass, + }); total_weight } @@ -239,11 +251,11 @@ pub mod pallet { <CurrentUdStorage<T>>::put(new_ud_amount); - Self::deposit_event(Event::UdReevalued( + Self::deposit_event(Event::UdReevalued { new_ud_amount, monetary_mass, members_count, - )); + }); total_weight } diff --git a/pallets/universal-dividend/src/tests.rs b/pallets/universal-dividend/src/tests.rs index c6551c5ad518b80ef69712489e84105b5a6cd83d..0f0510261b15c724e34919c8573d55ee5e938a71 100644 --- a/pallets/universal-dividend/src/tests.rs +++ b/pallets/universal-dividend/src/tests.rs @@ -47,7 +47,11 @@ fn test_ud_creation() { events[9], EventRecord { phase: Phase::Initialization, - event: Event::UniversalDividend(crate::Event::NewUdCreated(1000, 3)), + event: Event::UniversalDividend(crate::Event::NewUdCreated { + amount: 1_000, + monetary_mass: 3_000, + members_count: 3, + }), topics: vec![], } );