test(error): expect api error (#222)

* test(error): expect api error

* test(error): allow dead_code and track_caller

* build: allow dead code for tests

src/client_reqwest.rs

@@ -186,14 +186,10 @@ mod async_tests {
             .create_async()
             .await;
         let api = AsyncApi::new_url(server.url());
-
-        if let Err(Error::Api(error)) = dbg!(api.send_message(&params).await) {
-            assert_eq!(error.description, "Bad Request: chat not found");
-            assert_eq!(error.error_code, 400);
-            assert_eq!(error.parameters, None);
-            assert!(!error.ok);
-        } else {
-            panic!("API Error expected");
-        }
+        let error = api.send_message(&params).await.unwrap_err().unwrap_api();
+        assert_eq!(error.description, "Bad Request: chat not found");
+        assert_eq!(error.error_code, 400);
+        assert_eq!(error.parameters, None);
+        assert!(!error.ok);
     }
 }

src/client_ureq.rs

@@ -238,15 +238,11 @@ mod tests {
             .with_body(response_string)
             .create();
         let api = Api::new_url(server.url());
-
-        if let Err(Error::Api(error)) = dbg!(api.send_message(&params)) {
-            assert_eq!(error.description, "Bad Request: chat not found");
-            assert_eq!(error.error_code, 400);
-            assert_eq!(error.parameters, None);
-            assert!(!error.ok);
-        } else {
-            panic!("API Error expected");
-        }
+        let error = api.send_message(&params).unwrap_err().unwrap_api();
+        assert_eq!(error.description, "Bad Request: chat not found");
+        assert_eq!(error.error_code, 400);
+        assert_eq!(error.parameters, None);
+        assert!(!error.ok);
     }
 
     #[test]
@@ -350,7 +346,7 @@ mod tests {
             .create();
         let api = Api::new_url(server.url());
 
-        let response = api.close().err();
+        let response = api.close().unwrap_err();
 
         let json = serde_json::to_string(&response).unwrap();
         assert_eq!(response_string, json);

src/error.rs

@@ -15,3 +15,15 @@ pub enum Error {
     #[error("Encode Error {0}")]
     Encode(String),
 }
+
+impl Error {
+    #[cfg(test)]
+    #[track_caller]
+    pub(crate) fn unwrap_api(self) -> ErrorResponse {
+        if let Self::Api(api) = self {
+            api
+        } else {
+            panic!("API Error expected: {self}");
+        }
+    }
+}

src/lib.rs

@@ -1,4 +1,5 @@
 #![cfg_attr(docsrs, feature(doc_auto_cfg))]
+#![cfg_attr(test, allow(dead_code))]
 
 #[cfg(feature = "async-http-client")]
 pub use reqwest;