Rename AcknowledgementMsg::Err to ::Error

CosmWasm · Aug 23, 2023 · 793bc1c · 793bc1c
1 parent b959315
commit 793bc1c
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 11 deletions.
diff --git a/contracts/ibc-reflect-send/src/ibc.rs b/contracts/ibc-reflect-send/src/ibc.rs
@@ -146,7 +146,7 @@ fn acknowledge_who_am_i(
     // ignore errors (but mention in log)
     let WhoAmIResponse { account } = match ack {
         AcknowledgementMsg::Ok(res) => res,
-        AcknowledgementMsg::Err(e) => {
+        AcknowledgementMsg::Error(e) => {
             return Ok(IbcBasicResponse::new()
                 .add_attribute("action", "acknowledge_who_am_i")
                 .add_attribute("error", e))
@@ -176,7 +176,7 @@ fn acknowledge_balances(
     // ignore errors (but mention in log)
     let BalancesResponse { account, balances } = match ack {
         AcknowledgementMsg::Ok(res) => res,
-        AcknowledgementMsg::Err(e) => {
+        AcknowledgementMsg::Error(e) => {
             return Ok(IbcBasicResponse::new()
                 .add_attribute("action", "acknowledge_balances")
                 .add_attribute("error", e))

diff --git a/contracts/ibc-reflect-send/src/ibc_msg.rs b/contracts/ibc-reflect-send/src/ibc_msg.rs
@@ -23,22 +23,21 @@ pub enum PacketMsg {
 #[cw_serde]
 pub enum AcknowledgementMsg<S> {
     Ok(S),
-    #[serde(rename = "error")]
-    Err(String),
+    Error(String),
 }
 
 impl<S> AcknowledgementMsg<S> {
     pub fn unwrap(self) -> S {
         match self {
             AcknowledgementMsg::Ok(data) => data,
-            AcknowledgementMsg::Err(err) => panic!("{}", err),
+            AcknowledgementMsg::Error(err) => panic!("{}", err),
         }
     }
 
     pub fn unwrap_err(self) -> String {
         match self {
             AcknowledgementMsg::Ok(_) => panic!("not an error"),
-            AcknowledgementMsg::Err(err) => err,
+            AcknowledgementMsg::Error(err) => err,
         }
     }
 }

diff --git a/contracts/ibc-reflect/src/contract.rs b/contracts/ibc-reflect/src/contract.rs
@@ -224,7 +224,7 @@ pub fn migrate(_deps: DepsMut, _env: Env, _msg: Empty) -> StdResult<Response> {
 // this encode an error or error message into a proper acknowledgement to the recevier
 fn encode_ibc_error(msg: impl Into<String>) -> Binary {
     // this cannot error, unwrap to keep the interface simple
-    to_binary(&AcknowledgementMsg::<()>::Err(msg.into())).unwrap()
+    to_binary(&AcknowledgementMsg::<()>::Error(msg.into())).unwrap()
 }
 
 #[entry_point]

diff --git a/contracts/ibc-reflect/src/msg.rs b/contracts/ibc-reflect/src/msg.rs
@@ -62,22 +62,21 @@ pub enum PacketMsg {
 #[cw_serde]
 pub enum AcknowledgementMsg<S> {
     Ok(S),
-    #[serde(rename = "error")]
-    Err(String),
+    Error(String),
 }
 
 impl<S> AcknowledgementMsg<S> {
     pub fn unwrap(self) -> S {
         match self {
             AcknowledgementMsg::Ok(data) => data,
-            AcknowledgementMsg::Err(err) => panic!("{}", err),
+            AcknowledgementMsg::Error(err) => panic!("{}", err),
         }
     }
 
     pub fn unwrap_err(self) -> String {
         match self {
             AcknowledgementMsg::Ok(_) => panic!("not an error"),
-            AcknowledgementMsg::Err(err) => err,
+            AcknowledgementMsg::Error(err) => err,
         }
     }
 }