Interrupt tests rely on wall-clock sleeps
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 75/100
Línea de trabajo
Empieza por las esperas de reloj en main_e2e_test.go líneas 821-838 (1 s antes de la interrupción) y en main_signal_unix_test.go línea 27 (200 ms antes de SIGTERM). Revisa cómo se usa el punto de extensión signalReady existente y si el escaneo puede emitir una línea de resultado en la que puedas esperar. Está terminado cuando ninguna de las dos pruebas usa time.Sleep para decidir cuándo se dispara la interrupción, y go test -run 'Interrupt|Signal' -count=20 . pasa de forma repetida.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
Two CLI tests time their interrupt with a fixed sleep. They expect it to land in the middle of a scan, which a slow runner can miss.
Where
main_e2e_test.go:821-838: sleeps 1 s, then interrupts a scan expected to take about 2 smain_signal_unix_test.go:27: sleeps 200 ms before SIGTERM
Impact
On a slow or loaded runner the scan finishes before the interrupt arrives, and got >= len(want) fails. This is the same class as the recent flake fixes.
Suggested fix
Trigger the interrupt from an event (for example after the Nth result line on stdout, or through the existing signalReady seam) instead of a wall-clock sleep.
Acceptance criteria
- No
time.Sleepdecides when the interrupt fires in either test.
Getting started
- Relevant files:
main_e2e_test.go,main_signal_unix_test.go - How to run tests:
go test -run 'Interrupt|Signal' -count=20 . - Related docs:
docs/DEVELOPER_GUIDE.md
Filed by repo-audit/full-audit at 3f671b2. Severity: low. Effort: S. audit-key: testing|main_e2e_test.go|interrupt-sleep
- Lenguaje dominante
- Go
- Estrellas
- 1
- Forks
- 1
- Merge medio
- 7 h 48 min
- PR fusionados (30 d)
- 6
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- 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 TMHSDigital/subenum
-
enhancement good first issue severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
TMHSDigital/subenum#275 ·
Los mantenedores suelen responder en 1 día
-
community documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
TMHSDigital/subenum#135 ·
Los mantenedores suelen responder en 1 día
-
severity: medium tech-debt
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
TMHSDigital/subenum#277 ·
Los mantenedores suelen responder en 1 día
-
enhancement help wanted severity: medium
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
TMHSDigital/subenum#274 ·
Los mantenedores suelen responder en 1 día
-
enhancement help wanted severity: medium
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
TMHSDigital/subenum#273 ·
Los mantenedores suelen responder en 1 día
Todos los issues de TMHSDigital/subenum
Issues similares
-
area: interop area: sip-core v3
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
fonoster/routr#705 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
deadfin-org/deadfin#35 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
-
enhancement priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
labmonkeys-space/nl6#1037 ·
Los mantenedores suelen responder en 1 día
-
Setup Guide Go tab: say it needs Go 1.26 (the unversioned go get pulls OpenTelemetry Go v1.47.0)Abiertoenhancement follow-up
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día