Fix tests build

pallets/dkg-metadata/Cargo.toml

@@ -37,6 +37,7 @@ sp-core = { workspace = true }
 sp-io = { workspace = true }
 sp-staking = { workspace = true }
 sp-keystore = { workspace = true }
+sp-state-machine = { workspace = true }
 k256 = { version = "0.11.5", default-features = false, features = ["arithmetic", "ecdsa"] }
 
 [features]

pallets/dkg-metadata/src/mock.rs

@@ -15,7 +15,7 @@
 // construct_runtime requires this
 #![allow(clippy::from_over_into, clippy::unwrap_used)]
 use codec::{Decode, Encode, MaxEncodedLen};
-use frame_support::{construct_runtime, parameter_types, BasicExternalities};
+use frame_support::{construct_runtime, parameter_types};
 use frame_system::EnsureRoot;
 use sp_core::{
 	sr25519::{self, Signature},
@@ -33,6 +33,7 @@ use sp_runtime::{
 	},
 	BuildStorage, Percent, Permill,
 };
+use sp_state_machine::BasicExternalities;
 use std::{sync::Arc, vec};
 
 use crate as pallet_dkg_metadata;

pallets/dkg-proposal-handler/src/lib.rs

@@ -462,15 +462,11 @@ pub mod pallet {
 				let data = prop_batch.data();
 
 				// check the signature is valid
-				let _result = ensure_signed_by_dkg::<pallet_dkg_metadata::Pallet<T>>(
+				let result = ensure_signed_by_dkg::<pallet_dkg_metadata::Pallet<T>>(
 					&prop_batch.signature,
 					&data,
 				);
 
-				// Accept all signatures to make testing easier
-				#[cfg(test)]
-				let result: Result<(), dkg_runtime_primitives::utils::SignatureError> = Ok(());
-
 				match result {
 					Ok(_) => {
 						// Do nothing, it is all good.