Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

The tagged skills test sleeps through attachment instead of waiting for the chip

Abierto Apto para principiantes
  • ¿Algo no está bien?
#1,460 0 comentarios 0 reacciones 0 asignados Ver en GitHub

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
84/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
go
Área
testing-qa

Línea de trabajo

Comienza en internal/e2e/skills_e2e_test.go en las líneas 92 y 98, y compara la rama de attach de TestSkillsE2E con su uso existente de r.waitFor para la fila del picker. Ejecuta go test -tags e2e -run TestSkillsE2E ./internal/e2e/ con una respuesta de attach retrasada. Se considera terminado cuando la prueba espera al chip adjunto o al rechazo con nombre, espera al editor vaciado y no hay esperas fijas después del attach en ese escenario.

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

Descripción

area:tests bug hygiene sev:papercut

Found on santos/dev2 at 008363c98 (#1410). It reaches dev when #1410 merges.

What happened

The tagged skills e2e waits 700 ms after Enter and another 500 ms after clearing the box. Neither pause proves that the skill chip appeared. On a slow machine the test can type its next question before attachment settles; on a fast one it wastes time. No failure from this timing was observed.

Replication

Deterministic (no model). Read the attach arm of TestSkillsE2E: after r.keys("Enter") it uses fixed sleeps, then checks only for a refusal and proceeds. A delayed fake attach response longer than 700 ms makes the check run before the chip appears.

Where

internal/e2e/skills_e2e_test.go:92 and :98 contain the fixed sleeps at 008363c98; the same test already uses r.waitFor to wait for the picker row.

The fix

Wait for the attached chip or a named refusal, then wait for the cleared editor state before typing the question. Use the suite's bounded screen wait.

Acceptance

  • e2e: go test -tags e2e -run TestSkillsE2E ./internal/e2e/ asserts the chip before its next message and passes with a delayed attach response.
  • Unit: no fixed post-attach sleeps remain in that scenario.
  • The test change entry's invalidates names the former timing assumption.
Lenguaje dominante
Go
Estrellas
115
Forks
14
Merge medio
9 h 35 min
PR fusionados (30 d)
752

Preparar el entorno

Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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 Agent-Field/CodeAF

Todos los issues de Agent-Field/CodeAF

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.