Implement ConsensusState for AnyConsensusState (#1298)

* impl ConsensusState for AnyConsensusState * Add link to changelog entry Co-authored-by: Romain Ruetschi <romain.ruetschi@gmail.com>
informalsystems · Aug 18, 2021 · 1adea5a · 1adea5a
1 parent 882c9c0
commit 1adea5a
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/.changelog/unreleased/improvements/ibc/1297-impl-consensus-state.md b/.changelog/unreleased/improvements/ibc/1297-impl-consensus-state.md
@@ -0,0 +1,3 @@
+- Implement `ics02_client::client_consensus::ConsensusState` for `AnyConsensusState` ([#1297])
+
+[#1297]: https://github.com/informalsystems/ibc-rs/issues/1297
diff --git a/modules/src/ics02_client/client_consensus.rs b/modules/src/ics02_client/client_consensus.rs
@@ -161,11 +161,21 @@ impl ConsensusState for AnyConsensusState {
     }
 
     fn root(&self) -> &CommitmentRoot {
-        todo!()
+        match self {
+            Self::Tendermint(cs_state) => cs_state.root(),
+
+            #[cfg(any(test, feature = "mocks"))]
+            Self::Mock(mock_state) => mock_state.root(),
+        }
     }
 
     fn validate_basic(&self) -> Result<(), Infallible> {
-        todo!()
+        match self {
+            Self::Tendermint(cs_state) => cs_state.validate_basic(),
+
+            #[cfg(any(test, feature = "mocks"))]
+            Self::Mock(mock_state) => mock_state.validate_basic(),
+        }
     }
 
     fn wrap_any(self) -> AnyConsensusState {
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,3 @@
		- Implement `ics02_client::client_consensus::ConsensusState` for `AnyConsensusState` ([#1297])

		[#1297]: https://github.com/informalsystems/ibc-rs/issues/1297