Three uncovered pre-existing spots on the monitoring and dashboard surface
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
- 64/100
Línea de trabajo
Start with the existing tests for services/email/dashboard/context_processors.py and the reputation dashboard views, then run the provided pytest command with coverage. Add a test for the context processor's anonymous, no-org path and rendering assertions for the cited fbl_report_detail.html and fbl_report_list.html spots; done when those behaviors are covered by assertions, since template coverage percentages are unreliable here.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Three pre-existing lines on the dashboard and reputation surfaces have no test.
Work reference: branch codingjoe-weekly-digest, commits 2ace45d2 to 6f28eacf, weekly sending digest. This work is from the superJoe crew. testJoe measured the two apps for that change, and these lines were already dark.
side quest:services/email/dashboard/context_processors.py:8return {}when no org is selected is uncovered. Pre-existing, not in this diff. [services/email/dashboard/context_processors.py]
side quest:reputation/fbl_report_detail.html lines 43, 51-54, 72-77, 81-88 are uncovered. Pre-existing, not in this diff. [services/email/reputation/templates/reputation/fbl_report_detail.html]
side quest:reputation/fbl_report_list.html line 68 is uncovered. Pre-existing, not in this diff. [services/email/reputation/templates/reputation/fbl_report_list.html]
Measured with uv run --group test pytest services/email/reputation services/email/dashboard --cov=services/email/reputation --cov=services/email/dashboard --cov-report=term-missing: 216 passed, 87 percent. Template line counts read 0 percent for every template in that run, including ones that a rendering test provably renders. The fbl template lines therefore want a rendering assertion, not a coverage number. The context processor line is the anonymous, no-org path.
Related: #329, #299, #292, #296, #297.
- Lenguaje dominante
- Python
- Estrellas
- 4
- Forks
- 0
- Merge medio
- 2 h 40 min
- PR fusionados (30 d)
- 163
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin 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.
Más de codingjoe/relay
-
enhancement real sec
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
codingjoe/relay#410 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug side quest
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
bug side quest
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
bug side quest
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
side quest test
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
Todos los issues de codingjoe/relay
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
MIT-LCP/wfdb-python#590 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
pollen-robotics/reachy_mini#1517 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
NRLMMD-GEOIPS/pluginify#41 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
scverse/spatialdata#1293 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
UKGovernmentBEIS/inspect_evals#2788 ·
Los mantenedores suelen responder en 1 día