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

Trim slow retry from post-processing test

Browse source 
Keep the fuzzy-context post-processing test on the fast path by stubbing the retry sleep, matching the other retry-path coverage and avoiding a ~20s pause in the downloads test suite.
This commit is contained in:
Antti Kettunen 2026-04-28 21:27:01 +03:00
parent f75c180cb6
commit 7f3272f3ba
No known key found for this signature in database
GPG key ID: C6B2A3D250359BD7
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
tests/downloads/test_downloads_post_processing.py
View file

@ -323,7 +323,7 @@ def test_youtube_task_uses_get_download_status_to_resolve_path(monkeypatch):
assert any(c[0] == 'mark_completed' for c in rec.calls)
def test_fuzzy_context_matching_when_exact_key_missing():
def test_fuzzy_context_matching_when_exact_key_missing(monkeypatch):
"""When exact key isn't in matched_downloads_context, worker tries fuzzy match
constrained to same Soulseek username."""
download_tasks['t1'] = {
@ -339,6 +339,7 @@ def test_fuzzy_context_matching_when_exact_key_missing():
deps, rec = _build_deps(
find_completed_file=lambda *a, **kw: (None, None), # file not found
)
monkeypatch.setattr(pp.time, 'sleep', lambda s: None)
# Won't find file → marks failed. But the fuzzy match log path executes.
pp.run_post_processing_worker('t1', 'b1', deps)
assert download_tasks['t1']['status'] == 'failed'