keras-team/keras-preprocessing

Adding tests for Exception error messages

Aperta

#31 aperta il 2 ago 2018

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Python (453 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (1028 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

If any contributor is feeling up for it, it would be good to add unit tests that check that appropriate error messages are getting raised in various situations not yet covered. It seems that a few error messages were previously incorrectly formatted, because we don't have unit tests for some of these exceptions.

We have some such unit tests already, which look like this:

with pytest.raises(ValueError) as e_info:
      generator.flow((images, x_misc_err), np.arange(dsize), batch_size=3)
      assert 'All of the arrays in' in str(e_info.value)

Guida contributor

Direzione di ricerca
Esamina i file di test esistenti nel repository che controllano i messaggi di errore utilizzando pytest.raises. Identifica le classi di eccezione che non hanno tali test. Scrivi nuovi casi di test seguendo lo stesso schema, assicurando che ogni messaggio di eccezione sia asserito.
Tech stack
python
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
Pythonpytest
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80