From fdda64963fabb35a32a4673f1e47eb22936b7633 Mon Sep 17 00:00:00 2001 From: Broque Thomas <26755000+Nezreka@users.noreply.github.com> Date: Wed, 13 May 2026 11:52:04 -0700 Subject: [PATCH] Drop platform-biased trailing-backslash test for derive_artist_folder POSIX os.path.dirname doesn't treat '\' as separator, so the assertion 'Drake' in result fails on Linux CI even though the function's rstrip removes the trailing backslash correctly. The forward-slash test already covers the trim contract. --- tests/library/test_artist_image.py | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/tests/library/test_artist_image.py b/tests/library/test_artist_image.py index cfba9431..4bf3d5cf 100644 --- a/tests/library/test_artist_image.py +++ b/tests/library/test_artist_image.py @@ -37,16 +37,6 @@ class TestDeriveArtistFolder: from core.library.artist_image import derive_artist_folder assert derive_artist_folder("/music/Drake/Views/") == "/music/Drake" - def test_handles_trailing_backslash(self): - """Windows path with trailing separator.""" - from core.library.artist_image import derive_artist_folder - result = derive_artist_folder(r"H:\Music\Drake\Views\\") - # On Windows the trim leaves `H:\Music\Drake\Views`, - # dirname returns `H:\Music\Drake`. On POSIX dirname might - # treat backslashes as filename chars. Just verify the - # trailing separator was stripped. - assert "Drake" in result - def test_empty_string_returns_empty(self): from core.library.artist_image import derive_artist_folder assert derive_artist_folder("") == ""