Deal with all lints

[rbdr/blog] / src / command / publish.rs

diff --git a/src/command/publish.rs b/src/command/publish.rs
index 207b45ddeda0cb56d913ffa863a0ee30ad2a12a7..388792ff54bcd736d447252910a20070b84160c5 100644 (file)
--- a/src/command/publish.rs
+++ b/src/command/publish.rs
@@ -1,5 +1,5 @@
-use std::io::Result;

 use crate::configuration::Configuration;
 use crate::configuration::Configuration;

+use std::io::{Error, ErrorKind::Other, Result};

 use std::process::{Command, Stdio};
 
 const COMMAND: &str = "rsync";
 use std::process::{Command, Stdio};
 
 const COMMAND: &str = "rsync";


@@ -17,27 +17,34 @@ impl super::Command for Publish {
         vec![]
     }
 
         vec![]
     }
 

-    fn execute(&self, input: Option<&String>, configuration: &Configuration, _: &String) -> Result<()> {
-
-        let input = input.expect("You must provide a location to publish the blog");
+    fn execute(
+        &self,
+        input: Option<&String>,
+        configuration: &Configuration,
+        _: &str,
+    ) -> Result<()> {
+        let input = input
+            .ok_or_else(|| Error::new(Other, "You must provide a location to publish the blog"))?;

 
         Command::new(COMMAND)
             .arg("--version")
             .stdout(Stdio::null())
             .stderr(Stdio::null())
             .status()
 
         Command::new(COMMAND)
             .arg("--version")
             .stdout(Stdio::null())
             .stderr(Stdio::null())
             .status()

-            .expect("Publishing requires rsync");
-
+            .map_err(|_| Error::new(Other, "Publishing requires rsync"))?;

 
         Command::new(COMMAND)
             .arg("-r")
 
         Command::new(COMMAND)
             .arg("-r")

-            .arg(format!("{}/", &configuration.blog_output_directory.display()))
+            .arg(format!(
+                "{}/",
+                &configuration.blog_output_directory.display()
+            ))

             .arg(input.as_str())
             .stdout(Stdio::null())
             .stderr(Stdio::null())
             .status()
             .arg(input.as_str())
             .stdout(Stdio::null())
             .stderr(Stdio::null())
             .status()

-            .expect("Publishing requires rsync");
-        return Ok(())
+            .map_err(|_| Error::new(Other, "Rsync failed to publish."))?;
+        Ok(())

     }
 
     fn after_dependencies(&self) -> Vec<Box<dyn super::Command>> {
     }
 
     fn after_dependencies(&self) -> Vec<Box<dyn super::Command>> {