facebook/TestSlide

Suggest mock_(async)_callable from StrictMock

Open

#128 aperta il 16 mar 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (58 fork)auto 404
Better Error MessagesStrictMockenhancementgood first issuehelp wanted

Metriche repository

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

Descrizione

StrictMock raises when an attribute is accessed, but has not been previously defined

https://github.com/facebookincubator/TestSlide/blob/8d516487247b6591c5a310c3e069ddb32d822d5f/testslide/strict_mock.py#L70

The message simply mentions "setting" the attribute. Let's improve this message to, if the mock has a template, and the attribute is callable, suggest using mock_callable / mock_async_callable to set it.

Guida contributor

Direzione di ricerca
Esamina la classe StrictMock in testslide/strict mock.py, in particolare il metodo getattr alla riga 70, e modifica il messaggio di errore per verificare se l'attributo a cui si accede è chiamabile e se esiste un template, quindi suggerisci di usare mock callable o mock async callable.
Tech stack
python
Dominio
testing qa
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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