From 9b1222ebf705093c7069974eff562f18948c31dc Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Tue, 4 Aug 2026 12:33:52 +0000
Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=AA=20[LocalMusicFolderIdentity=20edge?=
 =?UTF-8?q?=20case=20testing]?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

🎯 What: Missing edge case tests for LocalMusicFolderIdentity.normalizePath were added.
📊 Coverage: Added scenarios for multiple slashes, mixed backslashes, and blank segments.
✨ Result: Reliable assertions preventing future path parsing regressions.

Co-authored-by: SayanthRock <202829406+SayanthRock@users.noreply.github.com>
---
 .../data/local/LocalMusicFolderIdentityTest.kt   | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/app/src/test/java/com/rockmusic/app/data/local/LocalMusicFolderIdentityTest.kt b/app/src/test/java/com/rockmusic/app/data/local/LocalMusicFolderIdentityTest.kt
index ad38b5a..cef1bef 100644
--- a/app/src/test/java/com/rockmusic/app/data/local/LocalMusicFolderIdentityTest.kt
+++ b/app/src/test/java/com/rockmusic/app/data/local/LocalMusicFolderIdentityTest.kt
@@ -28,4 +28,20 @@ class LocalMusicFolderIdentityTest {
     fun `keeps a top-level folder name`() {
         assertEquals("Music", LocalMusicFolderIdentity.displayName("Music/"))
     }
+
+    @Test
+    fun `normalizes edge cases with complex separators`() {
+        // multiple slashes
+        assertEquals("foo/bar", LocalMusicFolderIdentity.displayPath("foo///bar"))
+        // mixed slashes and backslashes
+        assertEquals("foo/bar", LocalMusicFolderIdentity.displayPath("foo\\//bar"))
+        // multiple backslashes
+        assertEquals("foo/bar", LocalMusicFolderIdentity.displayPath("foo\\\\\\bar"))
+        // leading and trailing mixed separators
+        assertEquals("foo", LocalMusicFolderIdentity.displayPath("\\/foo/\\"))
+        // only separators
+        assertEquals("Internal storage", LocalMusicFolderIdentity.displayPath("///\\\\\\"))
+        // spaces as segments (note: filter(String::isNotBlank) drops these)
+        assertEquals("foo/bar", LocalMusicFolderIdentity.displayPath("foo/ /bar/   /"))
+    }
 }