Merge pull request #7 from cluebbehusen/cjl-validate-status

Validate status

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "oura-api"
-version = "0.1.0"
+version = "0.1.1"
 edition = "2021"
 license = "MIT"
 description = "A client for the Oura V2 REST API"

src/lib.rs

@@ -68,6 +68,7 @@ macro_rules! generic_endpoint {
                 .get(&url)
                 .bearer_auth(&self.token)
                 .send()?
+                .error_for_status()?
                 .json::<$type>()?;
             Ok(response)
         }
@@ -85,6 +86,7 @@ macro_rules! list_endpoint {
                 .bearer_auth(&self.token)
                 .query(&query)
                 .send()?
+                .error_for_status()?
                 .json::<ListResponse<$type>>()?;
             Ok(response)
         }
@@ -101,6 +103,7 @@ macro_rules! get_endpoint {
                 .get(&url)
                 .bearer_auth(&self.token)
                 .send()?
+                .error_for_status()?
                 .json::<$type>()?;
             Ok(response)
         }

tests/oura_api.rs

@@ -48,6 +48,28 @@ fn it_applies_query_to_list_requests() {
     assert!(result.is_ok());
 }
 
+#[test]
+fn it_returns_error_for_error_status() {
+    let mut server = Server::new();
+    let base_url = server.url();
+    let client = OuraClient::build_with_base_url("token", &base_url);
+
+    let mock = server
+        .mock("GET", "/daily_activity")
+        .match_header("Authorization", "Bearer token")
+        .with_status(400)
+        .with_header("content-type", "application/json")
+        .create();
+
+    let result = client.list_daily_activity(get_empty_date_query());
+
+    mock.assert();
+    assert_eq!(
+        result.unwrap_err().status(),
+        Some(reqwest::StatusCode::BAD_REQUEST)
+    );
+}
+
 macro_rules! test_endpoint {
     ($test_name: ident, $client_function: ident, $fixture_path: literal, $url_path: literal, $type: ty, $($get_arguments: ident)?) => {
         #[test]