facebook/TestSlide

Validate if mock_callable() target has left over references

Open

#58 aperta il 25 ott 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (58 fork)auto 404
enhancementhelp wantedmock(_async)_callable

Metriche repository

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

Descrizione

mock_constructor has validation logic to find out if the class being mocked has other references that are not from the module where it lives. This is important, as there's no guarantees it is gonna work otherwise.

mock_callable has the same potential problem when mocking functions at modules for example: if there are left over references such as from module import func, it will not work.

Let's add such validation to mock_callable so users don't shoot themselves in the foot.

This logic can be added here.

Guida contributor

Direzione di ricerca
Studia la logica di validazione esistente in mock constructor.py (righe 275-295) per capire come rileva i riferimenti residui. Quindi, in mock callable.py, individua la posizione appropriata (riga 488) e implementa un controllo simile per la funzione o modulo target, assicurandoti di cogliere casi come 'from module import func'.
Tech stack
python
Dominio
testing qa
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40