linkedin/cruise-control

Missing EasyMock.verify in unit tests

Open

#1408 aperta il 3 dic 2020

Vedi su GitHub
 (8 commenti) (0 reazioni) (0 assegnatari)Java (649 fork)auto 404
good first issueproductivity

Metriche repository

Star
 (3036 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

In some unit tests, before resetting mocked objects or after mocked objects get used, EasyMock.verify does not get invoked on mocked objects. So, we should add it.

For example, in UserTaskManagerTest.java and KafkaCruiseControlServletEndpointTest.java. There could be more cases. So, I suggest thoroughly iterate all cases where EasyMock gets used.

It may require re-writing some unit tests since naively inserting verify before resetting mocks or at the end of unit tests fail in most cases.

Guida contributor

Direzione di ricerca
Ispeziona tutti i file di test unitari che utilizzano EasyMock. Identifica gli oggetti mock in cui manca verify. Aggiungi chiamate a EasyMock.verify prima di resettare i mock o alla fine dei test. Assicurati che tutti i test continuino a passare dopo le modifiche.
Tech stack
java
Dominio
testing qa
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
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
JavaEasyMockJUnit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45