Default trusting period to 2/3 of unbonding period if it is missing f…

…rom client settings

crates/cosmos/cosmos-chain-components/src/impls/client/create_client_payload.rs

@@ -57,12 +57,14 @@ where
 
         let unbonding_period = chain.query_unbonding_period().await?;
 
-        // Should we use a value for `trusting_period` in the config if it is not
+        // TODO: Should we use a value for `trusting_period` in the config if it is not
         // found in the client settings?
         // And if both are missing, should we default to another value?
+        // Current behaviour is to default to 2/3 of unbonding period if the
+        // trusting period is missing from the client settings.
         let trusting_period = create_client_options
             .trusting_period
-            .ok_or_else(|| Chain::raise_error("missing trusting period in client settings"))?;
+            .unwrap_or(2 * unbonding_period / 3);
 
         #[allow(deprecated)]
         let client_state = ClientState::new(

crates/cosmos/cosmos-chain-components/src/types/payloads/client.rs

@@ -1,12 +1,6 @@
 use cgp::prelude::*;
-use core::time::Duration;
 
-use ibc_proto::ibc::lightclients::tendermint::v1::Fraction;
-
-use crate::types::tendermint::{
-    ProtoTendermintClientState, ProtoTendermintConsensusState, TendermintClientState,
-    TendermintConsensusState, TendermintHeader,
-};
+use crate::types::tendermint::{TendermintClientState, TendermintConsensusState, TendermintHeader};
 
 #[derive(Debug, Clone)]
 pub struct CosmosUpdateClientPayload {
@@ -18,15 +12,3 @@ pub struct CosmosCreateClientPayload {
     pub client_state: TendermintClientState,
     pub consensus_state: TendermintConsensusState,
 }
-
-pub struct CosmosCreateClientOptions {
-    pub max_clock_drift: Duration,
-    pub trusting_period: Option<Duration>,
-    pub trust_threshold: Fraction,
-}
-
-#[derive(Debug, Clone, HasField)]
-pub struct CosmosCreateClientPayloadV2 {
-    pub client_state: ProtoTendermintClientState,
-    pub consensus_state: ProtoTendermintConsensusState,
-}