go-chi/chi

Refactor throttle middleware tests as prone to intermittent failure

Aberta

#608 aberto em 25 de mar. de 2021

Ver no GitHubGuia do colaborador
 (4 comentários) (0 reação) (0 responsável)Go (555 forks)batch import
help wanted

Métricas do repositório

Stars
 (8.127 estrelas)
Métricas de merge de PR
 (Mesclagem média 16h 36m) (2 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Examine o arquivo de teste do middleware de throttle em middleware/throttle test.go para identificar asserções dependentes de tempo. Considere implementar uma abordagem de teste determinística, como usar um relógio simulado ou sincronização baseada em canais, para eliminar falhas intermitentes. Revise os comentários existentes e as falhas de CI vinculadas para identificar padrões.
Pilha de tecnologia
go
Domain
backend
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Go
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75