[rust] Clean code after merging last PRs

bonigarcia · bonigarcia · commit 601b54a33fd9 · 2023-04-09T01:54:21.000+02:00
diff --git a/rust/src/main.rs b/rust/src/main.rs
@@ -141,15 +141,15 @@ fn main() {
         .set_timeout(cli.timeout)
         .and_then(|_| selenium_manager.set_proxy(cli.proxy.unwrap_or_default()))
         .and_then(|_| selenium_manager.resolve_driver())
-        .and_then(|path| {
+        .map(|path| {
             let log = selenium_manager.get_logger();
             log.info(path.display().to_string());
-            flush_and_exit(OK, &log);
+            flush_and_exit(OK, log);
         })
         .unwrap_or_else(|err| {
             let log = selenium_manager.get_logger();
             log.error(err.to_string());
-            flush_and_exit(DATAERR, &log);
+            flush_and_exit(DATAERR, log);
         });
 }
 
diff --git a/rust/src/metadata.rs b/rust/src/metadata.rs
@@ -17,9 +17,9 @@
 
 use std::fs;
 use std::fs::File;
+
 use std::path::PathBuf;
 use std::time::{SystemTime, UNIX_EPOCH};
-use std::io::Error;
 
 use serde::{Deserialize, Serialize};
 
@@ -156,10 +156,13 @@ pub fn write_metadata(metadata: &Metadata, log: &Logger) {
     .unwrap();
 }
 
-pub fn clear_metadata(log: &Logger) -> Result<(), Error> {
+pub fn clear_metadata(log: &Logger) {
     let metadata_path = get_metadata_path();
-    log.trace(format!("Deleting metadata file {}", metadata_path.display()));
-    fs::remove_file(metadata_path)?;
-    log.trace("Metadata file was deleted".to_string());
-    Ok(())
+    log.trace(format!(
+        "Deleting metadata file {}",
+        metadata_path.display()
+    ));
+    fs::remove_file(metadata_path).unwrap_or_else(|err| {
+        log.warn(format!("Error deleting metadata file: {}", err));
+    });
 }
