Improve ApiClient::new ergonomics

CHANGELOG.md

@@ -6,6 +6,11 @@ file. This change log follows the conventions of
 
 ## [Rust Unreleased][Unreleased]
 
+### Changed
+
+- The implementation of `ApiClient::new` has been improved to allow for the return of `ApiClient` instead of `Result<ApiClient, Error>`.
+- The default timeout for all requests has been reduced to 30 seconds from 300 seconds.
+
 ## [Python Unreleased][Unreleased]
 
 ## [Rust v0.7.9][v0.7.9] - 2024-01-27

tapo-py/src/api_client.rs

@@ -16,7 +16,7 @@ pub struct PyApiClient {
 impl PyApiClient {
     #[new]
     pub fn new(tapo_username: String, tapo_password: String) -> Result<Self, ErrorWrapper> {
-        let client = ApiClient::new(tapo_username, tapo_password)?;
+        let client = ApiClient::new(tapo_username, tapo_password);
         Ok(Self { client })
     }
 

tapo/examples/tapo_generic_device.rs

@@ -19,7 +19,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let tapo_password = env::var("TAPO_PASSWORD")?;
     let ip_address = env::var("IP_ADDRESS")?;
 
-    let device = ApiClient::new(tapo_username, tapo_password)?
+    let device = ApiClient::new(tapo_username, tapo_password)
         .generic_device(ip_address)
         .await?;
 

tapo/examples/tapo_generic_device_toggle.rs

@@ -19,7 +19,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let tapo_password = env::var("TAPO_PASSWORD")?;
     let ip_address = env::var("IP_ADDRESS")?;
 
-    let device = ApiClient::new(tapo_username, tapo_password)?
+    let device = ApiClient::new(tapo_username, tapo_password)
         .generic_device(ip_address)
         .await?;
 

tapo/examples/tapo_h100.rs

@@ -19,7 +19,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let tapo_password = env::var("TAPO_PASSWORD")?;
     let ip_address = env::var("IP_ADDRESS")?;
 
-    let hub = ApiClient::new(tapo_username, tapo_password)?
+    let hub = ApiClient::new(tapo_username, tapo_password)
         .h100(ip_address)
         .await?;
 

tapo/examples/tapo_ke100.rs

@@ -24,7 +24,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let device_id = env::var("DEVICE_ID")?;
     let target_temperature: u8 = env::var("TARGET_TEMPERATURE")?.parse()?;
 
-    let hub = ApiClient::new(tapo_username, tapo_password)?
+    let hub = ApiClient::new(tapo_username, tapo_password)
         .h100(ip_address)
         .await?;
 

tapo/examples/tapo_l510.rs

@@ -19,7 +19,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let tapo_password = env::var("TAPO_PASSWORD")?;
     let ip_address = env::var("IP_ADDRESS")?;
 
-    let device = ApiClient::new(tapo_username, tapo_password)?
+    let device = ApiClient::new(tapo_username, tapo_password)
         .l510(ip_address)
         .await?;
 

tapo/examples/tapo_l530.rs

@@ -19,7 +19,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let tapo_password = env::var("TAPO_PASSWORD")?;
     let ip_address = env::var("IP_ADDRESS")?;
 
-    let device = ApiClient::new(tapo_username, tapo_password)?
+    let device = ApiClient::new(tapo_username, tapo_password)
         .l530(ip_address)
         .await?;
 

tapo/examples/tapo_l930.rs

@@ -22,7 +22,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let tapo_password = env::var("TAPO_PASSWORD")?;
     let ip_address = env::var("IP_ADDRESS")?;
 
-    let device = ApiClient::new(tapo_username, tapo_password)?
+    let device = ApiClient::new(tapo_username, tapo_password)
         .l930(ip_address)
         .await?;
 

tapo/examples/tapo_p100.rs

@@ -19,7 +19,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let tapo_password = env::var("TAPO_PASSWORD")?;
     let ip_address = env::var("IP_ADDRESS")?;
 
-    let device = ApiClient::new(tapo_username, tapo_password)?
+    let device = ApiClient::new(tapo_username, tapo_password)
         .p100(ip_address)
         .await?;
 

tapo/examples/tapo_p110.rs

@@ -20,7 +20,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let tapo_password = env::var("TAPO_PASSWORD")?;
     let ip_address = env::var("IP_ADDRESS")?;
 
-    let device = ApiClient::new(tapo_username, tapo_password)?
+    let device = ApiClient::new(tapo_username, tapo_password)
         .p110(ip_address)
         .await?;