Fix conversions related to MockHeader and ClientType::Mock

modules/src/ics02_client/client_type.rs

@@ -39,7 +39,7 @@ impl std::str::FromStr for ClientType {
             "07-tendermint" => Ok(Self::Tendermint),
 
             #[cfg(any(test, feature = "mocks"))]
-            "mock" => Ok(Self::Mock),
+            "9999-mock" => Ok(Self::Mock),
 
             _ => Err(error::Kind::UnknownClientType(s.to_string()).into()),
         }
@@ -65,7 +65,7 @@ mod tests {
 
     #[test]
     fn parse_mock_client_type() {
-        let client_type = ClientType::from_str("mock");
+        let client_type = ClientType::from_str("9999-mock");
 
         match client_type {
             Ok(ClientType::Mock) => (),
@@ -85,4 +85,20 @@ mod tests {
             _ => panic!("parse didn't fail"),
         }
     }
+
+    #[test]
+    fn parse_mock_as_string_result() {
+        let client_type = ClientType::Mock;
+        let type_string = client_type.as_string();
+        let client_type_from_str = ClientType::from_str(type_string).unwrap();
+        assert_eq!(client_type_from_str, client_type);
+    }
+
+    #[test]
+    fn parse_tendermint_as_string_result() {
+        let client_type = ClientType::Tendermint;
+        let type_string = client_type.as_string();
+        let client_type_from_str = ClientType::from_str(type_string).unwrap();
+        assert_eq!(client_type_from_str, client_type);
+    }
 }

modules/src/mock/header.rs

@@ -40,14 +40,18 @@ impl TryFrom<RawMockHeader> for MockHeader {
 
 impl From<MockHeader> for RawMockHeader {
     fn from(value: MockHeader) -> Self {
-        value.into()
+        RawMockHeader {
+            height: Some(value.height.into()),
+            timestamp: value.timestamp.as_nanoseconds(),
+        }
     }
 }
 
 impl MockHeader {
     pub fn height(&self) -> Height {
         self.height
     }
+
     pub fn new(height: Height) -> Self {
         Self {
             height,
@@ -68,11 +72,11 @@ impl Header for MockHeader {
     }
 
     fn height(&self) -> Height {
-        todo!()
+        self.height
     }
 
     fn wrap_any(self) -> AnyHeader {
-        todo!()
+        AnyHeader::Mock(self)
     }
 }
 
@@ -81,3 +85,22 @@ impl From<MockHeader> for AnyConsensusState {
         AnyConsensusState::Mock(MockConsensusState(h))
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn encode_any() {
+        let header = MockHeader::new(Height::new(1, 10));
+        let bytes = header.wrap_any().encode_vec().unwrap();
+
+        assert_eq!(
+            &bytes,
+            &[
+                10, 16, 47, 105, 98, 99, 46, 109, 111, 99, 107, 46, 72, 101, 97, 100, 101, 114, 18,
+                6, 10, 4, 8, 1, 16, 10,
+            ]
+        );
+    }
+}