microsoft/Terminal

Validate that VT adapter tests cover all mock possibilities

Open

#987 aperta il 24 mag 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (3212 fork)batch import
Area-CodeHealthArea-VTHelp WantedIssue-TaskProduct-Conhost

Metriche repository

Star
 (35.764 star)
Metriche merge PR
 (Merge medio 27g 19h) (24 PR mergiate in 30 g)

Descrizione

All of the various mock definitions don't seem to be in use. When I stood up the original parser years ago and wrote the mock, every mock was checked somehow. Over time, it probably decayed and the only thing left is that the mock has to be implemented to satisfy compilation of the interface.

Originally posted by @miniksa in https://github.com/microsoft/terminal/pull/891/review_comment/create

The goal here would be to run through all the tests and ensure that the mocks are involved somehow in each operation to validate that we're getting what we expect back through the adapter interface for all potential results.

Guida contributor

Direzione di ricerca
Esamina la suite di test dell'adattatore VT (probabilmente in src/terminal/adapter/) per identificare tutte le classi mock. Per ogni mock, verifica se tutti i suoi metodi sono esercitati nei test esistenti. Scrivi test aggiuntivi per coprire le chiamate ai metodi mock mancanti. Rivedi eventuali modifiche recenti all'adattatore per assicurarti che anche i nuovi mock siano testati.
Tech stack
Nessuno
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Unit testing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50