refactor: move output formatter to separate mod

src/main.rs

@@ -6,6 +6,7 @@ use ntex::{
 };
 
 mod error;
+mod output;
 
 type Result<T> = core::result::Result<T, error::ErrorWrapper>;
 
@@ -61,18 +62,13 @@ async fn poweroff(cfg: State<Config>) -> Result<impl Responder> {
 
 #[web::get("/ping")]
 async fn ping() -> Result<impl Responder> {
-    let out = tokio::process::Command::new("/bin/ping")
+    Ok(output::format(
-        .arg("-c")
+        tokio::process::Command::new("/bin/ping")
-        .arg("4")
+            .arg("-c")
-        .arg("1.1.1.1")
+            .arg("4")
-        .output()
+            .arg("1.1.1.1")
-        .await?;
+            .output()
-
+            .await?,
-    Ok(format!(
-        "exited with {}\n\nstdout:\n{}\n----\n\nstderr:\n{}",
-        out.status.code().unwrap_or(-1),
-        String::from_utf8_lossy(&out.stdout),
-        String::from_utf8_lossy(&out.stderr),
     ))
 }
 

src/output.rs

@@ -0,0 +1,10 @@
+use std::process::Output;
+
+pub fn format(out: Output) -> String {
+    format!(
+        "exited with {}\n\nstdout:\n{}\n----\n\nstderr:\n{}",
+        out.status.code().unwrap_or(-1),
+        String::from_utf8_lossy(&out.stdout),
+        String::from_utf8_lossy(&out.stderr),
+    )
+}