test : add unit tests for health_check and root endpoint functions
@tmdeveloper007 ya está trabajando en esto.
Desde el 3/8/2026.
- #2278 de @tmdeveloper007 — cerrado sin fusionar
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Summary of What Needs to be Done:
Add unit tests for the health_check and root endpoint functions in backend/secuscan/main.py. These are FastAPI route handlers that can be tested in isolation by constructing a minimal FastAPI app or by testing them as async functions with mocked dependencies.
Changes that Need to be Made:
Create testing/backend/unit/test_main_endpoints.py (standalone unit tests) covering:
-
health_check():
- Returns a JSONResponse with status field
- Returns 200 status code
- Response body contains expected fields (e.g., status, version)
-
root():
- Returns an HTMLResponse
- Response has 200 status code
- Response body contains expected HTML content
Use FastAPI TestClient or construct minimal app state for testing. Mock any app-level dependencies that are not relevant to these endpoints. Run with --noconftest to avoid the full app initialization.
Import from backend.secuscan.main directly. Do NOT import the full app (which requires all routes to be loaded).
Impact that it would Provide:
- Reliability: Ensures health check and root endpoints return expected responses
- Testing: Adds coverage for main.py which currently has no unit tests
- Maintainability: Documents the expected API surface for these endpoints
Note: This task is being handled by tmdeveloper007 — please assign to that account when picking it up.
- Lenguaje dominante
- Python
- Estrellas
- 36
- Forks
- 234
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una 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.
Más de utksh1/SecuScan
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
area:frontend good first issue level:intermediate priority:low type:testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
-
area:frontend good first issue level:intermediate priority:low type:testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
area:frontend good first issue level:beginner priority:low type:testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
area:frontend good first issue level:intermediate priority:low type:testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Todos los issues de utksh1/SecuScan
Issues similares
-
[BUG] HTTP lifecycle dispatch treats a NO_PROXY-only environment as corporate egress and skips the resolve-and-pinned sessionPosiblemente ocupada @bodapatisaikrishna la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
[integration]: NoMacSandboxAbiertoexternal
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
langchain-ai/docs#7652 ·
Los mantenedores suelen responder en 1 día
-
[oblt-aw][autodoc] Document new e2e-secret-leak-artifacts workflow and update e2e-all leaf listAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
elastic/oblt-aw#2513 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug p2
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
amd/gaia#7326 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
NCAR/CLM-Tutorial#113 · 1 comentario ·
Los mantenedores suelen responder en 1 día