cssinjs/jss

Throw Error instead of warning when running tests

Open

#891 aperta il 14 ott 2018

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (434 fork)batch import
help wantedideaimportant

Metriche repository

Star
 (7000 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I see in the logs many warnings, which I think shouldn't be there, which means we are ignoring problems without noticing in the CI.

I am wondering if we should replace the warning when running tests with a function that throws errors so that our tests can't ignore them.

As a bonus we won't ned to rewire in so many places, but instead use expect for catching.

Guida contributor

Direzione di ricerca
Investigare il meccanismo di avviso utilizzato nei test, identificare tutte le istanze in cui vengono chiamati gli avvisi e sostituirli con un pattern di lancio, assicurandosi che i test che si aspettano avvisi utilizzino try/catch.
Tech stack
javascript
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65