Uncovered pre-existing branches in the Redis backend
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- Mezza giornata
- Idoneità per principianti
- 84/100
Direzione di ricerca
Inizia in threadmill/backends/redis.py alle righe 147-148, 445 e 494-495, quindi esamina i test esistenti del backend Redis. Esegui uv run pytest per confermare i rami non coperti e aggiungi una copertura diretta per REDIS_URL mancante, un id del risultato sconosciuto e telemetria malformata; il lavoro è completato quando i test passano senza che queste righe rimangano non coperte.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Found by testJoe while adding coverage for the fair multi-queue rotation (codingjoe-fair-multi-queue-scheduling), and confirmed pre-existing (untouched by that diff):
defer: pre-existing uncovered lines threadmill/backends/redis.py L147-148, L445, L494-495. Unrelated to this diff. [threadmill/backends/redis.py]
The four lines are:
L147-148— theValueErrorraised whenREDIS_URLis missing from the backend params.L445—raise TaskResultDoesNotExist(...)inget_result().L494-495— theexcept ValueError: continuethat drops malformed pub/sub payloads inworker_telemetry().
None are reachable from the current test suite, so uv run pytest reports them as missing. Add direct tests (a backend without REDIS_URL, an unknown result id, a malformed telemetry message) or delete the branches that no caller can reach.
- Lingua principale
- Python
- Stelle
- 12
- Fork
- 1
- Merge medio
- 19h 11m
- PR unite (30g)
- 12
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di codingjoe/threadmill
-
WorkerProcess passes None poll intervals to the backend, breaking acquire in the consumer thread Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
codingjoe/threadmill#53 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
codingjoe/threadmill#51 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
codingjoe/threadmill#47 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
codingjoe/threadmill#55 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
codingjoe/threadmill#54 ·
Tutte le issue di codingjoe/threadmill
Issue simili
-
agent-ready documentation needs-triage
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
-
instance instance add
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
searxng/searx-instances#939 · 1 commento ·
-
area-deployment area-integrations triage:bot-seen
Difficoltà 2/5 Mezza giornata Idoneità per principianti 86/100