refactor: factor out notification sending

src/daemon.rs

@@ -1,7 +1,7 @@
+use crate::notification::send_notifictation;
 use crate::pomodoro::Pomodoro;
 pub use crate::timer::Timer;
 use anyhow::Context;
-use notify_rust::Notification;
 use serde::{Deserialize, Serialize};
 use std::fmt::{Display, Formatter};
 use std::{
@@ -100,21 +100,14 @@ impl Daemon {
                 }
 
                 if self.notify {
-                    match Notification::new()
+                    send_notifictation(
-                        .summary("󰀠 Timers")
+                        format!(
-                        .body(
+                            "Started timer {} for {}",
-                            format!(
+                            &name,
-                                "Started timer {} for {}",
+                            humantime::format_duration(duration)
-                                &name,
-                                humantime::format_duration(duration)
-                            )
-                            .as_str(),
                         )
-                        .show()
+                        .as_str(),
-                    {
+                    );
-                        Ok(_) => println!("Sent notification sucessfully."),
-                        Err(_) => println!("Failed to send notification."),
-                    };
                 }
 
                 let timer = Timer::new(name, duration);
@@ -135,14 +128,7 @@ impl Daemon {
                 long_pause,
                 pauses_till_long,
             } => {
-                match Notification::new()
+                send_notifictation("Started pomodoro.");
-                    .summary("󰀠 Timers")
-                    .body("Started pomodoro.")
-                    .show()
-                {
-                    Ok(_) => println!("Sent notification sucessfully."),
-                    Err(_) => println!("Failed to send notification."),
-                };
                 self.pomodoro = Some(Pomodoro::new(work, pause, long_pause, pauses_till_long));
                 Ok(Answer::Ok)
             }

src/main.rs

@@ -1,7 +1,8 @@
-pub mod cli;
+mod cli;
-pub mod daemon;
+mod daemon;
-pub mod pomodoro;
+mod pomodoro;
-pub mod timer;
+mod timer;
+mod notification;
 
 use crate::cli::{send_command, Cli, Command as CliCommand};
 use crate::daemon::{Command as DaemonCommand, Daemon};

src/notification.rs

@@ -0,0 +1,8 @@
+use notify_rust::Notification;
+
+pub fn send_notifictation(msg: &str) {
+    match Notification::new().summary("󰀠 Timers").body(msg).show() {
+        Ok(_) => println!("Sent notification sucessfully."),
+        Err(_) => println!("Failed to send notification."),
+    };
+}

src/timer.rs

@@ -1,4 +1,4 @@
-use notify_rust::Notification;
+use crate::notification::send_notifictation;
 use serde::{Deserialize, Serialize};
 use std::{
     fmt::{Display, Formatter},
@@ -74,10 +74,7 @@ impl Timer {
         let msg = format!("Timer {} has expired!", self.name);
         println!("{}", &msg);
         if notify {
-            match Notification::new().summary("󰀠 Timers").body(&msg).show() {
+            send_notifictation(msg.as_str());
-                Ok(_) => println!("Sent notification sucessfully."),
-                Err(_) => println!("Failed to send notification."),
-            }
         }
     }
 }