use rstest cases

ibc-testkit/src/fixtures/clients/tendermint.rs

@@ -169,10 +169,24 @@ pub fn dummy_ics07_header() -> Header {
 mod tests {
 
     use ibc::primitives::proto::Any;
+    use rstest::rstest;
 
     use super::*;
 
-    fn try_tm_client_state_conversions(frozen_height: RawHeight) {
+    #[rstest]
+    // try conversions for when the client is not frozen
+    #[case::valid_client(0, 0)]
+    // try conversions for when the client is frozen
+    #[case::frozen_client(0, 1)]
+    fn tm_client_state_conversions_healthy(
+        #[case] revision_number: u64,
+        #[case] revision_height: u64,
+    ) {
+        let frozen_height = RawHeight {
+            revision_number,
+            revision_height,
+        };
+
         // check client state creation path from a proto type
         let tm_client_state_from_raw = dummy_tm_client_state_from_raw(frozen_height);
         assert!(tm_client_state_from_raw.is_ok());
@@ -191,21 +205,6 @@ mod tests {
         );
     }
 
-    #[test]
-    fn tm_client_state_conversions_healthy() {
-        // try conversions for when the client is not frozen
-        try_tm_client_state_conversions(RawHeight {
-            revision_number: 0,
-            revision_height: 0,
-        });
-
-        // try conversions for when the client is frozen
-        try_tm_client_state_conversions(RawHeight {
-            revision_number: 0,
-            revision_height: 1,
-        });
-    }
-
     #[test]
     fn tm_client_state_from_header_healthy() {
         // check client state creation path from a tendermint header