Uncovered pre-existing branches in the Redis backend

Abierto Apto para principiantes
#52 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
Medio día
Aptitud para principiantes
84/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python, redis
Área
backend, databases, testing

Línea de trabajo

Comienza en threadmill/backends/redis.py en las líneas 147-148, 445 y 494-495; después, inspecciona las pruebas existentes del backend de Redis. Ejecuta uv run pytest para confirmar las ramas no cubiertas y añade cobertura directa para la ausencia de REDIS_URL, un id de resultado desconocido y telemetría malformada; se considera terminado cuando las pruebas pasan sin que esas líneas sigan sin cobertura.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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 — the ValueError raised when REDIS_URL is missing from the backend params.
  • L445 — raise TaskResultDoesNotExist(...) in get_result().
  • L494-495 — the except ValueError: continue that drops malformed pub/sub payloads in worker_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.

Lenguaje dominante
Python
Estrellas
12
Forks
1
Merge medio
1 d 1 h
PR fusionados (30 d)
10

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de codingjoe/threadmill

Todos los issues de codingjoe/threadmill

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.