Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

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

Aperta Adatta ai principianti
#1,460 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
84/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
go
Ambito
testing-qa

Direzione di ricerca

Inizia in internal/e2e/skills_e2e_test.go alle righe 92 e 98 e confronta il ramo attach di TestSkillsE2E con l'uso esistente di r.waitFor per la riga del picker. Esegui go test -tags e2e -run TestSkillsE2E ./internal/e2e/ con una risposta attach ritardata. È completato quando il test attende il chip allegato o il rifiuto denominato, attende l'editor svuotato e in quello scenario non ci sono sleep fissi dopo l'attach.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.
Lingua principale
Go
Stelle
115
Fork
14
Merge medio
9h 38m
PR unite (30g)
730

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Agent-Field/CodeAF

Tutte le issue di Agent-Field/CodeAF

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.