Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

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

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#1,460 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
84/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
go
Bereich
testing-qa

Rechercherichtung

Beginne in internal/e2e/skills_e2e_test.go bei den Zeilen 92 und 98 und vergleiche den Attach-Zweig von TestSkillsE2E mit seiner bestehenden Verwendung von r.waitFor für die Picker-Zeile. Führe go test -tags e2e -run TestSkillsE2E ./internal/e2e/ mit einer verzögerten Attach-Antwort aus. Erledigt ist die Aufgabe, wenn der Test auf den angehängten Chip oder die benannte Ablehnung wartet, auf den geleerten Editor wartet und es in diesem Szenario keine festen Sleeps nach dem Attach gibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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.
Vorherrschende Sprache
Go
Sterne
115
Forks
14
Ø Merge
15 Std. 56 Min.
Gemergte PRs (30 T.)
231

Entwicklungsumgebung

Die Einrichtungsdateien dieses Projekts haben wir noch nicht geprüft. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Agent-Field/CodeAF

Alle Issues in Agent-Field/CodeAF

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.