Remove height information from IbcEvent

.changelog/unreleased/improvements/2542-remove-ibcevent-height

@@ -0,0 +1 @@
+- Remove height attribute from `IbcEvent` and its variants ([#2542](https://github.com/informalsystems/ibc-rs/issues/2542))

modules/src/core/ics02_client/events.rs

@@ -11,9 +11,6 @@ use crate::core::ics24_host::identifier::ClientId;
 use crate::events::{IbcEvent, IbcEventType};
 use crate::prelude::*;
 
-/// The content of the `key` field for the attribute containing the height.
-pub const HEIGHT_ATTRIBUTE_KEY: &str = "height";
-
 /// The content of the `key` field for the attribute containing the client identifier.
 pub const CLIENT_ID_ATTRIBUTE_KEY: &str = "client_id";
 
@@ -53,7 +50,6 @@ impl From<NewBlock> for IbcEvent {
 
 #[derive(Debug, Deserialize, Serialize, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub struct Attributes {
-    pub height: Height,
     pub client_id: ClientId,
     pub client_type: ClientType,
     pub consensus_height: Height,
@@ -62,7 +58,6 @@ pub struct Attributes {
 impl Default for Attributes {
     fn default() -> Self {
         Attributes {
-            height: Height::new(0, 1).unwrap(),
             client_id: Default::default(),
             client_type: ClientType::Tendermint,
             consensus_height: Height::new(0, 1).unwrap(),
@@ -79,34 +74,26 @@ impl Default for Attributes {
 /// Once tendermint-rs improves the API of the `Key` and `Value` types,
 /// we will be able to remove the `.parse().unwrap()` calls.
 impl From<Attributes> for Vec<Tag> {
-    fn from(a: Attributes) -> Self {
-        let height = Tag {
-            key: HEIGHT_ATTRIBUTE_KEY.parse().unwrap(),
-            value: a.height.to_string().parse().unwrap(),
-        };
+    fn from(attrs: Attributes) -> Self {
         let client_id = Tag {
             key: CLIENT_ID_ATTRIBUTE_KEY.parse().unwrap(),
-            value: a.client_id.to_string().parse().unwrap(),
+            value: attrs.client_id.to_string().parse().unwrap(),
         };
         let client_type = Tag {
             key: CLIENT_TYPE_ATTRIBUTE_KEY.parse().unwrap(),
-            value: a.client_type.as_str().parse().unwrap(),
+            value: attrs.client_type.as_str().parse().unwrap(),
         };
         let consensus_height = Tag {
             key: CONSENSUS_HEIGHT_ATTRIBUTE_KEY.parse().unwrap(),
-            value: a.height.to_string().parse().unwrap(),
+            value: attrs.consensus_height.to_string().parse().unwrap(),
         };
-        vec![height, client_id, client_type, consensus_height]
+        vec![client_id, client_type, consensus_height]
     }
 }
 
 impl core::fmt::Display for Attributes {
     fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> Result<(), core::fmt::Error> {
-        write!(
-            f,
-            "h: {}, cs_h: {}({})",
-            self.height, self.client_id, self.consensus_height
-        )
+        write!(f, "cs_h: {}({})", self.client_id, self.consensus_height)
     }
 }
 
@@ -118,12 +105,6 @@ impl CreateClient {
     pub fn client_id(&self) -> &ClientId {
         &self.0.client_id
     }
-    pub fn height(&self) -> Height {
-        self.0.height
-    }
-    pub fn set_height(&mut self, height: Height) {
-        self.0.height = height;
-    }
 }
 
 impl From<Attributes> for CreateClient {
@@ -170,14 +151,6 @@ impl UpdateClient {
         self.common.client_type
     }
 
-    pub fn height(&self) -> Height {
-        self.common.height
-    }
-
-    pub fn set_height(&mut self, height: Height) {
-        self.common.height = height;
-    }
-
     pub fn consensus_height(&self) -> Height {
         self.common.consensus_height
     }
@@ -236,12 +209,6 @@ impl ClientMisbehaviour {
     pub fn client_id(&self) -> &ClientId {
         &self.0.client_id
     }
-    pub fn height(&self) -> Height {
-        self.0.height
-    }
-    pub fn set_height(&mut self, height: Height) {
-        self.0.height = height;
-    }
 }
 
 impl From<Attributes> for ClientMisbehaviour {
@@ -271,12 +238,6 @@ impl From<ClientMisbehaviour> for AbciEvent {
 pub struct UpgradeClient(pub Attributes);
 
 impl UpgradeClient {
-    pub fn set_height(&mut self, height: Height) {
-        self.0.height = height;
-    }
-    pub fn height(&self) -> Height {
-        self.0.height
-    }
     pub fn client_id(&self) -> &ClientId {
         &self.0.client_id
     }

modules/src/core/ics02_client/handler/create_client.rs

@@ -56,7 +56,6 @@ pub fn process(
 
     let event_attributes = Attributes {
         client_id,
-        height: ctx.host_height(),
         ..Default::default()
     };
     output.emit(IbcEvent::CreateClient(event_attributes.into()));
@@ -78,7 +77,6 @@ mod tests {
     use crate::core::ics02_client::client_consensus::AnyConsensusState;
     use crate::core::ics02_client::client_state::ClientState;
     use crate::core::ics02_client::client_type::ClientType;
-    use crate::core::ics02_client::context::ClientReader;
     use crate::core::ics02_client::handler::{dispatch, ClientResult};
     use crate::core::ics02_client::msgs::create_client::MsgCreateAnyClient;
     use crate::core::ics02_client::msgs::ClientMsg;
@@ -118,7 +116,6 @@ mod tests {
                 assert!(
                     matches!(event, IbcEvent::CreateClient(ref e) if e.client_id() == &expected_client_id)
                 );
-                assert_eq!(event.height(), ctx.host_height());
                 match result {
                     ClientResult::Create(create_result) => {
                         assert_eq!(create_result.client_type, ClientType::Mock);
@@ -187,7 +184,6 @@ mod tests {
                     assert!(
                         matches!(event, IbcEvent::CreateClient(ref e) if e.client_id() == &expected_client_id)
                     );
-                    assert_eq!(event.height(), ctx.host_height());
                     match result {
                         ClientResult::Create(create_res) => {
                             assert_eq!(create_res.client_type, msg.client_state.client_type());
@@ -251,7 +247,6 @@ mod tests {
                 assert!(
                     matches!(event, IbcEvent::CreateClient(ref e) if e.client_id() == &expected_client_id)
                 );
-                assert_eq!(event.height(), ctx.host_height());
                 match result {
                     ClientResult::Create(create_res) => {
                         assert_eq!(create_res.client_type, ClientType::Tendermint);

modules/src/core/ics02_client/handler/update_client.rs

@@ -93,7 +93,6 @@ pub fn process(
 
     let event_attributes = Attributes {
         client_id,
-        height: ctx.host_height(),
         ..Default::default()
     };
     output.emit(IbcEvent::UpdateClient(event_attributes.into()));
@@ -109,7 +108,6 @@ mod tests {
     use crate::core::ics02_client::client_consensus::AnyConsensusState;
     use crate::core::ics02_client::client_state::{AnyClientState, ClientState};
     use crate::core::ics02_client::client_type::ClientType;
-    use crate::core::ics02_client::context::ClientReader;
     use crate::core::ics02_client::error::{Error, ErrorDetail};
     use crate::core::ics02_client::handler::dispatch;
     use crate::core::ics02_client::handler::ClientResult::Update;
@@ -157,7 +155,6 @@ mod tests {
                 assert!(
                     matches!(event, IbcEvent::UpdateClient(ref e) if e.client_id() == &msg.client_id)
                 );
-                assert_eq!(event.height(), ctx.host_height());
                 assert!(log.is_empty());
                 // Check the result
                 match result {
@@ -241,7 +238,6 @@ mod tests {
                     assert!(
                         matches!(event, IbcEvent::UpdateClient(ref e) if e.client_id() == &msg.client_id)
                     );
-                    assert_eq!(event.height(), ctx.host_height());
                     assert!(log.is_empty());
                 }
                 Err(err) => {
@@ -309,7 +305,6 @@ mod tests {
                 assert!(
                     matches!(event, IbcEvent::UpdateClient(ref e) if e.client_id() == &msg.client_id)
                 );
-                assert_eq!(event.height(), ctx.host_height());
                 assert!(log.is_empty());
                 // Check the result
                 match result {
@@ -387,7 +382,6 @@ mod tests {
                 assert!(
                     matches!(event, IbcEvent::UpdateClient(ref e) if e.client_id() == &msg.client_id)
                 );
-                assert_eq!(event.height(), ctx.host_height());
                 assert!(log.is_empty());
                 // Check the result
                 match result {
@@ -466,7 +460,6 @@ mod tests {
                 assert!(
                     matches!(event, IbcEvent::UpdateClient(ref e) if e.client_id() == &msg.client_id)
                 );
-                assert_eq!(event.height(), ctx.host_height());
                 assert!(log.is_empty());
                 // Check the result
                 match result {

modules/src/core/ics02_client/handler/upgrade_client.rs

@@ -66,7 +66,6 @@ pub fn process(
     });
     let event_attributes = Attributes {
         client_id,
-        height: ctx.host_height(),
         ..Default::default()
     };
 
@@ -80,7 +79,6 @@ mod tests {
 
     use core::str::FromStr;
 
-    use crate::core::ics02_client::context::ClientReader;
     use crate::core::ics02_client::error::{Error, ErrorDetail};
     use crate::core::ics02_client::handler::dispatch;
     use crate::core::ics02_client::handler::ClientResult::Upgrade;
@@ -125,7 +123,6 @@ mod tests {
                 assert!(
                     matches!(event, IbcEvent::UpgradeClient(ref e) if e.client_id() == &msg.client_id)
                 );
-                assert_eq!(event.height(), ctx.host_height());
                 assert!(log.is_empty());
                 // Check the result
                 match result {

modules/src/core/ics03_connection/events.rs

@@ -4,39 +4,24 @@ use serde_derive::{Deserialize, Serialize};
 use tendermint::abci::tag::Tag;
 use tendermint::abci::Event as AbciEvent;
 
-use crate::core::ics02_client::height::Height;
 use crate::core::ics24_host::identifier::{ClientId, ConnectionId};
 use crate::events::{IbcEvent, IbcEventType};
 use crate::prelude::*;
 
 /// The content of the `key` field for the attribute containing the connection identifier.
-pub const HEIGHT_ATTRIBUTE_KEY: &str = "height";
 pub const CONN_ID_ATTRIBUTE_KEY: &str = "connection_id";
 pub const CLIENT_ID_ATTRIBUTE_KEY: &str = "client_id";
 pub const COUNTERPARTY_CONN_ID_ATTRIBUTE_KEY: &str = "counterparty_connection_id";
 pub const COUNTERPARTY_CLIENT_ID_ATTRIBUTE_KEY: &str = "counterparty_client_id";
 
-#[derive(Debug, Deserialize, Serialize, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
+#[derive(Debug, Default, Deserialize, Serialize, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub struct Attributes {
-    pub height: Height,
     pub connection_id: Option<ConnectionId>,
     pub client_id: ClientId,
     pub counterparty_connection_id: Option<ConnectionId>,
     pub counterparty_client_id: ClientId,
 }
 
-impl Default for Attributes {
-    fn default() -> Self {
-        Self {
-            height: Height::new(0, 1).unwrap(),
-            connection_id: Default::default(),
-            client_id: Default::default(),
-            counterparty_connection_id: Default::default(),
-            counterparty_client_id: Default::default(),
-        }
-    }
-}
-
 /// Convert attributes to Tendermint ABCI tags
 ///
 /// # Note
@@ -48,11 +33,6 @@ impl Default for Attributes {
 impl From<Attributes> for Vec<Tag> {
     fn from(a: Attributes) -> Self {
         let mut attributes = vec![];
-        let height = Tag {
-            key: HEIGHT_ATTRIBUTE_KEY.parse().unwrap(),
-            value: a.height.to_string().parse().unwrap(),
-        };
-        attributes.push(height);
         if let Some(conn_id) = a.connection_id {
             let conn_id = Tag {
                 key: CONN_ID_ATTRIBUTE_KEY.parse().unwrap(),
@@ -82,7 +62,7 @@ impl From<Attributes> for Vec<Tag> {
 }
 
 #[derive(Debug, Serialize, Clone, PartialEq, Eq)]
-pub struct OpenInit(Attributes);
+pub struct OpenInit(pub Attributes);
 
 impl OpenInit {
     pub fn attributes(&self) -> &Attributes {
@@ -91,12 +71,6 @@ impl OpenInit {
     pub fn connection_id(&self) -> Option<&ConnectionId> {
         self.0.connection_id.as_ref()
     }
-    pub fn height(&self) -> Height {
-        self.0.height
-    }
-    pub fn set_height(&mut self, height: Height) {
-        self.0.height = height;
-    }
 }
 
 impl From<Attributes> for OpenInit {
@@ -122,7 +96,7 @@ impl From<OpenInit> for AbciEvent {
 }
 
 #[derive(Debug, Serialize, Clone, PartialEq, Eq)]
-pub struct OpenTry(Attributes);
+pub struct OpenTry(pub Attributes);
 
 impl OpenTry {
     pub fn attributes(&self) -> &Attributes {
@@ -131,12 +105,6 @@ impl OpenTry {
     pub fn connection_id(&self) -> Option<&ConnectionId> {
         self.0.connection_id.as_ref()
     }
-    pub fn height(&self) -> Height {
-        self.0.height
-    }
-    pub fn set_height(&mut self, height: Height) {
-        self.0.height = height;
-    }
 }
 
 impl From<Attributes> for OpenTry {
@@ -162,7 +130,7 @@ impl From<OpenTry> for AbciEvent {
 }
 
 #[derive(Debug, Serialize, Clone, PartialEq, Eq)]
-pub struct OpenAck(Attributes);
+pub struct OpenAck(pub Attributes);
 
 impl OpenAck {
     pub fn attributes(&self) -> &Attributes {
@@ -171,12 +139,6 @@ impl OpenAck {
     pub fn connection_id(&self) -> Option<&ConnectionId> {
         self.0.connection_id.as_ref()
     }
-    pub fn height(&self) -> Height {
-        self.0.height
-    }
-    pub fn set_height(&mut self, height: Height) {
-        self.0.height = height;
-    }
 }
 
 impl From<Attributes> for OpenAck {
@@ -202,7 +164,7 @@ impl From<OpenAck> for AbciEvent {
 }
 
 #[derive(Debug, Serialize, Clone, PartialEq, Eq)]
-pub struct OpenConfirm(Attributes);
+pub struct OpenConfirm(pub Attributes);
 
 impl OpenConfirm {
     pub fn attributes(&self) -> &Attributes {
@@ -211,12 +173,6 @@ impl OpenConfirm {
     pub fn connection_id(&self) -> Option<&ConnectionId> {
         self.0.connection_id.as_ref()
     }
-    pub fn height(&self) -> Height {
-        self.0.height
-    }
-    pub fn set_height(&mut self, height: Height) {
-        self.0.height = height;
-    }
 }
 
 impl From<Attributes> for OpenConfirm {