From f9a473d5acde6bed088c859d904870ddbbd22f82 Mon Sep 17 00:00:00 2001
From: Chris Cochrun <chris@cochrun.xyz>
Date: Tue, 10 Mar 2026 14:59:42 -0500
Subject: [PATCH] better test for updating song model

---
 src/core/songs.rs | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/src/core/songs.rs b/src/core/songs.rs
index 3093f2c..13f19eb 100644
--- a/src/core/songs.rs
+++ b/src/core/songs.rs
@@ -1194,7 +1194,7 @@ mod test {
     use crate::ui::text_svg::text_svg_generator_with_cache;
 
     use super::*;
-    use pretty_assertions::assert_eq;
+    use pretty_assertions::{assert_eq, assert_ne};
     use rayon::iter::{IntoParallelIterator, ParallelIterator};
     use resvg::usvg::fontdb;
 
@@ -1397,12 +1397,17 @@ You saved my soul"
         let cloned_song = song.clone();
         let mut song_model: Model<Song> = model().await;
         song_model.load_from_db(&mut db).await;
+        let test_song = song_model.get_item(2);
+        assert_ne!(test_song, Some(&cloned_song));
 
         match song_model.update_item(song, 2) {
-            Ok(()) => assert_eq!(
-                &cloned_song,
-                song_model.find(|s| s.id == 7).unwrap()
-            ),
+            Ok(()) => {
+                assert_eq!(
+                    &cloned_song,
+                    song_model.find(|s| s.id == 7).unwrap()
+                );
+                assert_eq!(Some(&cloned_song), song_model.get_item(2))
+            }
             Err(e) => assert!(false, "{e}"),
         }
     }