facebook/TestSlide

Add constructor input checks for matchers

Open

#233 aperta il 1 set 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Python (58 fork)auto 404
Hacktoberfestenhancement

Metriche repository

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

Descrizione

When we do:

AnyInstanceOf(not_a_type)

we end up raising an exception not at the constructor, but rather when the comparison is being made, and isinstanceof (at AnyInstanceOf's code) will complain that not_a_type is... not a type.

Let's revisit all matchers & ensure that all of them raise ValueError if the constructor arguments are invalid from the constructor itself.

CC @deathowl

Guida contributor

Direzione di ricerca
Ispeziona tutte le classi matcher nel codebase di TestSlide e aggiungi la validazione degli input nei loro metodi init per sollevare ValueError per argomenti non validi.
Tech stack
python
Dominio
testing
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70