fix(client): drop unused import/ctor, needless ? (clippy -D warnings)

src/client/http_client.rs

@@ -57,7 +57,7 @@ use reqwest::ClientBuilder;
 use reqwest_middleware::ClientWithMiddleware;
 use reqwest_retry::policies::ExponentialBackoff;
 use reqwest_retry::RetryTransientMiddleware;
-use reqwest_retry::{DefaultRetryableStrategy, Jitter, RetryDecision, RetryPolicy};
+use reqwest_retry::{Jitter, RetryDecision, RetryPolicy};
 use thiserror::Error;
 
 use super::retry_after::RetryAfterMiddleware;
@@ -345,7 +345,7 @@ async fn build_client(
         }
         None => None,
     };
-    Ok(build_client_with_pems(config, ca_pem, client_pems)?)
+    build_client_with_pems(config, ca_pem, client_pems)
 }
 
 fn build_client_sync(

src/client/retry_after.rs

@@ -80,12 +80,6 @@ impl RetryAfterMiddleware {
         }
     }
 
-    #[cfg(test)]
-    fn with_parts(capacity: usize, ceiling: Duration, now: fn() -> SystemTime) -> Self {
-        let _ = now;
-        Self::with_capacity_and_ceiling(capacity, ceiling)
-    }
-
     fn record(&self, url: Url, deadline: SystemTime) {
         let mut deadlines = self.deadlines.lock().unwrap_or_else(|e| e.into_inner());
         if !deadlines.contains_key(&url) && deadlines.len() >= self.capacity {