bench: attempt to fix default accountid for dids

KILTprotocol · Feb 1, 2022 · 1c2acfb · 1c2acfb
1 parent 844b60b
commit 1c2acfb
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/pallets/did/src/benchmarking.rs b/pallets/did/src/benchmarking.rs
@@ -119,7 +119,8 @@ benchmarks! {
 	where_clause {
 		where
 		T::DidIdentifier: From<AccountId>,
-		<T as frame_system::Config>::Origin: From<RawOrigin<T::DidIdentifier>>
+		<T as frame_system::Config>::Origin: From<RawOrigin<T::DidIdentifier>>,
+		<T as frame_system::Config>::AccountId: From<AccountId>,
 	}
 
 	/* create extrinsic */

diff --git a/pallets/did/src/mock_utils.rs b/pallets/did/src/mock_utils.rs
@@ -93,12 +93,16 @@ pub fn generate_base_did_creation_details<T: Config>(
 	}
 }
 
-pub fn generate_base_did_details<T: Config>(authentication_key: DidVerificationKey) -> DidDetails<T> {
+pub fn generate_base_did_details<T>(authentication_key: DidVerificationKey) -> DidDetails<T>
+where
+	T: crate::Config,
+	<T as frame_system::Config>::AccountId: From<runtime_common::AccountId>,
+{
 	DidDetails::new(
 		authentication_key,
 		BlockNumberOf::<T>::default(),
 		Deposit {
-			owner: Default::default(),
+			owner: runtime_common::AccountId::new([0u8; 32]).into(),
 			amount: Zero::zero(),
 		},
 	)

diff --git a/pallets/did/src/tests.rs b/pallets/did/src/tests.rs
@@ -2834,7 +2834,7 @@ fn check_null_key_error() {
 		did,
 		caller.clone(),
 	);
-	let signature = ed25519::Signature::default();
+	let signature = ed25519::Signature::from_raw([0u8; 64]);
 
 	ExtBuilder::default().build(None).execute_with(|| {
 		assert_noop!(