tsk-50: Changes and code formatting:

.gitea/workflows/tag_release.yml

@@ -10,7 +10,7 @@ on:
 
 jobs:
   release:
-    runs-on: debian-13
+    runs-on: ubuntu-24.04
     steps:
       - name: Checkout code
         uses: actions/checkout@v5
@@ -20,7 +20,7 @@ jobs:
       - name: Install Rust
         uses: actions-rs/toolchain@v1
         with:
-          toolchain: 1.94
+          toolchain: 1.90.0
           components: cargo
 
       - name: Extract Version from Cargo.toml

.gitea/workflows/workflow.yml

@@ -11,12 +11,12 @@ on:
 jobs:
   check:
     name: Check
-    runs-on: debian-13
+    runs-on: ubuntu-24.04
     steps:
       - uses: actions/checkout@v5
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.94
+          toolchain: 1.90.0
       - run: |
           mkdir -p ~/.ssh
           echo "${{ secrets.MYREPO_TOKEN }}" > ~/.ssh/gitlab_deploy_key
@@ -28,13 +28,13 @@ jobs:
 
   test:
     name: Test Suite
-    runs-on: debian-13
+    runs-on: ubuntu-24.04
     needs: setup_ssh
     steps:
       - uses: actions/checkout@v5
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.94
+          toolchain: 1.90.0
       - run: |
           mkdir -p ~/.ssh
           echo "${{ secrets.MYREPO_TOKEN }}" > ~/.ssh/gitlab_deploy_key
@@ -46,13 +46,13 @@ jobs:
 
   fmt:
     name: Rustfmt
-    runs-on: debian-13
+    runs-on: ubuntu-24.04
     needs: setup_ssh
     steps:
       - uses: actions/checkout@v5
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.94
+          toolchain: 1.90.0
       - run: rustup component add rustfmt
       - run: |
           mkdir -p ~/.ssh
@@ -65,13 +65,13 @@ jobs:
 
   clippy:
     name: Clippy
-    runs-on: debian-13
+    runs-on: ubuntu-24.04
     needs: setup_ssh
     steps:
       - uses: actions/checkout@v5
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.94
+          toolchain: 1.90.0
       - run: rustup component add clippy
       - run: |
           mkdir -p ~/.ssh
@@ -84,13 +84,13 @@ jobs:
 
   build:
     name: build
-    runs-on: debian-13
+    runs-on: ubuntu-24.04
     needs: setup_ssh
     steps:
       - uses: actions/checkout@v5
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: 1.94
+          toolchain: 1.90.0
       - run: |
           mkdir -p ~/.ssh
           echo "${{ secrets.MYREPO_TOKEN }}" > ~/.ssh/gitlab_deploy_key

Cargo.lock

@@ -104,7 +104,7 @@ dependencies = [
 
 [[package]]
 name = "icarus_meta"
-version = "0.4.5"
+version = "0.4.4"
 dependencies = [
  "imghdr",
  "infer",

Cargo.toml

@@ -1,8 +1,8 @@
 [package]
 name = "icarus_meta"
-version = "0.4.5"
+version = "0.4.4"
 edition = "2024"
-rust-version = "1.94"
+rust-version = "1.90"
 
 [dependencies]
 lofty = { version = "0.22.4" }

src/properties/mod.rs

@@ -43,25 +43,16 @@ mod tests {
         match test_util::util::file_exists(&dir, &filename) {
             Ok(_) => {
                 let filepath = test_util::util::get_full_path(&dir, &filename).unwrap();
-                let new_filepath = test_util::util::generate_newfilepath(&dir).unwrap();
+                match super::get_duration(&filepath) {
+                    Ok(duration) => {
+                        let song_duration: u64 = 41;
+                        let fetched_song_duration = duration.as_secs();
 
-                match test_util::util::copy_file(&filepath, &new_filepath) {
+                        assert_eq!(
-                    Ok(_) => match super::get_duration(&new_filepath) {
+                            song_duration, fetched_song_duration,
-                        Ok(duration) => {
+                            "Durations should match, but they don't {song_duration} {fetched_song_duration} ({duration:?})"
-                            test_util::util::remove_file(&new_filepath).unwrap();
+                        );
-
+                    }
-                            let song_duration: u64 = 41;
-                            let fetched_song_duration = duration.as_secs();
-
-                            assert_eq!(
-                                song_duration, fetched_song_duration,
-                                "Durations should match, but they don't {song_duration} {fetched_song_duration} ({duration:?})"
-                            );
-                        }
-                        Err(err) => {
-                            assert!(false, "Error: {err:?}");
-                        }
-                    },
                     Err(err) => {
                         assert!(false, "Error: {err:?}");
                     }
@@ -84,10 +75,8 @@ mod tests {
                 let new_filepath = test_util::util::generate_newfilepath(&dir).unwrap();
 
                 match test_util::util::copy_file(&filepath, &new_filepath) {
-                    Ok(_) => match super::get_song_properties(&new_filepath) {
+                    Ok(_) => match super::get_song_properties(&filepath) {
                         Ok(song_properties) => {
-                            test_util::util::remove_file(&new_filepath).unwrap();
-
                             let song_duration: u64 = 41;
                             let bitrate: u32 = 1;
                             let overall_bitrate: u32 = 3;