fix: support chain identifiers without {chain_name}-{revision_number} pattern

.changelog/unreleased/bug-fixes/940-optional-revision-number-in-chain-id.md

@@ -0,0 +1,2 @@
+- Support chain identifiers without the `{chain_name}-{revision_number}` pattern
+  of Tendermint chains. ([\#940](https://github.com/cosmos/ibc-rs/issues/940)).

crates/ibc/src/clients/ics07_tendermint/client_state.rs

@@ -827,7 +827,7 @@ mod tests {
     fn client_state_new() {
         // Define a "default" set of parameters to reuse throughout these tests.
         let default_params: ClientStateParams = ClientStateParams {
-            id: ChainId::new("ibc", 0).unwrap(),
+            id: ChainId::new("ibc-0").unwrap(),
             trust_level: TrustThreshold::ONE_THIRD,
             trusting_period: Duration::new(64000, 0),
             unbonding_period: Duration::new(128000, 0),
@@ -870,17 +870,17 @@ mod tests {
                 want_pass: true,
             },
             Test {
-                name: "Valid long (50 chars) chain-id that satisfies revision_number length < `u16::MAX` length".to_string(),
+                name: "Valid long (50 chars) chain-id that satisfies revision_number length < `u64::MAX` length".to_string(),
                 params: ClientStateParams {
-                    id: ChainId::new(&"a".repeat(29), 0).unwrap(),
+                    id: ChainId::new(&format!("{}-{}", "a".repeat(29), 0)).unwrap(),
                     ..default_params.clone()
                 },
                 want_pass: true,
             },
             Test {
                 name: "Invalid too-long (51 chars) chain-id".to_string(),
                 params: ClientStateParams {
-                    id: ChainId::new(&"a".repeat(30), 0).unwrap(),
+                    id: ChainId::new(&format!("{}-{}", "a".repeat(30), 0)).unwrap(),
                     ..default_params.clone()
                 },
                 want_pass: false,
@@ -993,7 +993,7 @@ mod tests {
     fn client_state_verify_height() {
         // Define a "default" set of parameters to reuse throughout these tests.
         let default_params: ClientStateParams = ClientStateParams {
-            id: ChainId::new("ibc", 1).unwrap(),
+            id: ChainId::new("ibc-1").unwrap(),
             trust_level: TrustThreshold::ONE_THIRD,
             trusting_period: Duration::new(64000, 0),
             unbonding_period: Duration::new(128000, 0),
@@ -1173,7 +1173,7 @@ pub mod test_util {
     pub fn get_dummy_raw_tm_client_state(frozen_height: RawHeight) -> RawTmClientState {
         #[allow(deprecated)]
         RawTmClientState {
-            chain_id: ChainId::new("ibc", 0).expect("Never fails").to_string(),
+            chain_id: ChainId::new("ibc-0").expect("Never fails").to_string(),
             trust_level: Some(Fraction {
                 numerator: 1,
                 denominator: 3,

crates/ibc/src/core/ics02_client/handler/update_client.rs

@@ -196,7 +196,7 @@ mod tests {
     /// contained in the store and has not expired.
     #[test]
     fn test_consensus_state_pruning() {
-        let chain_id = ChainId::new("mockgaiaA", 1).unwrap();
+        let chain_id = ChainId::new("mockgaiaA-1").unwrap();
 
         let client_height = Height::new(1, 1).unwrap();
 
@@ -295,10 +295,10 @@ mod tests {
         let client_id = ClientId::new(tm_client_type(), 0).unwrap();
         let client_height = Height::new(1, 20).unwrap();
         let update_height = Height::new(1, 21).unwrap();
-        let chain_id_b = ChainId::new("mockgaiaB", 1).unwrap();
+        let chain_id_b = ChainId::new("mockgaiaB-1").unwrap();
 
         let mut ctx = MockContext::new(
-            ChainId::new("mockgaiaA", 1).unwrap(),
+            ChainId::new("mockgaiaA-1").unwrap(),
             HostType::Mock,
             5,
             Height::new(1, 1).unwrap(),
@@ -343,10 +343,10 @@ mod tests {
     fn test_update_synthetic_tendermint_client_validator_change_ok() {
         let client_id = ClientId::new(tm_client_type(), 0).unwrap();
         let client_height = Height::new(1, 20).unwrap();
-        let chain_id_b = ChainId::new("mockgaiaB", 1).unwrap();
+        let chain_id_b = ChainId::new("mockgaiaB-1").unwrap();
 
         let mut ctx_a = MockContextConfig::builder()
-            .host_id(ChainId::new("mockgaiaA", 1).unwrap())
+            .host_id(ChainId::new("mockgaiaA-1").unwrap())
             .latest_height(Height::new(1, 1).unwrap())
             .build()
             .with_client_config(
@@ -435,10 +435,10 @@ mod tests {
     fn test_update_synthetic_tendermint_client_validator_change_fail() {
         let client_id = ClientId::new(tm_client_type(), 0).unwrap();
         let client_height = Height::new(1, 20).unwrap();
-        let chain_id_b = ChainId::new("mockgaiaB", 1).unwrap();
+        let chain_id_b = ChainId::new("mockgaiaB-1").unwrap();
 
         let ctx_a = MockContextConfig::builder()
-            .host_id(ChainId::new("mockgaiaA", 1).unwrap())
+            .host_id(ChainId::new("mockgaiaA-1").unwrap())
             .latest_height(Height::new(1, 1).unwrap())
             .build()
             .with_client_config(
@@ -515,10 +515,10 @@ mod tests {
         let client_id = ClientId::new(tm_client_type(), 0).unwrap();
         let client_height = Height::new(1, 20).unwrap();
         let update_height = Height::new(1, 21).unwrap();
-        let chain_id_b = ChainId::new("mockgaiaB", 1).unwrap();
+        let chain_id_b = ChainId::new("mockgaiaB-1").unwrap();
 
         let mut ctx = MockContext::new(
-            ChainId::new("mockgaiaA", 1).unwrap(),
+            ChainId::new("mockgaiaA-1").unwrap(),
             HostType::Mock,
             5,
             Height::new(1, 1).unwrap(),
@@ -565,8 +565,8 @@ mod tests {
         let client_id = ClientId::new(tm_client_type(), 0).unwrap();
         let client_height = Height::new(1, 20).unwrap();
 
-        let ctx_a_chain_id = ChainId::new("mockgaiaA", 1).unwrap();
-        let ctx_b_chain_id = ChainId::new("mockgaiaB", 1).unwrap();
+        let ctx_a_chain_id = ChainId::new("mockgaiaA-1").unwrap();
+        let ctx_b_chain_id = ChainId::new("mockgaiaB-1").unwrap();
         let start_height = Height::new(1, 11).unwrap();
 
         let mut ctx_a = MockContext::new(ctx_a_chain_id, HostType::Mock, 5, start_height)
@@ -695,7 +695,7 @@ mod tests {
         let chain_start_height = Height::new(1, 11).unwrap();
 
         let ctx = MockContext::new(
-            ChainId::new("mockgaiaA", 1).unwrap(),
+            ChainId::new("mockgaiaA-1").unwrap(),
             HostType::Mock,
             5,
             chain_start_height,
@@ -708,7 +708,7 @@ mod tests {
         );
 
         let ctx_b = MockContext::new(
-            ChainId::new("mockgaiaB", 1).unwrap(),
+            ChainId::new("mockgaiaB-1").unwrap(),
             HostType::SyntheticTendermint,
             5,
             client_height,
@@ -835,11 +835,11 @@ mod tests {
         let client_id = ClientId::new(tm_client_type(), 0).unwrap();
         let client_height = Height::new(1, 20).unwrap();
         let misbehaviour_height = Height::new(1, 21).unwrap();
-        let chain_id_b = ChainId::new("mockgaiaB", 1).unwrap();
+        let chain_id_b = ChainId::new("mockgaiaB-1").unwrap();
 
         // Create a mock context for chain-A with a synthetic tendermint light client for chain-B
         let mut ctx_a = MockContext::new(
-            ChainId::new("mockgaiaA", 1).unwrap(),
+            ChainId::new("mockgaiaA-1").unwrap(),
             HostType::Mock,
             5,
             Height::new(1, 1).unwrap(),
@@ -896,11 +896,11 @@ mod tests {
         let client_id = ClientId::new(tm_client_type(), 0).unwrap();
         let client_height = Height::new(1, 20).unwrap();
         let misbehaviour_height = Height::new(1, 21).unwrap();
-        let chain_id_b = ChainId::new("mockgaiaB", 1).unwrap();
+        let chain_id_b = ChainId::new("mockgaiaB-1").unwrap();
 
         // Create a mock context for chain-A with a synthetic tendermint light client for chain-B
         let mut ctx_a = MockContext::new(
-            ChainId::new("mockgaiaA", 1).unwrap(),
+            ChainId::new("mockgaiaA-1").unwrap(),
             HostType::Mock,
             5,
             Height::new(1, 1).unwrap(),
@@ -955,7 +955,7 @@ mod tests {
 
     #[test]
     fn test_expired_client() {
-        let chain_id_b = ChainId::new("mockgaiaB", 1).unwrap();
+        let chain_id_b = ChainId::new("mockgaiaB-1").unwrap();
 
         let update_height = Height::new(1, 21).unwrap();
         let client_height = update_height.sub(3).unwrap();
@@ -967,7 +967,7 @@ mod tests {
         let trusting_period = Duration::from_secs(64);
 
         let mut ctx = MockContextConfig::builder()
-            .host_id(ChainId::new("mockgaiaA", 1).unwrap())
+            .host_id(ChainId::new("mockgaiaA-1").unwrap())
             .latest_height(Height::new(1, 1).unwrap())
             .latest_timestamp(timestamp)
             .build()
@@ -995,7 +995,7 @@ mod tests {
 
     #[test]
     fn test_client_update_max_clock_drift() {
-        let chain_id_b = ChainId::new("mockgaiaB", 1).unwrap();
+        let chain_id_b = ChainId::new("mockgaiaB-1").unwrap();
 
         let client_height = Height::new(1, 20).unwrap();
 
@@ -1006,7 +1006,7 @@ mod tests {
         let max_clock_drift = Duration::from_secs(64);
 
         let ctx_a = MockContextConfig::builder()
-            .host_id(ChainId::new("mockgaiaA", 1).unwrap())
+            .host_id(ChainId::new("mockgaiaA-1").unwrap())
             .latest_height(Height::new(1, 1).unwrap())
             .latest_timestamp(timestamp)
             .build()

crates/ibc/src/core/ics03_connection/handler/conn_open_ack.rs

@@ -261,7 +261,7 @@ mod tests {
 
         let ctx_default = MockContext::default();
         let ctx_new = MockContext::new(
-            ChainId::new("mockgaia", latest_height.revision_number()).unwrap(),
+            ChainId::new(&format!("mockgaia-{}", latest_height.revision_number())).unwrap(),
             HostType::Mock,
             max_history_size,
             latest_height,

crates/ibc/src/core/ics03_connection/handler/conn_open_try.rs

@@ -257,7 +257,7 @@ mod tests {
         };
 
         let ctx_new = MockContext::new(
-            ChainId::new("mockgaia", 0).unwrap(),
+            ChainId::new("mockgaia-0").unwrap(),
             HostType::Mock,
             max_history_size,
             host_chain_height,