Version bump

.gitea/workflows/tag_release.yaml

@@ -1,7 +1,7 @@
 name: Release Tagging
 
 on:
-  push:
+  pull_request:
     branches:
       - main
 
@@ -10,14 +10,14 @@ jobs:
     runs-on: ubuntu-24.04
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v3
         with:
           fetch-depth: 0 # Important for git describe --tags
 
       - name: Install Rust
         uses: actions-rs/toolchain@v1
         with:
-          toolchain: 1.90.0
+          toolchain: 1.88.0
           components: cargo
 
       - name: Extract Version from Cargo.toml
@@ -25,15 +25,12 @@ jobs:
         run: |
           VERSION=$(grep '^version = "' Cargo.toml | sed -E 's/version = "([^"]+)"/\1/')
           PROJECT_COMMIT_HASH=$(git rev-parse HEAD | cut -c 1-10)
-          BRANCH_REF="${{ gitea.ref }}"
+          BRANCH_REF="${GITHUB_REF}"
           BRANCH_NAME=$(echo "$BRANCH_REF" | cut -d '/' -f 3)
-          PROJECT_TAG_RELEASE="v$VERSION-$BRANCH_NAME-$PROJECT_COMMIT_HASH-111"
-
-          echo "::set-output name=project_tag_release::$PROJECT_TAG_RELEASE"
-
+          PROJECT_TAG_RELEASE="v$VERSION-$BRANCH_NAME-$PROJECT_COMMIT_HASH"
+          echo "::set-output name=project_tag_release::$PROJECT_TAG_RELEASE-111"
           echo "Version: $VERSION"
           echo "Hash: $PROJECT_COMMIT_HASH"
-          echo "Branh ref: $BRANCH_REF"
           echo "Branch: $BRANCH_NAME"
           echo "Tag Release: $PROJECT_TAG_RELEASE"
 

.gitea/workflows/workflow.yaml

@@ -13,30 +13,30 @@ jobs:
     name: Check
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.90.0
+          toolchain: 1.88.0
       - run: cargo check
 
   test:
     name: Test Suite
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.90.0
+          toolchain: 1.88.0
       - run: cargo test
 
   fmt:
     name: Rustfmt
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.90.0
+          toolchain: 1.88.0
       - run: rustup component add rustfmt
       - run: cargo fmt --all -- --check
 
@@ -44,10 +44,10 @@ jobs:
     name: Clippy
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.90.0
+          toolchain: 1.88.0
       - run: rustup component add clippy
       - run: cargo clippy -- -D warnings
 
@@ -55,8 +55,10 @@ jobs:
     name: build
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.90.0
+          toolchain: 1.88.0
       - run: cargo build
+
+

Cargo.lock

@@ -10,9 +10,9 @@ checksum = "320119579fcad9c21884f5c4861d16174d0e06250625266f50fe6898340abefa"
 
 [[package]]
 name = "aho-corasick"
-version = "1.1.4"
+version = "1.1.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ddd31a130427c27518df266943a5308ed92d4b226cc639f5a8f1002816174301"
+checksum = "8e60d3430d3a69478ad0993f19238d2df97c507009a52b3c10addcd7f6bcb916"
 dependencies = [
  "memchr",
 ]
@@ -31,9 +31,9 @@ checksum = "72b3254f16251a8381aa12e40e3c4d2f0199f8c6508fbecb9d91f575e0fbb8c6"
 
 [[package]]
 name = "bitflags"
-version = "2.10.0"
+version = "2.9.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "812e12b5285cc515a9c72a5c1d3b6d46a19dac5acfef5265968c166106e31dd3"
+checksum = "2261d10cca569e4643e526d8dc2e62e433cc8aba21ab764233731f8d369bf394"
 
 [[package]]
 name = "bumpalo"
@@ -43,9 +43,9 @@ checksum = "46c5e41b57b8bba42a04676d81cb89e9ee8e859a1a66f80a5a72e1cb76b34d43"
 
 [[package]]
 name = "cc"
-version = "1.2.43"
+version = "1.2.41"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "739eb0f94557554b3ca9a86d2d37bebd49c5e6d0c1d2bda35ba5bdac830befc2"
+checksum = "ac9fe6cdbb24b6ade63616c0a0688e45bb56732262c158df3c0c4bea4ca47cb7"
 dependencies = [
  "find-msvc-tools",
  "shlex",
@@ -53,9 +53,9 @@ dependencies = [
 
 [[package]]
 name = "cfg-if"
-version = "1.0.4"
+version = "1.0.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9330f8b2ff13f34540b44e946ef35111825727b38d33286ef986142615121801"
+checksum = "2fd1289c04a9ea8cb22300a459a72a385d7c73d3259e2ed7dcb2af674838cfa9"
 
 [[package]]
 name = "crc32fast"
@@ -68,9 +68,9 @@ dependencies = [
 
 [[package]]
 name = "deranged"
-version = "0.5.5"
+version = "0.5.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ececcb659e7ba858fb4f10388c250a7252eb0a27373f1a72b8748afdd248e587"
+checksum = "a41953f86f8a05768a6cda24def994fd2f424b04ec5c719cf89989779f199071"
 dependencies = [
  "powerfmt",
  "serde_core",
@@ -106,9 +106,9 @@ checksum = "52051878f80a721bb68ebfbc930e07b65ba72f2da88968ea5c06fd6ca3d3a127"
 
 [[package]]
 name = "flate2"
-version = "1.1.5"
+version = "1.1.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "bfe33edd8e85a12a67454e37f8c75e730830d83e313556ab9ebf9ee7fbeb3bfb"
+checksum = "dc5a4e564e38c699f2880d3fda590bedc2e69f3f84cd48b457bd892ce61d0aa9"
 dependencies = [
  "crc32fast",
  "miniz_oxide",
@@ -131,14 +131,14 @@ checksum = "00b0228411908ca8685dba7fc2cdd70ec9990a6e753e89b6ac91a84c40fbaf4b"
 
 [[package]]
 name = "getrandom"
-version = "0.3.4"
+version = "0.3.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "899def5c37c4fd7b2664648c28120ecec138e4d395b459e5ca34f9cce2dd77fd"
+checksum = "26145e563e54f2cadc477553f1ec5ee650b00862f0a58bcd12cbdc5f0ea2d2f4"
 dependencies = [
  "cfg-if",
  "libc",
  "r-efi",
- "wasip2",
+ "wasi",
 ]
 
 [[package]]
@@ -149,7 +149,7 @@ checksum = "5419bdc4f6a9207fbeba6d11b604d481addf78ecd10c11ad51e76c2f6482748d"
 
 [[package]]
 name = "icarus_models"
-version = "0.9.0"
+version = "0.6.7"
 dependencies = [
  "josekit",
  "rand",
@@ -163,9 +163,9 @@ dependencies = [
 
 [[package]]
 name = "indexmap"
-version = "2.12.0"
+version = "2.11.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6717a8d2a5a929a1a2eb43a12812498ed141a0bcfb7e8f7844fbdbe4303bba9f"
+checksum = "4b0f83760fb341a774ed326568e19f5a863af4a952def8c39f9ab92fd95b88e5"
 dependencies = [
  "equivalent",
  "hashbrown",
@@ -198,9 +198,9 @@ dependencies = [
 
 [[package]]
 name = "js-sys"
-version = "0.3.82"
+version = "0.3.81"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b011eec8cc36da2aab2d5cff675ec18454fad408585853910a202391cf9f8e65"
+checksum = "ec48937a97411dcb524a265206ccd4c90bb711fca92b2792c407f268825b9305"
 dependencies = [
  "once_cell",
  "wasm-bindgen",
@@ -218,6 +218,12 @@ version = "0.11.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "df1d3c3b53da64cf5760482273a98e575c651a67eec7f77df96b5b642de8f039"
 
+[[package]]
+name = "log"
+version = "0.4.28"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "34080505efa8e45a4b816c349525ebe327ceaa8559756f0356cba97ef3bf7432"
+
 [[package]]
 name = "memchr"
 version = "2.7.6"
@@ -248,9 +254,9 @@ checksum = "42f5e15c9953c5e4ccceeb2e7382a716482c34515315f7b03532b8b4e8393d2d"
 
 [[package]]
 name = "openssl"
-version = "0.10.74"
+version = "0.10.73"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "24ad14dd45412269e1a30f52ad8f0664f0f4f4a89ee8fe28c3b3527021ebb654"
+checksum = "8505734d46c8ab1e19a1dce3aef597ad87dcb4c37e7188231769bd6bd51cebf8"
 dependencies = [
  "bitflags",
  "cfg-if",
@@ -274,9 +280,9 @@ dependencies = [
 
 [[package]]
 name = "openssl-sys"
-version = "0.9.110"
+version = "0.9.109"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "0a9f0075ba3c21b09f8e8b2026584b1d18d49388648f2fbbf3c97ea8deced8e2"
+checksum = "90096e2e47630d78b7d1c20952dc621f957103f8bc2c8359ec81290d75238571"
 dependencies = [
  "cc",
  "libc",
@@ -307,9 +313,9 @@ dependencies = [
 
 [[package]]
 name = "proc-macro2"
-version = "1.0.103"
+version = "1.0.101"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "5ee95bc4ef87b8d5ba32e8b7714ccc834865276eab0aed5c9958d00ec45f49e8"
+checksum = "89ae43fd86e4158d6db51ad8e2b80f313af9cc74f5c0e03ccb87de09998732de"
 dependencies = [
  "unicode-ident",
 ]
@@ -360,9 +366,9 @@ dependencies = [
 
 [[package]]
 name = "regex"
-version = "1.12.2"
+version = "1.12.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "843bc0191f75f3e22651ae5f1e72939ab2f72a4bc30fa80a066bd66edefc24d4"
+checksum = "4a52d8d02cacdb176ef4678de6c052efb4b3da14b78e4db683a4252762be5433"
 dependencies = [
  "aho-corasick",
  "memchr",
@@ -372,9 +378,9 @@ dependencies = [
 
 [[package]]
 name = "regex-automata"
-version = "0.4.13"
+version = "0.4.12"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "5276caf25ac86c8d810222b3dbb938e512c55c6831a10f3e6ed1c93b84041f1c"
+checksum = "722166aa0d7438abbaa4d5cc2c649dac844e8c56d82fb3d33e9c34b5cd268fc6"
 dependencies = [
  "aho-corasick",
  "memchr",
@@ -383,9 +389,9 @@ dependencies = [
 
 [[package]]
 name = "regex-syntax"
-version = "0.8.8"
+version = "0.8.7"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7a2d987857b319362043e95f5353c0535c1f58eec5336fdfcf626430af7def58"
+checksum = "c3160422bbd54dd5ecfdca71e5fd59b7b8fe2b1697ab2baf64f6d05dcc66d298"
 
 [[package]]
 name = "rustix"
@@ -470,9 +476,9 @@ checksum = "d66dc143e6b11c1eddc06d5c423cfc97062865baf299914ab64caa38182078fe"
 
 [[package]]
 name = "syn"
-version = "2.0.108"
+version = "2.0.106"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "da58917d35242480a05c2897064da0a80589a2a0476c9a3f2fdc83b53502e917"
+checksum = "ede7c438028d4436d71104916910f5bb611972c5cfd7f89b8300a8186e6fada6"
 dependencies = [
  "proc-macro2",
  "quote",
@@ -545,9 +551,9 @@ dependencies = [
 
 [[package]]
 name = "unicode-ident"
-version = "1.0.20"
+version = "1.0.19"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "462eeb75aeb73aea900253ce739c8e18a67423fadf006037cd3ff27e82748a06"
+checksum = "f63a545481291138910575129486daeaf8ac54aee4387fe7906919f7830c7d9d"
 
 [[package]]
 name = "utoipa"
@@ -591,6 +597,15 @@ version = "0.2.15"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "accd4ea62f7bb7a82fe23066fb0957d48ef677f6eeb8215f372f52e48bb32426"
 
+[[package]]
+name = "wasi"
+version = "0.14.7+wasi-0.2.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "883478de20367e224c0090af9cf5f9fa85bed63a95c1abf3afc5c083ebc06e8c"
+dependencies = [
+ "wasip2",
+]
+
 [[package]]
 name = "wasip2"
 version = "1.0.1+wasi-0.2.4"
@@ -602,9 +617,9 @@ dependencies = [
 
 [[package]]
 name = "wasm-bindgen"
-version = "0.2.105"
+version = "0.2.104"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "da95793dfc411fbbd93f5be7715b0578ec61fe87cb1a42b12eb625caa5c5ea60"
+checksum = "c1da10c01ae9f1ae40cbfac0bac3b1e724b320abfcf52229f80b547c0d250e2d"
 dependencies = [
  "cfg-if",
  "once_cell",
@@ -614,22 +629,13 @@ dependencies = [
 ]
 
 [[package]]
-name = "wasm-bindgen-macro"
-version = "0.2.105"
+name = "wasm-bindgen-backend"
+version = "0.2.104"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "04264334509e04a7bf8690f2384ef5265f05143a4bff3889ab7a3269adab59c2"
-dependencies = [
- "quote",
- "wasm-bindgen-macro-support",
-]
-
-[[package]]
-name = "wasm-bindgen-macro-support"
-version = "0.2.105"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "420bc339d9f322e562942d52e115d57e950d12d88983a14c79b86859ee6c7ebc"
+checksum = "671c9a5a66f49d8a47345ab942e2cb93c7d1d0339065d4f8139c486121b43b19"
 dependencies = [
  "bumpalo",
+ "log",
  "proc-macro2",
  "quote",
  "syn",
@@ -637,10 +643,33 @@ dependencies = [
 ]
 
 [[package]]
-name = "wasm-bindgen-shared"
-version = "0.2.105"
+name = "wasm-bindgen-macro"
+version = "0.2.104"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "76f218a38c84bcb33c25ec7059b07847d465ce0e0a76b995e134a45adcb6af76"
+checksum = "7ca60477e4c59f5f2986c50191cd972e3a50d8a95603bc9434501cf156a9a119"
+dependencies = [
+ "quote",
+ "wasm-bindgen-macro-support",
+]
+
+[[package]]
+name = "wasm-bindgen-macro-support"
+version = "0.2.104"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "9f07d2f20d4da7b26400c9f4a0511e6e0345b040694e8a75bd41d578fa4421d7"
+dependencies = [
+ "proc-macro2",
+ "quote",
+ "syn",
+ "wasm-bindgen-backend",
+ "wasm-bindgen-shared",
+]
+
+[[package]]
+name = "wasm-bindgen-shared"
+version = "0.2.104"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "bad67dc8b2a1a6e5448428adec4c3e84c43e561d8c9ee8a9e5aabeb193ec41d1"
 dependencies = [
  "unicode-ident",
 ]

Cargo.toml

@@ -1,8 +1,8 @@
 [package]
 name = "icarus_models"
-version = "0.9.0"
+version = "0.6.7"
 edition = "2024"
-rust-version = "1.90"
+rust-version = "1.88"
 description = "models used for the icarus project"
 license = "MIT"
 

README.md

@@ -1,3 +1,2 @@
-A library containing commonly used structs, functions, enums, constants and other code
-that is used throughout the icarus projects. Code from this library serves as the model
-for other projects in the icarus project.
+A library containing commonly used models and functions that is used throughout
+icarus projects. This reduces the amount of duplicated code without a benefit.

src/constants.rs

@@ -12,9 +12,3 @@ pub mod file_extensions {
         pub const PNGEXTENSION: &str = ".png";
     }
 }
-
-pub mod error {
-    pub const DIRECTORY_NOT_INITIALIZED: &str = "Directory has not been initialized";
-    pub const FILENAME_NOT_INITIALIZED: &str = "Filename has not bee initialized";
-    pub const LAST_CHARACTER_IN_DIRECTORY: &str = "Could not access last character of directory";
-}

src/coverart.rs

@@ -1,28 +1,28 @@
 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,
     #[serde(skip)]
-    pub directory: String,
-    pub filename: String,
+    pub path: String,
     #[serde(skip)]
     pub data: Vec<u8>,
     pub song_id: uuid::Uuid,
 }
 
 pub mod init {
-    /// Initializes the CoverArt with just the directory and filename
-    pub fn init_coverart_dir_and_filename(directory: &str, filename: &str) -> super::CoverArt {
-        super::CoverArt {
-            directory: String::from(directory),
-            filename: String::from(filename),
-            ..Default::default()
+    use crate::coverart::CoverArt;
+
+    pub fn init_coverart_only_path(path: String) -> CoverArt {
+        CoverArt {
+            id: uuid::Uuid::nil(),
+            title: String::new(),
+            path: path.clone(),
+            data: Vec::new(),
+            song_id: uuid::Uuid::nil(),
         }
     }
 }
@@ -30,12 +30,9 @@ pub mod init {
 impl CoverArt {
     /// Saves the coverart to the filesystem
     pub fn save_to_filesystem(&self) -> Result<(), std::io::Error> {
-        match self.get_path() {
-            Ok(path) => match std::fs::File::create(&path) {
-                Ok(mut file) => match file.write_all(&self.data) {
-                    Ok(_) => Ok(()),
-                    Err(err) => Err(err),
-                },
+        match std::fs::File::create(&self.path) {
+            Ok(mut file) => match file.write_all(&self.data) {
+                Ok(_) => Ok(()),
                 Err(err) => Err(err),
             },
             Err(err) => Err(err),
@@ -44,86 +41,18 @@ impl CoverArt {
 
     /// Removes the coverart from the filesystem
     pub fn remove_from_filesystem(&self) -> Result<(), std::io::Error> {
-        match self.get_path() {
-            Ok(path) => {
-                let p = std::path::Path::new(&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",
-                    ))
-                }
+        let p = std::path::Path::new(&self.path);
+        if p.exists() {
+            match std::fs::remove_file(p) {
+                Ok(_) => Ok(()),
+                Err(err) => Err(err),
             }
-            Err(err) => Err(err),
+        } else {
+            Err(std::io::Error::other(
+                "Cannot delete file that does not exist",
+            ))
         }
     }
-
-    /// Gets the path of the CoverArt
-    pub fn get_path(&self) -> Result<String, std::io::Error> {
-        if self.directory.is_empty() {
-            return Err(std::io::Error::other(
-                crate::constants::error::DIRECTORY_NOT_INITIALIZED,
-            ));
-        } else if self.filename.is_empty() {
-            return Err(std::io::Error::other(
-                crate::constants::error::FILENAME_NOT_INITIALIZED,
-            ));
-        }
-
-        let directory = &self.directory;
-        let last_index = directory.len() - 1;
-
-        match crate::util::concatenate_path(directory, &self.filename, last_index) {
-            Ok(path) => Ok(path),
-            Err(err) => Err(err),
-        }
-    }
-}
-
-/// Generates filename for a CoverArt
-pub fn generate_filename(
-    typ: crate::types::CoverArtType,
-    randomize: bool,
-) -> Result<String, std::io::Error> {
-    let file_extension = match typ {
-        crate::types::CoverArtType::PngExtension => {
-            String::from(crate::constants::file_extensions::image::PNGEXTENSION)
-        }
-        crate::types::CoverArtType::JpegExtension => {
-            String::from(crate::constants::file_extensions::image::JPEGEXTENSION)
-        }
-        crate::types::CoverArtType::JpgExtension => {
-            String::from(crate::constants::file_extensions::image::JPGEXTENSION)
-        }
-        crate::types::CoverArtType::None => {
-            return Err(std::io::Error::other("Unsupported CoverArtTypes"));
-        }
-    };
-
-    let filename: String = if randomize {
-        let mut filename: String = String::from("coverart-");
-        let some_chars: String = String::from("abcdefghij0123456789");
-        let some_chars_length = some_chars.len();
-        let mut rng = rand::rng();
-
-        for _ in 0..FILENAME_LENGTH {
-            let index = rng.random_range(0..=some_chars_length);
-            let rando_char = some_chars.chars().nth(index);
-
-            if let Some(c) = rando_char {
-                filename.push(c);
-            }
-        }
-        format!("{filename}{file_extension}")
-    } else {
-        format!("coverart-output{file_extension}")
-    };
-
-    Ok(filename)
 }
 
 pub mod io {
@@ -131,15 +60,11 @@ pub mod io {
 
     /// Gets the raw data of the cover art
     pub fn to_data(coverart: &super::CoverArt) -> Result<Vec<u8>, std::io::Error> {
-        match coverart.get_path() {
-            Ok(path) => {
-                let mut file = std::fs::File::open(path)?;
-                let mut buffer = Vec::new();
-                match file.read_to_end(&mut buffer) {
-                    Ok(_) => Ok(buffer),
-                    Err(err) => Err(err),
-                }
-            }
+        let path: &String = &coverart.path;
+        let mut file = std::fs::File::open(path)?;
+        let mut buffer = Vec::new();
+        match file.read_to_end(&mut buffer) {
+            Ok(_) => Ok(buffer),
             Err(err) => Err(err),
         }
     }
@@ -151,11 +76,9 @@ mod tests {
 
     #[test]
     fn test_cover_art_image() {
-        let dir = String::from("./");
-        let filename = String::from("CoverArt.png");
-        let coverart = coverart::init::init_coverart_dir_and_filename(&dir, &filename);
+        let path: String = String::from("somepath");
+        let coverart = coverart::init::init_coverart_only_path(path.clone());
 
-        assert_eq!(dir, coverart.directory);
-        assert_eq!(filename, coverart.filename);
+        assert_eq!(path, coverart.path);
     }
 }

src/lib.rs

@@ -7,7 +7,6 @@ pub mod song;
 pub mod token;
 pub mod types;
 pub mod user;
-pub mod util;
 
 pub mod init {
     pub fn is_id_valid(num: &i32) -> bool {

src/login_result.rs

@@ -2,7 +2,7 @@ use std::default::Default;
 
 use serde::{Deserialize, Serialize};
 
-#[derive(Clone, Debug, Default, Deserialize, Serialize, utoipa::ToSchema)]
+#[derive(Clone, Debug, Deserialize, Serialize, utoipa::ToSchema)]
 pub struct LoginResult {
     pub id: uuid::Uuid,
     pub username: String,
@@ -12,14 +12,20 @@ pub struct LoginResult {
     pub expiration: i64,
 }
 
-impl LoginResult {
-    pub fn to_json(&self) -> Result<String, serde_json::Error> {
-        serde_json::to_string_pretty(&self)
-    }
-
-    pub fn token_expired(&self) -> bool {
-        let current_time = time::OffsetDateTime::now_utc();
-        let expired = time::OffsetDateTime::from_unix_timestamp(self.expiration).unwrap();
-        current_time > expired
+impl Default for LoginResult {
+    fn default() -> Self {
+        LoginResult {
+            id: uuid::Uuid::nil(),
+            username: String::new(),
+            token: String::new(),
+            token_type: String::new(),
+            expiration: -1,
+        }
+    }
+}
+
+impl LoginResult {
+    pub fn _to_json(&self) -> Result<String, serde_json::Error> {
+        serde_json::to_string_pretty(&self)
     }
 }

src/song.rs

@@ -39,8 +39,8 @@ pub struct Song {
     pub track_count: i32,
     #[serde(skip_serializing_if = "String::is_empty")]
     pub audio_type: String,
-    #[serde(with = "time::serde::rfc3339::option")]
-    pub date_created: Option<time::OffsetDateTime>,
+    #[serde(skip_serializing_if = "String::is_empty")]
+    pub date_created: String,
     #[serde(skip_serializing_if = "String::is_empty")]
     pub filename: String,
     #[serde(skip_serializing_if = "init::is_uuid_nil")]
@@ -69,24 +69,27 @@ impl Song {
         }
     }
 
-    /// Gets the path of a Song
     pub fn song_path(&self) -> Result<String, std::io::Error> {
         if self.directory.is_empty() {
-            return Err(std::io::Error::other(
-                crate::constants::error::DIRECTORY_NOT_INITIALIZED,
-            ));
-        } else if self.filename.is_empty() {
-            return Err(std::io::Error::other(
-                crate::constants::error::FILENAME_NOT_INITIALIZED,
-            ));
+            return Err(std::io::Error::other("Directory does not exist"));
         }
 
         let directory = &self.directory;
+        let mut buffer: String = directory.clone();
         let last_index = directory.len() - 1;
 
-        match crate::util::concatenate_path(directory, &self.filename, last_index) {
-            Ok(path) => Ok(path),
-            Err(err) => Err(err),
+        if let Some(character) = directory.chars().nth(last_index) {
+            if character != '/' {
+                buffer += "/";
+            }
+
+            buffer += &self.filename.clone();
+
+            Ok(buffer)
+        } else {
+            Err(std::io::Error::other(
+                "Could not access last character of directory",
+            ))
         }
     }
 
@@ -125,44 +128,42 @@ impl Song {
     }
 }
 
-/// Generates a filename. In order to save a song to the filesystem
-pub fn generate_filename(typ: types::MusicType, randomize: bool) -> Result<String, std::io::Error> {
+/// Generates a filename. In order to save a song to the filesystem, the song must have
+/// a directory and filename
+pub fn generate_filename(typ: types::MusicTypes, randomize: bool) -> String {
     let file_extension = match typ {
-        types::MusicType::DefaultMusicExtension => {
+        types::MusicTypes::DefaultMusicExtension => {
             String::from(constants::file_extensions::audio::DEFAULTMUSICEXTENSION)
         }
-        types::MusicType::WavExtension => {
+
+        types::MusicTypes::WavExtension => {
             String::from(constants::file_extensions::audio::WAVEXTENSION)
         }
-        types::MusicType::FlacExtension => {
+        types::MusicTypes::FlacExtension => {
             String::from(constants::file_extensions::audio::FLACEXTENSION)
         }
-        types::MusicType::MPThreeExtension => {
+        types::MusicTypes::MPThreeExtension => {
             String::from(constants::file_extensions::audio::MPTHREEEXTENSION)
         }
-        types::MusicType::None => return Err(std::io::Error::other("Unsupported MusicTypes")),
     };
 
-    let filename: String = if randomize {
-        let mut filename: String = String::from("track-");
+    if randomize {
+        let mut filename: String = String::new();
         let some_chars: String = String::from("abcdefghij0123456789");
-        let some_chars_length = some_chars.len();
         let mut rng = rand::rng();
 
         for _ in 0..FILENAME_LENGTH {
-            let index = rng.random_range(0..=some_chars_length);
+            let index = rng.random_range(0..=19);
             let rando_char = some_chars.chars().nth(index);
 
             if let Some(c) = rando_char {
                 filename.push(c);
             }
         }
-        format!("{filename}{file_extension}")
+        filename + &file_extension
     } else {
-        format!("track-output{file_extension}")
-    };
-
-    Ok(filename)
+        "track-output".to_string() + &file_extension
+    }
 }
 
 /// I/O operations for songs

src/token.rs

@@ -2,7 +2,7 @@ use std::default::Default;
 
 use serde::{Deserialize, Serialize};
 
-#[derive(Clone, Debug, Default, Deserialize, Serialize)]
+#[derive(Clone, Debug, Deserialize, Serialize)]
 pub struct Token {
     pub scope: String,
     pub expiration: i64,
@@ -27,16 +27,23 @@ pub struct AccessToken {
     pub message: String,
 }
 
-impl AccessToken {
-    /// Get the token fit for Bearer authentication
-    pub fn bearer_token(&self) -> String {
-        format!("Bearer {}", self.token)
+impl Default for Token {
+    fn default() -> Self {
+        Token {
+            scope: String::new(),
+            expiration: -1,
+            audience: String::new(),
+            issuer: String::new(),
+            issued: -1,
+        }
     }
+}
 
-    pub fn token_expired(&self) -> bool {
-        let current_time = time::OffsetDateTime::now_utc();
-        let expired = time::OffsetDateTime::from_unix_timestamp(self.expiration).unwrap();
-        current_time > expired
+impl AccessToken {
+    pub fn bearer_token(&self) -> String {
+        let mut token: String = String::from("Bearer ");
+        token += &self.token.clone();
+        token
     }
 }
 
@@ -45,10 +52,9 @@ impl Token {
         serde_json::to_string_pretty(&self)
     }
 
+    // TODO: Implement
     pub fn token_expired(&self) -> bool {
-        let current_time = time::OffsetDateTime::now_utc();
-        let expired = time::OffsetDateTime::from_unix_timestamp(self.expiration).unwrap();
-        current_time > expired
+        false
     }
 
     pub fn contains_scope(&self, des_scope: &String) -> bool {

src/types.rs

@@ -1,16 +1,6 @@
-#[derive(Debug)]
-pub enum MusicType {
+pub enum MusicTypes {
     DefaultMusicExtension,
     WavExtension,
     FlacExtension,
     MPThreeExtension,
-    None,
-}
-
-#[derive(Debug)]
-pub enum CoverArtType {
-    PngExtension,
-    JpegExtension,
-    JpgExtension,
-    None,
 }

src/util/mod.rs

@@ -1,19 +0,0 @@
-pub fn concatenate_path(
-    directory: &str,
-    filename: &str,
-    last_index: usize,
-) -> Result<String, std::io::Error> {
-    if let Some(character) = directory.chars().nth(last_index) {
-        let buffer: String = if character != '/' {
-            format!("{directory}/")
-        } else {
-            String::from(directory)
-        };
-
-        Ok(format!("{buffer}{filename}"))
-    } else {
-        Err(std::io::Error::other(
-            crate::constants::error::LAST_CHARACTER_IN_DIRECTORY,
-        ))
-    }
-}

tests/tests.rs

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