test(adapters/linear): assert the deadline branch fires in TestRateLimitDeadlineReturnsTypedRateLimited
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
- 75/100
Rechercherichtung
Beginne mit TestRateLimitDeadlineReturnsTypedRateLimited in adapters/linear/harden_capability_test.go und mit der Deadline-Prüfung in adapters/linear/retry.go, und kläre, ob dieser Zweig ein beobachtbares Signal wie eine Log-Zeile "deadline" ausgibt. Deaktiviere den Zweig vorübergehend und führe den Test aus, um zu sehen, dass er trotzdem besteht. Fertig ist, wenn der Test prüft, dass der Zweig ausgelöst wurde, bei deaktiviertem Zweig fehlschlägt und ohne zeitliche Annahmen auskommt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Follow-up from #110 (an advisor suggestion, not a bug).
TestRateLimitDeadlineReturnsTypedRateLimited (adapters/linear/harden_capability_test.go) checks that a 429 whose backoff would end past the caller's deadline returns *linear.RateLimited. It does not check which branch produced that error. If a later retry-policy change sent the error out through a different check, the test would still pass.
Acceptance: the test also asserts that the deadline branch in adapters/linear/retry.go fired (for example, the "deadline" log line or an equivalent observable), and it fails when that branch is disabled. Keep the test free of timing assumptions, as #110 made it.
This gap existed before #110.
Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: medium
- Vorherrschende Sprache
- Go
- Sterne
- 2
- Forks
- 0
- Ø Merge
- 19 Std. 10 Min.
- Gemergte PRs (30 T.)
- 16
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus coder/chat
-
ready-for-agent
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
feat(adapters/msteams): land Microsoft Teams adapter as experimental (needs live-tenant validation)Offenenhancement ready-for-human
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
Maintainer antworten meist innerhalb von 1 Tag
Ähnliche Issues
-
fix(handlers): explicitly set Content-Type header in static Index handlerEvtl. vergeben @kage-zero hat das heute übernommen. Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
Quick Open: restore dependency-directory exclusions in filesystem fallbackEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
eugenioenko/ttt#1206 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
open-policy-agent/opa#9426 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
rorkai/App-Store-Connect-CLI#3192 ·
Maintainer antworten meist innerhalb von 1 Tag
-
t:urlDecodeUni does not map U+3002 and U+FF61 to a dot, unlike ModSecurityEvtl. vergeben @fzipi hat das heute übernommen. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag