9ff2e7084a
Persist organize_by_playlist on mirrored playlists and run playlist-folder downloads from the auto-sync pipeline instead of the global wishlist phase. Register SoulSync library rows after playlist-folder post-processing, route failed organize batches to the wishlist correctly, and skip sync-time unmatched wishlist only when organize download handles retries. Invalidate stale playlist track caches on refresh (Spotify and Deezer ARL), re-mirror on refetch, and improve standalone playlist modals (re-analysis, Open in Mirrored). Add filesystem missing-track detection and tests. Co-authored-by: Cursor <cursoragent@cursor.com>
88 lines
3.1 KiB
Python
88 lines
3.1 KiB
Python
"""Tests for playlist-folder existence detection."""
|
|
|
|
import os
|
|
from unittest.mock import patch
|
|
|
|
import pytest
|
|
|
|
from core.downloads.playlist_folder import (
|
|
candidate_playlist_folder_paths,
|
|
resolve_playlist_folder_mode_for_batch,
|
|
track_exists_in_playlist_folder,
|
|
)
|
|
|
|
|
|
class _FakeDB:
|
|
def __init__(self, mirrored=None):
|
|
self._mirrored = mirrored
|
|
|
|
def resolve_mirrored_playlist(self, playlist_ref, profile_id=1, default_source='spotify'):
|
|
return self._mirrored
|
|
|
|
|
|
def test_track_exists_in_playlist_folder_finds_file(tmp_path):
|
|
playlist_dir = tmp_path / 'My Playlist'
|
|
playlist_dir.mkdir()
|
|
track_file = playlist_dir / 'Artist A - Song One.flac'
|
|
track_file.write_bytes(b'x')
|
|
|
|
with patch('core.downloads.playlist_folder._get_config_manager') as cfg:
|
|
cfg.return_value.get.return_value = str(tmp_path)
|
|
with patch('core.downloads.playlist_folder.docker_resolve_path', side_effect=lambda p: p):
|
|
with patch(
|
|
'core.downloads.playlist_folder.get_file_path_from_template',
|
|
return_value=('', ''),
|
|
):
|
|
assert track_exists_in_playlist_folder('My Playlist', 'Artist A', 'Song One')
|
|
|
|
|
|
def test_track_exists_in_playlist_folder_missing(tmp_path):
|
|
with patch('core.downloads.playlist_folder._get_config_manager') as cfg:
|
|
cfg.return_value.get.return_value = str(tmp_path)
|
|
with patch('core.downloads.playlist_folder.docker_resolve_path', side_effect=lambda p: p):
|
|
with patch(
|
|
'core.downloads.playlist_folder.get_file_path_from_template',
|
|
return_value=('', ''),
|
|
):
|
|
assert not track_exists_in_playlist_folder('My Playlist', 'Artist A', 'Song One')
|
|
|
|
|
|
def test_candidate_paths_template_layout(tmp_path):
|
|
with patch('core.downloads.playlist_folder._get_config_manager') as cfg:
|
|
cfg.return_value.get.return_value = str(tmp_path)
|
|
with patch('core.downloads.playlist_folder.docker_resolve_path', side_effect=lambda p: p):
|
|
with patch(
|
|
'core.downloads.playlist_folder.get_file_path_from_template',
|
|
return_value=('Cool Mix', 'Artist - Title'),
|
|
):
|
|
paths = candidate_playlist_folder_paths('Cool Mix', 'Artist', 'Title')
|
|
assert any(p.endswith('.flac') for p in paths)
|
|
assert all('Cool Mix' in p for p in paths)
|
|
|
|
|
|
def test_resolve_playlist_folder_mode_from_mirrored():
|
|
db = _FakeDB(mirrored={
|
|
'id': 5,
|
|
'name': 'Rekordbox Set',
|
|
'organize_by_playlist': True,
|
|
})
|
|
enabled, name = resolve_playlist_folder_mode_for_batch(
|
|
db,
|
|
playlist_id='37i9dQZF1',
|
|
playlist_name='Other Name',
|
|
batch_playlist_folder_mode=False,
|
|
)
|
|
assert enabled is True
|
|
assert name == 'Rekordbox Set'
|
|
|
|
|
|
def test_resolve_playlist_folder_mode_batch_flag():
|
|
db = _FakeDB()
|
|
enabled, name = resolve_playlist_folder_mode_for_batch(
|
|
db,
|
|
playlist_id='1',
|
|
playlist_name='Batch Name',
|
|
batch_playlist_folder_mode=True,
|
|
)
|
|
assert enabled is True
|
|
assert name == 'Batch Name'
|