envoyproxy/envoy

Split FM tests out of conn_manager_impl_test

Open

#13.278 aperta il 25 set 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)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

While the FilterManager code has been split out of the HCM, its still just being tested via the HCM tests. To reduce the size of the HCM tests and set the FM up for use cases outside the HCM, we should move tests that make sense into a FM specific set of tests.

Guida contributor

Direzione di ricerca
Esamina l'attuale conn manager impl test.cc per identificare i test relativi a FilterManager. Comprendi l'interfaccia di FilterManager e il suo attuale utilizzo nei test HCM. Pianifica un nuovo file di test filter manager impl test.cc e sposta i test pertinenti, assicurandoti che compilino e superino i test.
Tech stack
c
Dominio
backend
Tipo issue
Test
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++GitTesting frameworks (e.g., Google Test)
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55