Cargo.toml

@@ -16,7 +16,7 @@ reqwest = "0.11.22"
 tokio = { version = "1.32.0", features=["macros", "rt-multi-thread"] }
 futures = "0.3.28"
 serial_test = "2.0.0"
-num-traits = "0.2.17"
+num-traits = "0.2.0"
 num-derive = "0.4.0"
 
 #crypto

src/command/request.rs

@@ -2,9 +2,6 @@ use super::{body::Body, header::Header, Command, MessageType};
 use std::ops::Deref;
 
 pub trait RequestSerializer {
-    fn message_type(&self) -> MessageType;
-    fn body_type(&self) -> u8;
-
     fn serialize(&self) -> Vec<u8>;
 }
 
@@ -68,14 +65,6 @@ impl RequestSerializer for CommandRequest {
 
         stream
     }
-
-    fn message_type(&self) -> MessageType {
-        self.header().message_type()
-    }
-
-    fn body_type(&self) -> u8 {
-        self.body().body_type()
-    }
 }
 
 pub struct CommandQuerySubtype {
@@ -113,14 +102,6 @@ impl RequestSerializer for CommandQuerySubtype {
     fn serialize(&self) -> Vec<u8> {
         self.command.serialize()
     }
-
-    fn message_type(&self) -> MessageType {
-        self.header().message_type()
-    }
-
-    fn body_type(&self) -> u8 {
-        self.body().body_type()
-    }
 }
 
 pub struct CommandQueryCustom {
@@ -159,14 +140,6 @@ impl RequestSerializer for CommandQueryCustom {
     fn serialize(&self) -> Vec<u8> {
         self.command.serialize()
     }
-
-    fn message_type(&self) -> MessageType {
-        self.header().message_type()
-    }
-
-    fn body_type(&self) -> u8 {
-        self.body().body_type()
-    }
 }
 
 pub struct CommandHeartbeat;
@@ -175,12 +148,4 @@ impl RequestSerializer for CommandHeartbeat {
     fn serialize(&self) -> Vec<u8> {
         vec![0x00]
     }
-
-    fn message_type(&self) -> MessageType {
-        MessageType::None
-    }
-
-    fn body_type(&self) -> u8 {
-        0
-    }
 }

src/device.rs

@@ -124,6 +124,7 @@ impl Device {
         }
 
         for cmd in cmds {
+            'sending: loop {
                 self.build_send(cmd.clone())?;
 
                 loop {
@@ -138,11 +139,12 @@ impl Device {
                     }
 
                     match self.parse_message(&buf[..bytes_read])? {
-                    ParseMessage::Success => break,
+                        ParseMessage::Success => break 'sending,
                         ParseMessage::Padding => continue,
                     }
                 }
             }
+        }
 
         Ok(())
     }
@@ -224,11 +226,7 @@ impl Device {
     fn send_message(&self, msg: &[u8]) -> Result<()> {
         let data = self.security.encode_8370(msg, MsgType::ENCRYPTED_REQUEST)?;
 
-        self.socket
-            .lock()
-            .unwrap()
-            .write(&data)
-            .context(format!("{msg:?}"))?;
+        self.socket.lock().unwrap().write(&data)?;
 
         Ok(())
     }
@@ -239,26 +237,15 @@ impl Device {
     }
 
     fn build_send(&self, cmd: impl RequestSerializer) -> Result<()> {
-        let message_type = cmd.message_type();
-        let body_type = cmd.body_type();
-
         let data = PacketBuilder::builder(self.info.id, cmd).finalize(1);
 
-        Ok(self.send_message(&data).context(format!(
-            "message_type: {message_type:?}, body_type: {body_type}"
-        ))?)
+        self.send_message(&data)
     }
 }
 
 #[cfg(test)]
 mod test {
-    use std::{
-        net::{Ipv4Addr, SocketAddr},
-        sync::{
-            atomic::{AtomicBool, Ordering},
-            Arc,
-        },
-    };
+    use std::net::{Ipv4Addr, SocketAddr};
 
     use anyhow::{Context, Result};
     use futures::future::try_join;
@@ -355,29 +342,19 @@ mod test {
             addr: SocketAddr::new(std::net::IpAddr::V4(Ipv4Addr::new(192, 168, 178, 94)), 6444),
         };
 
-        let power: Arc<AtomicBool> = Arc::new(AtomicBool::new(false));
-
-        let device = Device::connect(device_info, token, key)
+        let mut device = Device::connect(device_info, token, key)
             .await?
-            .register_update({
-                let power = power.clone();
-
-                move |attributes| {
+            .register_update(|attributes| {
                 println!("{attributes:?}");
 
-                    power.store(attributes["power"].bool(), Ordering::SeqCst);
-
                 Ok(())
-                }
             });
 
         device
-            .set_attribute("power", AttributeValue::Bool(true))
+            .set_attribute("attribute", AttributeValue::Bool(false))
             .await?;
 
-        device
-            .set_attribute("power", AttributeValue::Bool(false))
-            .await?;
+        device.refresh_status().await?;
 
         Ok(())
     }

src/devices/e1.rs

@@ -127,7 +127,7 @@ impl DeviceBackend for E1 {
                 cmd.request()
             }
 
-            _ => todo!("attribute: {attribute}"),
+            _ => todo!(),
         }
     }