v0.1.5

Author: Mario Savarese

src/lib.rs

@@ -422,7 +422,6 @@ impl GuardianContentClient {
     ///         .send()
     ///         .await?;
     /// ```
-    ///
     #[allow(clippy::too_many_arguments)]
     pub fn datetime_to(
         &mut self,
@@ -436,8 +435,12 @@ impl GuardianContentClient {
     ) -> &mut GuardianContentClient {
         let formatted_datetime =
             crate::helpers::datetime(year, month, day, hour, min, sec, timezone);
-        self.request
-            .insert(String::from("to-date"), formatted_datetime);
+
+        if !formatted_datetime.is_empty() {
+            self.request
+                .insert(String::from("to-date"), formatted_datetime);
+        }
+
         self
     }
 
@@ -780,11 +783,13 @@ mod helpers {
 
         let offset =
             offset(timezone.abs() * 3600).unwrap_or_else(|| FixedOffset::west_opt(0).unwrap());
-        let LocalResult::Single(date) = offset.with_ymd_and_hms(year, month, day, hour, min, sec) else {
-             return "".to_owned()
-        };
 
-        date.to_rfc3339()
+        if let LocalResult::Single(date) = offset.with_ymd_and_hms(year, month, day, hour, min, sec)
+        {
+            date.to_rfc3339()
+        } else {
+            String::new()
+        }
     }
 
     pub(crate) fn mock_response() -> SearchResponse {

src/tests.rs

@@ -153,7 +153,26 @@ mod tests {
     }
 
     #[test]
-    fn test_datetime_to_2() {
+    fn test_datetime_from_wrong_offset() {
+        let mut client = setup();
+        // Invalid offset
+        client.datetime_from(2021, 12, 31, 0, 0, 0, 1024);
+        assert_eq!(
+            client.request.get("from-date").unwrap(),
+            "2021-12-31T00:00:00+00:00"
+        );
+    }
+
+    #[test]
+    fn test_datetime_from_wrong_ymd_hms() {
+        let mut client = setup();
+        // Invalid YMD
+        client.datetime_from(2021, 13, 40, 0, 999, 0, 5);
+        assert_eq!(client.request.get("from-date"), None);
+    }
+
+    #[test]
+    fn test_datetime_to_wrong_offset() {
         let mut client = setup();
         // Invalid offset
         client.datetime_to(2021, 12, 31, 0, 0, 0, 999);
@@ -164,11 +183,11 @@ mod tests {
     }
 
     #[test]
-    fn test_datetime_to_3() {
+    fn test_datetime_to_wrong_ymd_hms() {
         let mut client = setup();
         // Invalid YMD
-        client.datetime_to(2021, 13, 40, 0, 0, 0, 5);
-        assert_eq!(client.request.get("to-date").unwrap(), "");
+        client.datetime_to(2021, 13, 40, 0, 999, 0, 5);
+        assert_eq!(client.request.get("to-date"), None);
     }
 
     #[test]