feat: delegation pallet storage refactor

pallets/attestation/src/default_weights.rs

@@ -19,29 +19,24 @@
 //! Autogenerated weights for attestation
 //!
 //! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 3.0.0
-//! DATE: 2021-06-11, STEPS: {{cmd.steps}}\, REPEAT: {{cmd.repeat}}\, LOW RANGE: {{cmd.lowest_range_values}}\, HIGH RANGE: {{cmd.highest_range_values}}\
+//! DATE: 2021-07-21, STEPS: {{cmd.steps}}\, REPEAT: {{cmd.repeat}}\, LOW RANGE: {{cmd.lowest_range_values}}\, HIGH RANGE: {{cmd.highest_range_values}}\
 //! EXECUTION: Some(Wasm), WASM-EXECUTION: Compiled, CHAIN: Some("dev"), DB CACHE: 128
 
 // Executed Command:
-// ./kilt-parachain
+// target/release/kilt-parachain
 // benchmark
-// --chain
-// dev
-// --execution
-// wasm
-// --wasm-execution
-// compiled
-// --pallet
-// attestation
-// --extrinsic
-// *
-// --steps
-// 1
-// --repeat
-// 10
+// --chain=dev
+// --execution=wasm
+// --wasm-execution=Compiled
 // --heap-pages=4096
-// --output=../../pallets/attestation/src/default_weights.rs
-// --template=../../.maintain/weight-template.hbs
+// --extrinsic=*
+// --pallet=attestation
+// --steps=1
+// --repeat=20
+// --template
+// .maintain/weight-template.hbs
+// --output
+// pallets/attestation/src/default_weights.rs
 
 
 #![cfg_attr(rustfmt, rustfmt_skip)]
@@ -61,14 +56,14 @@ pub trait WeightInfo {
 pub struct SubstrateWeight<T>(PhantomData<T>);
 impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 	fn add() -> Weight {
-		(37_981_000_u64)
+		(38_091_000_u64)
 			.saturating_add(T::DbWeight::get().reads(5_u64))
 			.saturating_add(T::DbWeight::get().writes(2_u64))
 	}
 	fn revoke(d: u32, ) -> Weight {
-		(23_806_000_u64)
-			// Standard Error: 20_000
-			.saturating_add((4_670_000_u64).saturating_mul(d as Weight))
+		(25_042_000_u64)
+			// Standard Error: 28_000
+			.saturating_add((4_866_000_u64).saturating_mul(d as Weight))
 			.saturating_add(T::DbWeight::get().reads(2_u64))
 			.saturating_add(T::DbWeight::get().reads((1_u64).saturating_mul(d as Weight)))
 			.saturating_add(T::DbWeight::get().writes(1_u64))
@@ -78,14 +73,14 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 // For backwards compatibility and tests
 impl WeightInfo for () {
 	fn add() -> Weight {
-		(37_981_000_u64)
+		(38_091_000_u64)
 			.saturating_add(RocksDbWeight::get().reads(5_u64))
 			.saturating_add(RocksDbWeight::get().writes(2_u64))
 	}
 	fn revoke(d: u32, ) -> Weight {
-		(23_806_000_u64)
-			// Standard Error: 20_000
-			.saturating_add((4_670_000_u64).saturating_mul(d as Weight))
+		(25_042_000_u64)
+			// Standard Error: 28_000
+			.saturating_add((4_866_000_u64).saturating_mul(d as Weight))
 			.saturating_add(RocksDbWeight::get().reads(2_u64))
 			.saturating_add(RocksDbWeight::get().reads((1_u64).saturating_mul(d as Weight)))
 			.saturating_add(RocksDbWeight::get().writes(1_u64))

pallets/attestation/src/lib.rs

@@ -224,21 +224,22 @@ pub mod pallet {
 
 			// Check for validity of the delegation node if specified.
 			if let Some(delegation_id) = delegation_id {
-				let delegation = <delegation::Delegations<T>>::get(delegation_id)
+				let delegation = <delegation::DelegationNodes<T>>::get(delegation_id)
 					.ok_or(delegation::Error::<T>::DelegationNotFound)?;
 
-				ensure!(!delegation.revoked, Error::<T>::DelegationRevoked);
+				ensure!(!delegation.details.revoked, Error::<T>::DelegationRevoked);
 
-				ensure!(delegation.owner == attester, Error::<T>::NotDelegatedToAttester);
+				ensure!(delegation.details.owner == attester, Error::<T>::NotDelegatedToAttester);
 
 				ensure!(
-					(delegation.permissions & delegation::Permissions::ATTEST) == delegation::Permissions::ATTEST,
+					(delegation.details.permissions & delegation::Permissions::ATTEST)
+						== delegation::Permissions::ATTEST,
 					Error::<T>::DelegationUnauthorizedToAttest
 				);
 
 				// Check if the CType of the delegation is matching the CType of the attestation
-				let root =
-					<delegation::Roots<T>>::get(delegation.root_id).ok_or(delegation::Error::<T>::RootNotFound)?;
+				let root = <delegation::DelegationHierarchies<T>>::get(delegation.hierarchy_root_id)
+					.ok_or(delegation::Error::<T>::HierarchyNotFound)?;
 				ensure!(root.ctype_hash == ctype_hash, Error::<T>::CTypeMismatch);
 
 				// If the attestation is based on a delegation, store separately