Fix disruptions

- Better ? vs & detection for queries
- Allow service_time to deser to None for an empty string.
- Properly use QUIET env

src/helpers.rs

@@ -58,11 +58,17 @@ where
 {
     let s: Option<String> = Option::deserialize(deserializer)?;
     match s {
-        Some(s) => Ok(Some(
-            NaiveDateTime::parse_from_str(&s, "%H:%M:%S").map_err(|e| {
-                serde::de::Error::custom(format!("Error deser service_time '{s}': {e:?}"))
-            })?,
-        )),
+        Some(s) => {
+            if s.is_empty() {
+                Ok(None)
+            } else {
+                Ok(Some(
+                    NaiveDateTime::parse_from_str(&s, "%H:%M:%S").map_err(|e| {
+                        serde::de::Error::custom(format!("Error deser service_time '{s}': {e:?}"))
+                    })?,
+                ))
+            }
+        }
         None => Ok(None),
     }
 }

src/lib.rs

@@ -26,10 +26,14 @@ impl Client {
         let path = format!(
             "/{path}{}devid={}",
             {
-                if path.ends_with('?') {
-                    ""
+                if !path.contains('?') {
+                    "?"
                 } else {
-                    "&"
+                    if path.ends_with('?') {
+                        "" 
+                    } else {
+                        "&"
+                    }
                 }
             },
             self.devid

src/ty.rs

@@ -531,6 +531,8 @@ pub struct DisruptionDirection {
     #[serde(rename = "direction_name")]
     pub name: String,
     /// Time of service to which disruption applies. Returns None if disruption applies to multiple, or no services
+    ///
+    /// This doesn't use null, it uses a blank string. I hate it here.
     #[serde(deserialize_with = "de_service_time")]
     pub service_time: Option<NaiveDateTime>,
 }

tests/main.rs

@@ -109,7 +109,7 @@ pub mod test {
                     "passed".green(),
                     elapsed,
                     {
-                        if std::env::var("quiet").is_ok() {
+                        if std::env::var("QUIET").is_err() {
                             format!("\n{}", res.cyan())
                         } else {
                             " ...".cyan().to_string()