envoyproxy/envoy

Reduce boilerplate and enforce InSequence in RouterTest

Open

#4.916 aberto em 30 de out. de 2018

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
help wantedtech debt

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

The tests in RouterTest (test/common/router/router_test.cc) are very verbose and hard to work with. We should 1) reduce the amount of boilerplate necessary and 2) use InSequence in every test.

Guia do colaborador

Direção de pesquisa
Familiarize se com a configuração existente do RouterTest, estude como o InSequence é usado em outros testes e identifique padrões de código repetitivo que podem ser abstraídos ou removidos.
Pilha de tecnologia
cpp
Domain
testing
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Google TestBazel
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65