Untested branches in telemetry, backend errors, and CLI error paths
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 72/100
Línea de trabajo
Start by locating the existing tests for the interfaces in threadmill/backends/base.py and threadmill/backends/redis.py, then inspect threadmill/management/commands/threadmill.py and its command tests. Add coverage for both worker_telemetry paths, the Redis error raises, the invalid-backend and unsupported-queue errors, and KeyboardInterrupt shutdown. Done means these listed branches are exercised and the relevant coverage remains complete.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Deferred from the task-prefetching review (crew run). All four items predate that branch and the branch does not touch them. The review reported them instead of adding tests.
ThreadmillTaskBackend.worker_telemetryhas no test, although every other interface method has one. [threadmill/backends/base.py]RedisTaskBackend.worker_telemetryhas no test for the pub/sub streaming loop and the invalid-payload skip. [threadmill/backends/redis.py]RedisTaskBackendhas no test for theValueErrorof a missingREDIS_URLor for theTaskResultDoesNotExistraise inget_result. [threadmill/backends/redis.py]WorkerCommandhas no test for the invalid-backend and unsupported-queueCommandErrorbranches or for theKeyboardInterruptshutdown path. [threadmill/management/commands/threadmill.py]
The patch of the branch has 100% line and branch coverage. These items are the remaining coverage gaps in the files that the branch touches.
- Lenguaje dominante
- Python
- Estrellas
- 21
- Forks
- 2
- Merge medio
- 12 h 38 min
- PR fusionados (30 d)
- 27
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Issues similares
-
Python Volcengine provider returns failures instead of raising themPosiblemente ocupada @Ricardo-M-L la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
agent-infra/sandbox#248 ·
-
Textractor leaves empty temporary files when remote retrieval failsPosiblemente ocupada @HuaTNA la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
test(bm_generation): pin the learning parameters and CPU sync shortcutPosiblemente ocupada @iloveeyjafjalla la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
qboson/kaiwu-pytorch-plugin#1341 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
elder-plinius/V3R1T4S#625 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
foxglove/mcap#1907 · 1 comentario ·
Los mantenedores suelen responder en 1 día