danvics/soulsync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Drop platform-biased trailing-backslash test for derive_artist_folder

Browse source 
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.
This commit is contained in:
Broque Thomas 2026-05-13 11:52:04 -07:00
parent 89246a7304
commit fdda64963f
1 changed files with 0 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
tests/library/test_artist_image.py
View file

@ -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("") == ""