Version bump

Cargo.lock

@@ -149,7 +149,7 @@ checksum = "5419bdc4f6a9207fbeba6d11b604d481addf78ecd10c11ad51e76c2f6482748d"
 
 [[package]]
 name = "icarus_models"
-version = "0.8.4"
+version = "0.8.3"
 dependencies = [
  "josekit",
  "rand",

Cargo.toml

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

src/coverart.rs

@@ -1,10 +1,11 @@
 use std::io::Write;
 
 use rand::Rng;
+use serde::{Deserialize, Serialize};
 
 const FILENAME_LENGTH: i32 = 16;
 
-#[derive(Clone, Debug, Default, serde::Deserialize, serde::Serialize, utoipa::ToSchema)]
+#[derive(Clone, Debug, Default, Deserialize, Serialize, utoipa::ToSchema)]
 pub struct CoverArt {
     pub id: uuid::Uuid,
     pub title: String,
@@ -17,9 +18,11 @@ pub struct CoverArt {
 }
 
 pub mod init {
+    use super::CoverArt;
+
     /// Initializes the CoverArt with just the directory and filename
-    pub fn init_coverart_dir_and_filename(directory: &str, filename: &str) -> super::CoverArt {
+    pub fn init_coverart_dir_and_filename(directory: &str, filename: &str) -> CoverArt {
-        super::CoverArt {
+        CoverArt {
             directory: String::from(directory),
             filename: String::from(filename),
             ..Default::default()
@@ -85,10 +88,7 @@ impl CoverArt {
 }
 
 /// Generates filename for a CoverArt
-pub fn generate_filename(
+pub fn generate_filename(typ: crate::types::CoverArtTypes, randomize: bool) -> String {
-    typ: crate::types::CoverArtTypes,
-    randomize: bool,
-) -> Result<String, std::io::Error> {
     let file_extension = match typ {
         crate::types::CoverArtTypes::PngExtension => {
             String::from(crate::constants::file_extensions::image::PNGEXTENSION)
@@ -99,12 +99,9 @@ pub fn generate_filename(
         crate::types::CoverArtTypes::JpgExtension => {
             String::from(crate::constants::file_extensions::image::JPGEXTENSION)
         }
-        crate::types::CoverArtTypes::None => {
-            return Err(std::io::Error::other("Unsupported CoverArtTypes"));
-        }
     };
 
-    let filename: String = if randomize {
+    if randomize {
         let mut filename: String = String::from("coverart-");
         let some_chars: String = String::from("abcdefghij0123456789");
         let some_chars_length = some_chars.len();
@@ -118,12 +115,10 @@ pub fn generate_filename(
                 filename.push(c);
             }
         }
-        format!("{filename}{file_extension}")
+        filename + &file_extension
     } else {
-        format!("coverart-output{file_extension}")
+        "coverart-output".to_string() + &file_extension
-    };
+    }
-
-    Ok(filename)
 }
 
 pub mod io {

src/song.rs

@@ -126,10 +126,7 @@ impl Song {
 }
 
 /// Generates a filename. In order to save a song to the filesystem
-pub fn generate_filename(
+pub fn generate_filename(typ: types::MusicTypes, randomize: bool) -> String {
-    typ: types::MusicTypes,
-    randomize: bool,
-) -> Result<String, std::io::Error> {
     let file_extension = match typ {
         types::MusicTypes::DefaultMusicExtension => {
             String::from(constants::file_extensions::audio::DEFAULTMUSICEXTENSION)
@@ -143,10 +140,9 @@ pub fn generate_filename(
         types::MusicTypes::MPThreeExtension => {
             String::from(constants::file_extensions::audio::MPTHREEEXTENSION)
         }
-        types::MusicTypes::None => return Err(std::io::Error::other("Unsupported MusicTypes")),
     };
 
-    let filename: String = if randomize {
+    if randomize {
         let mut filename: String = String::from("track-");
         let some_chars: String = String::from("abcdefghij0123456789");
         let some_chars_length = some_chars.len();
@@ -160,12 +156,10 @@ pub fn generate_filename(
                 filename.push(c);
             }
         }
-        format!("{filename}{file_extension}")
+        filename + &file_extension
     } else {
-        format!("track-output{file_extension}")
+        "track-output".to_string() + &file_extension
-    };
+    }
-
-    Ok(filename)
 }
 
 /// I/O operations for songs

src/types.rs

@@ -4,7 +4,6 @@ pub enum MusicTypes {
     WavExtension,
     FlacExtension,
     MPThreeExtension,
-    None,
 }
 
 #[derive(Debug)]
@@ -12,5 +11,4 @@ pub enum CoverArtTypes {
     PngExtension,
     JpegExtension,
     JpgExtension,
-    None,
 }

tests/tests.rs

@@ -107,21 +107,12 @@ mod song_tests {
         };
 
         assert_eq!(song.directory.is_empty(), false);
-        match song::generate_filename(types::MusicTypes::FlacExtension, true) {
+        song_cpy.filename = song::generate_filename(types::MusicTypes::FlacExtension, true);
-            Ok(filename) => {
-                song_cpy.filename = filename;
-            }
-            Err(err) => {
-                assert!(false, "Error generatig filename: {err:?}");
-            }
-        };
         println!("Directory: {:?}", song_cpy.directory);
         println!("File to be created: {:?}", song_cpy.filename);
 
         match song::io::copy_song(&song, &mut song_cpy) {
-            Ok(_) => {
+            Ok(_) => {}
-                println!("Song copied");
-            }
             Err(err) => {
                 assert!(false, "Error copying song: Error: {err:?}")
             }