From 1e81ef7ad9a6fe2eb2b8be2889020771b8906e4d Mon Sep 17 00:00:00 2001
From: Jun Kimura <jun.kimura@datachain.jp>
Date: Sun, 10 Nov 2024 22:52:45 +0900
Subject: [PATCH] clarify not supported functions

Signed-off-by: Jun Kimura <jun.kimura@datachain.jp>
---
 crates/ibc/src/client_state.rs | 70 +++++++++++++++++-----------------
 1 file changed, 36 insertions(+), 34 deletions(-)

diff --git a/crates/ibc/src/client_state.rs b/crates/ibc/src/client_state.rs
index 79370a0..156c7a6 100644
--- a/crates/ibc/src/client_state.rs
+++ b/crates/ibc/src/client_state.rs
@@ -279,10 +279,6 @@ impl<const SYNC_COMMITTEE_SIZE: usize> ClientState<SYNC_COMMITTEE_SIZE> {
 }
 
 impl<const SYNC_COMMITTEE_SIZE: usize> Ics2ClientState for ClientState<SYNC_COMMITTEE_SIZE> {
-    fn chain_id(&self) -> ChainId {
-        todo!()
-    }
-
     fn client_type(&self) -> ClientType {
         eth_client_type()
     }
@@ -295,15 +291,6 @@ impl<const SYNC_COMMITTEE_SIZE: usize> Ics2ClientState for ClientState<SYNC_COMM
         self.frozen_height
     }
 
-    #[allow(unused_variables)]
-    fn expired(&self, elapsed: Duration) -> bool {
-        todo!()
-    }
-
-    fn zero_custom_fields(&mut self) {
-        todo!()
-    }
-
     fn initialise(
         &self,
         consensus_state: Any,
@@ -446,27 +433,6 @@ impl<const SYNC_COMMITTEE_SIZE: usize> Ics2ClientState for ClientState<SYNC_COMM
             .into_box())
     }
 
-    #[allow(unused_variables)]
-    fn verify_upgrade_client(
-        &self,
-        upgraded_client_state: Any,
-        upgraded_consensus_state: Any,
-        proof_upgrade_client: ibc_proto::ibc::core::commitment::v1::MerkleProof,
-        proof_upgrade_consensus_state: ibc_proto::ibc::core::commitment::v1::MerkleProof,
-        root: &ibc::core::ics23_commitment::commitment::CommitmentRoot,
-    ) -> Result<(), ClientError> {
-        todo!()
-    }
-
-    #[allow(unused_variables)]
-    fn update_state_with_upgrade_client(
-        &self,
-        upgraded_client_state: Any,
-        upgraded_consensus_state: Any,
-    ) -> Result<UpdatedState, ClientError> {
-        todo!()
-    }
-
     fn verify_client_consensus_state(
         &self,
         proof_height: ibc::Height,
@@ -653,6 +619,42 @@ impl<const SYNC_COMMITTEE_SIZE: usize> Ics2ClientState for ClientState<SYNC_COMM
             receipt_path.clone(),
         )
     }
+
+    // `chain_id`, `expired`, `zero_custom_fields`, `verify_upgrade_client`, `update_state_with_upgrade_client` are not supported for Ethereum client
+
+    fn chain_id(&self) -> ChainId {
+        unimplemented!()
+    }
+
+    #[allow(unused_variables)]
+    fn expired(&self, elapsed: Duration) -> bool {
+        unimplemented!()
+    }
+
+    fn zero_custom_fields(&mut self) {
+        unimplemented!()
+    }
+
+    #[allow(unused_variables)]
+    fn verify_upgrade_client(
+        &self,
+        upgraded_client_state: Any,
+        upgraded_consensus_state: Any,
+        proof_upgrade_client: ibc_proto::ibc::core::commitment::v1::MerkleProof,
+        proof_upgrade_consensus_state: ibc_proto::ibc::core::commitment::v1::MerkleProof,
+        root: &ibc::core::ics23_commitment::commitment::CommitmentRoot,
+    ) -> Result<(), ClientError> {
+        unimplemented!()
+    }
+
+    #[allow(unused_variables)]
+    fn update_state_with_upgrade_client(
+        &self,
+        upgraded_client_state: Any,
+        upgraded_consensus_state: Any,
+    ) -> Result<UpdatedState, ClientError> {
+        unimplemented!()
+    }
 }
 
 fn validate_state_timestamp_within_trusting_period(