refactor(ics24-host): make 07-tendermint-0 a constant in tests

Author: tuantran1702

ibc-core/ics24-host/types/src/path.rs

@@ -1169,7 +1169,7 @@ fn parse_upgrades(components: &[&str]) -> Option<Path> {
 #[cfg(test)]
 mod tests {
     use super::*;
-
+    const DUMMY_CLIENT_ID: &str = "07-tendermint-0";
     #[rstest::rstest]
     #[case(NEXT_CLIENT_SEQUENCE, Path::NextClientSequence(NextClientSequencePath))]
     #[case(
@@ -1182,28 +1182,27 @@ mod tests {
     )]
     #[case(
         "clients/07-tendermint-0/clientState",
-        Path::ClientState(ClientStatePath("07-tendermint-0".into()))
-    )]
+        Path::ClientState(ClientStatePath(DUMMY_CLIENT_ID.into())))]
     #[case(
         "clients/07-tendermint-0/consensusStates/15-31",
         Path::ClientConsensusState(ClientConsensusStatePath {
-            client_id: "07-tendermint-0".into(),
+            client_id: DUMMY_CLIENT_ID.into(),
             revision_number: 15,
             revision_height: 31,
         })
     )]
     #[case(
         "clients/07-tendermint-0/consensusStates/15-31/processedTime",
         Path::ClientUpdateTime(ClientUpdateTimePath {
-            client_id: "07-tendermint-0".into(),
+            client_id: DUMMY_CLIENT_ID.into(),
             revision_number: 15,
             revision_height: 31,
         })
     )]
     #[case(
         "clients/07-tendermint-0/consensusStates/15-31/processedHeight",
         Path::ClientUpdateHeight(ClientUpdateHeightPath {
-            client_id: "07-tendermint-0".into(),
+            client_id: DUMMY_CLIENT_ID.into(),
             revision_number: 15,
             revision_height: 31,
         })
@@ -1289,7 +1288,7 @@ mod tests {
         assert_eq!(
             parse_client_paths(&components),
             Some(Path::ClientState(ClientStatePath(ClientId::from(
-                "07-tendermint-0"
+                DUMMY_CLIENT_ID
             ))))
         );
 
@@ -1299,7 +1298,7 @@ mod tests {
         assert_eq!(
             parse_client_paths(&components),
             Some(Path::ClientConsensusState(ClientConsensusStatePath {
-                client_id: "07-tendermint-0".into(),
+                client_id: DUMMY_CLIENT_ID.into(),
                 revision_number: 15,
                 revision_height: 31,
             }))
@@ -1314,7 +1313,7 @@ mod tests {
         assert_eq!(
             parse_client_paths(&components),
             Some(Path::ClientUpdateTime(ClientUpdateTimePath {
-                client_id: "07-tendermint-0".into(),
+                client_id: DUMMY_CLIENT_ID.into(),
                 revision_number: 15,
                 revision_height: 31,
             }))
@@ -1326,7 +1325,7 @@ mod tests {
         assert_eq!(
             parse_client_paths(&components),
             Some(Path::ClientUpdateHeight(ClientUpdateHeightPath {
-                client_id: "07-tendermint-0".into(),
+                client_id: DUMMY_CLIENT_ID.into(),
                 revision_number: 15,
                 revision_height: 31,
             }))