challenge period

pallet-ismp/src/host.rs

@@ -15,10 +15,10 @@
 
 //! Host implementation for ISMP
 use crate::{
-    dispatcher::Receipt, primitives::ConsensusClientProvider, Config, ConsensusClientUpdateTime,
-    ConsensusStateClient, ConsensusStates, FrozenConsensusClients, FrozenHeights,
-    LatestStateMachineHeight, Nonce, RequestCommitments, RequestReceipts, ResponseReceipts,
-    StateCommitments, UnbondingPeriod,
+    dispatcher::Receipt, primitives::ConsensusClientProvider, ChallengePeriod, Config,
+    ConsensusClientUpdateTime, ConsensusStateClient, ConsensusStates, FrozenConsensusClients,
+    FrozenHeights, LatestStateMachineHeight, Nonce, RequestCommitments, RequestReceipts,
+    ResponseReceipts, StateCommitments, UnbondingPeriod,
 };
 use alloc::{format, string::ToString};
 use core::time::Duration;
@@ -162,8 +162,8 @@ where
         sp_io::hashing::keccak_256(bytes).into()
     }
 
-    fn challenge_period(&self, id: ConsensusClientId) -> Option<Duration> {
-        <T as Config>::ConsensusClientProvider::challenge_period(id)
+    fn challenge_period(&self, id: ConsensusStateId) -> Option<Duration> {
+        UnbondingPeriod::<T>::get(&id).map(Duration::from_secs)
     }
 
     fn ismp_router(&self) -> Box<dyn IsmpRouter> {
@@ -244,9 +244,10 @@ where
 
     fn store_challenge_period(
         &self,
-        _consensus_state_id: ConsensusStateId,
-        _period: u64,
+        consensus_state_id: ConsensusStateId,
+        period: u64,
     ) -> Result<(), Error> {
-        todo!()
+        ChallengePeriod::<T>::insert(consensus_state_id, period);
+        Ok(())
     }
 }

pallet-ismp/src/lib.rs

@@ -186,6 +186,11 @@ pub mod pallet {
     pub type UnbondingPeriod<T: Config> =
         StorageMap<_, Blake2_128Concat, ConsensusStateId, u64, OptionQuery>;
 
+    /// A mapping of ConsensusStateId to Challenge periods
+    #[pallet::storage]
+    pub type ChallengePeriod<T: Config> =
+        StorageMap<_, Blake2_128Concat, ConsensusStateId, u64, OptionQuery>;
+
     /// Holds a map of consensus clients frozen due to byzantine
     /// behaviour
     #[pallet::storage]

pallet-ismp/src/mock.rs

@@ -19,7 +19,11 @@ use crate::*;
 use crate::primitives::ConsensusClientProvider;
 use frame_support::traits::{ConstU32, ConstU64, Get};
 use frame_system::EnsureRoot;
-use ismp_rs::{consensus::ConsensusClient, module::IsmpModule, router::IsmpRouter};
+use ismp_rs::{
+    consensus::{ConsensusClient, ConsensusStateId},
+    module::IsmpModule,
+    router::IsmpRouter,
+};
 
 use crate::ismp_mocks::{MockConsensusClient, MockModule};
 use sp_core::H256;
@@ -60,7 +64,7 @@ impl ConsensusClientProvider for ConsensusProvider {
         Ok(Box::new(MockConsensusClient))
     }
 
-    fn challenge_period(_id: ConsensusClientId) -> Option<Duration> {
+    fn challenge_period(_id: ConsensusStateId) -> Option<Duration> {
         Some(Duration::from_secs(60 * 60))
     }
 }

pallet-ismp/src/primitives.rs

@@ -17,7 +17,7 @@
 use core::time::Duration;
 use frame_support::{PalletId, RuntimeDebug};
 use ismp_primitives::mmr::{LeafIndex, NodeIndex};
-use ismp_rs::consensus::{ConsensusClient, ConsensusClientId};
+use ismp_rs::consensus::{ConsensusClient, ConsensusClientId, ConsensusStateId};
 use scale_info::TypeInfo;
 use sp_core::{crypto::AccountId32, H160};
 use sp_std::prelude::*;
@@ -60,8 +60,8 @@ pub trait ConsensusClientProvider {
         id: ConsensusClientId,
     ) -> Result<Box<dyn ConsensusClient>, ismp_rs::error::Error>;
 
-    /// Returns the challenge period configured for a consensus client
-    fn challenge_period(id: ConsensusClientId) -> Option<Duration>;
+    /// Returns the challenge period configured for a consensus state
+    fn challenge_period(id: ConsensusStateId) -> Option<Duration>;
 }
 
 /// Module identification types supported by ismp