Change to return Err when an awaited response is an error response

Signed-off-by: trivernis <trivernis@protonmail.com>

Cargo.lock

@@ -500,7 +500,7 @@ dependencies = [
 
 [[package]]
 name = "rmp-ipc"
-version = "0.7.2"
+version = "0.8.0"
 dependencies = [
  "criterion",
  "lazy_static",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "rmp-ipc"
-version = "0.7.2"
+version = "0.8.0"
 authors = ["trivernis <trivernis@protonmail.com>"]
 edition = "2018"
 readme = "README.md"

src/error.rs

@@ -1,3 +1,4 @@
+use crate::error_event::ErrorEventData;
 use thiserror::Error;
 use tokio::sync::oneshot;
 
@@ -25,6 +26,9 @@ pub enum Error {
 
     #[error("Send Error")]
     SendError,
+
+    #[error("Error response: {0}")]
+    ErrorEvent(#[from] ErrorEventData),
 }
 
 impl From<String> for Error {

src/events/error_event.rs

@@ -1,4 +1,6 @@
 use serde::{Deserialize, Serialize};
+use std::error::Error;
+use std::fmt::{Display, Formatter};
 
 pub static ERROR_EVENT_NAME: &str = "error";
 
@@ -11,3 +13,11 @@ pub struct ErrorEventData {
     pub code: u16,
     pub message: String,
 }
+
+impl Error for ErrorEventData {}
+
+impl Display for ErrorEventData {
+    fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {
+        write!(f, "IPC Code {}: '{}'", self.code, self.message)
+    }
+}

src/ipc/stream_emitter.rs

@@ -1,4 +1,5 @@
 use crate::error::Result;
+use crate::error_event::{ErrorEventData, ERROR_EVENT_NAME};
 use crate::events::event::Event;
 use crate::events::payload::EventSendPayload;
 use crate::ipc::context::Context;
@@ -119,6 +120,10 @@ impl EmitMetadata {
     #[tracing::instrument(skip(self, ctx), fields(self.message_id))]
     pub async fn await_reply(&self, ctx: &Context) -> Result<Event> {
         let reply = ctx.await_reply(self.message_id).await?;
-        Ok(reply)
+        if reply.name() == ERROR_EVENT_NAME {
+            Err(reply.data::<ErrorEventData>()?.into())
+        } else {
+            Ok(reply)
+        }
     }
 }

src/tests/ipc_tests.rs

@@ -205,7 +205,6 @@ async fn test_error_responses() {
         .await
         .unwrap()
         .await_reply(&ctx)
-        .await
+        .await;
-        .unwrap();
+    assert!(reply.is_err());
-    assert_eq!(reply.name(), "error");
 }