go-chi/chi

Refactor throttle middleware tests as prone to intermittent failure

Offen

#608 geöffnet am 25.03.2021

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (555 Forks)batch import
help wanted

Repository-Metriken

Stars
 (8.127 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 16h 36m) (2 gemergte PRs in 30 T)

Beschreibung

https://github.com/go-chi/chi/blob/master/middleware/throttle_test.go

It's often that when I submit a PR, and the code runs through github ci, it will fail -- https://github.com/go-chi/chi/actions

I have to re-run the jobs to have it pass, it seems to be a timing issue. When running locally, it seems to work well pretty consistently, but I've seen the rare failing case.

Contributor Guide

Research-Richtung
Untersuche die Throttle Middleware Testdatei unter middleware/throttle test.go, um zeitabhängige Assertions zu identifizieren. Erwäge die Implementierung eines deterministischen Testansatzes, wie z.B. die Verwendung einer Mock Uhr oder kanalbasierten Synchronisation, um intermittierende Fehler zu beseitigen. Überprüfe vorhandene Kommentare und verknüpfte CI Fehler auf Muster.
Tech Stack
go
Domain
backend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75