Mise à jour de la dépendance factory boy #517
Conversation
| @@ -18,7 +19,7 @@ def _generate_valid_phone(): | |||
| does not consider valid which makes tests to randomly fail. This | |||
| function always generate a valid french phone number for PhoneNumberField""" | |||
| for try_attempt in range(10): | |||
| phone = factory.Faker("phone_number", locale="fr_FR").generate() | |||
| phone = Faker("phone_number").evaluate(None, None, {"locale": "fr_FR"}) | |||
There was a problem hiding this comment.
Celui-ci est particulièrement relou. La nouvelle méthode a une signature probablement plus logique pour son utilisation en interne mais pas pour son utilisation publique.
There was a problem hiding this comment.
Je ne sais pas si je suis très rassurée par les breaking changes dans factory mais à la limite je peux m'en accommoder en faisant des copier-coller, comme d'habitude.
Ce qui m'embête davantage c'est le mélange entre cette PR et #516, ainsi que la modif sur flake8 dont je ne sais pas d'où elle sort. En l'état je ne suis pas sereine pour merge cette PR.
Tu peux commencer par rebase cette branche sur celle de #516 pour bien séparer les sujets. Pour le sujet flake8, selon si c'est nécessaire pour factory boy ou pas on le laisse ici ou pas, mais là comme je ne comprends pas et je ne suis pas rassurée.
christophehenry
force-pushed
the
bump-factory-boy
branch
from
e7e99d4
to
97a0ec1
Compare
|
C'est fait. À la base, j'avais prévu de mettre à jour qu'une poignée de dépendances majeures de test mais celle de |
|
Ok, y'a un problème, ici : les tests disent que tout passe sur la CI mais moi, ils sont cassés en local. Quelqu'un d'autre peut confirmer ? @tut-tuuut, @mrjmad, @liviaribeiro ? Edit : trouvé. J'était sur une mauvaise version d'une autre dépendance. |
christophehenry
force-pushed
the
bump-factory-boy
branch
from
97a0ec1
to
771a7e3
Compare
🌮 Objectif
Mise à jour tricky. Ça casse des trucs. En particulier des chemins d'imports qui changent. Du coup, j'ai décider de faire sauter l'import global de
factoryet importer au cas par cas.