facebook / facebook/TestSlide

Validate if mock_callable() target has left over references

Ouverte
#58 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
enhancement help wanted mock(_async)_callable
Langage dominant
Python
Étoiles
149
Forks
62
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

mock_constructor has [validation logic](https://github.com/facebookincubator/TestSlide/blob/master/testslide/mock_constructor.py#L275-L295) 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](https://github.com/facebookincubator/TestSlide/blob/master/testslide/mock_callable.py#L488).

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.