got-feedBack/feedBack
1
0
Fork 0
mirror of https://github.com/got-feedBack/feedBack.git synced 2026-09-12 22:18:31 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

test: tighten the enrichment-reset fixture (CodeRabbit)

Browse Source 
Narrow the import guard to ImportError (not blind Exception, BLE001), and stop
clearing _caa_index_locks — it's guarded by _caa_index_locks_guard, so an
unlocked clear() would race a still-alive worker, and its per-release mutexes
carry no test state.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
byrongamatos
2026-07-11 11:28:30 +02:00
co-authored by Claude Opus 4.8
parent 9093230320
commit f0223ba007
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/conftest.py
+5 -2
View File
@@ -22,7 +22,7 @@ def _reset_enrichment_state():
""" """
try: try:
import enrichment import enrichment
except Exception: except ImportError:
yield yield
return return
enrichment._enrich_cancel.clear() enrichment._enrich_cancel.clear()
@@ -32,7 +32,10 @@ def _reset_enrichment_state():
"total": 0, "matched": 0, "current": None}) "total": 0, "matched": 0, "current": None})
enrichment._enrich_last_fetch = 0.0 enrichment._enrich_last_fetch = 0.0
enrichment._artist_alias_cache.clear() enrichment._artist_alias_cache.clear()
enrichment._caa_index_locks.clear() # _caa_index_locks is deliberately left alone: it's guarded by
# _caa_index_locks_guard, so clearing it here (unlocked) would race a
# still-alive worker thread, and its entries are stateless per-release
# mutexes that don't leak test state anyway.
yield yield