envoyproxy/envoy

Reduce boilerplate and enforce InSequence in RouterTest

Open

#4916 aperta il 30 ott 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
help wantedtech debt

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Acquisisci familiarità con la configurazione esistente di RouterTest, studia come viene utilizzato InSequence in altri test e identifica i pattern boilerplate che possono essere astratti o rimossi.
Tech stack
cpp
Dominio
testing
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Google TestBazel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65