go-chi/chi

Refactor throttle middleware tests as prone to intermittent failure

Aperta

#608 aperta il 25 mar 2021

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)Go (555 fork)batch import
help wanted

Metriche repository

Star
 (8127 stelle)
Metriche merge PR
 (Merge medio 16h 36m) (2 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina il file di test del middleware di throttling in middleware/throttle test.go per identificare le asserzioni dipendenti dal tempo. Considera l'implementazione di un approccio di test deterministico, come l'uso di un orologio simulato o di una sincronizzazione basata su canali, per eliminare i fallimenti intermittenti. Rivedi i commenti esistenti e i fallimenti CI collegati per individuare pattern.
Tech stack
go
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75