Merge pull request 'Coverart changes' (#65) from coverart_changes into main

Reviewed-on: #65

Cargo.lock

@@ -142,7 +142,7 @@ checksum = "5971ac85611da7067dbfcabef3c70ebb5606018acd9e2a3903a0da507521e0d5"
 
 [[package]]
 name = "icarus_models"
-version = "0.6.5"
+version = "0.6.6"
 dependencies = [
  "josekit",
  "rand",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "icarus_models"
-version = "0.6.5"
+version = "0.6.6"
 edition = "2024"
 rust-version = "1.88"
 description = "models used for the icarus project"

src/coverart.rs

@@ -39,7 +39,20 @@ impl CoverArt {
         }
     }
 
-    // TODO: Add method to remove from filesystem
+    /// Removes the coverart from the filesystem
+    pub fn remove_from_filesystem(&self) -> Result<(), std::io::Error> {
+        let p = std::path::Path::new(&self.path);
+        if p.exists() {
+            match std::fs::remove_file(p) {
+                Ok(_) => Ok(()),
+                Err(err) => Err(err),
+            }
+        } else {
+            Err(std::io::Error::other(
+                "Cannot delete file that does not exist",
+            ))
+        }
+    }
 }
 
 pub mod io {

src/song.rs

@@ -118,7 +118,9 @@ impl Song {
                         Err(err) => Err(err),
                     }
                 } else {
-                    Ok(())
+                    Err(std::io::Error::other(
+                        "Cannot delete file that does not exist",
+                    ))
                 }
             }
             Err(err) => Err(err),