Development #68527
Utiliser la lib responses dans les tests
0%
Description
https://github.com/getsentry/responses.
En remplaçant les mock.patch('wcs.qommon.misc.urlopen') et apparentés par des with responses.RequestsMock() as rsps: (https://github.com/getsentry/responses#responses-as-a-context-manager)
Fichiers
Demandes liées
Révisions associées
tests: use more responses module to mock HTTP responses (#68527)
Replace every mock.patch('http_related') pattern.
Historique
Mis à jour par Frédéric Péters il y a plus d'un an
Pour info (avant de voir ce ticket) j'ai commencé par l'autre bout, pour utiliser responses dans la fixture http_requests (https://git.entrouvert.org/wcs.git/log/?h=wip/xxxxx-responses-as-mock-library).
Mis à jour par Frédéric Péters il y a plus d'un an
- Fichier 0001-tests-use-responses-module-to-mock-HTTP-responses-68.patch 0001-tests-use-responses-module-to-mock-HTTP-responses-68.patch ajouté
- Statut changé de Nouveau à En cours
Voilà déjà cette partie autour de la fixture http_requests, 9 files changed, 514 insertions(+), 513 deletions(-).
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Il y a ce diff qui s'est glissé dans ce patch qui ne m'a pas l'air d'être lié :
def get_subjects(self):
for em in mail.outbox:
yield em.subject
Mis à jour par Frédéric Péters il y a plus d'un an
Il est lié (utile voire nécessaire pour le développement de ces tests, obtenir les sujets, il m'a semblé pertinent de le laisser dans le code, pouvant être à nouveau utile).
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Frédéric Péters a écrit :
Il est lié (utile voire nécessaire pour le développement de ces tests, obtenir les sujets, il m'a semblé pertinent de le laisser dans le code, pouvant être à nouveau utile).
Je ne vois aucun appel à get_subjects, d'où ma remarque, si ça arrive ensuite pas de soucis.
Mis à jour par Frédéric Péters il y a plus d'un an
Ça a été utilisé lors du développement du commit, c'est laissé sur l'idée que ça pourra être utilisé lors du développement d'autres commits. (ça peut tout à fait être retiré et si plus tard il y a à nouveau besoin lors du développement d'un test d'obtenir ça, ça pourra être recodé facilement).
Mis à jour par Emmanuel Cazenave il y a plus d'un an
- Sujet changé de Utilser la lib responses dans les tests à Utiliser la lib responses dans les tests
Mis à jour par Emmanuel Cazenave il y a plus d'un an
- Fichier 0002-tests-use-more-responses-module-to-mock-HTTP-respons.patch 0002-tests-use-more-responses-module-to-mock-HTTP-respons.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Et la deuxième partie qui remplace tous les mock.patch (poussé sur la branche par dessus le premier commit).
A l'exception de deux mock.patch('requests.Session.request') dans test_misc.py où est testée la méthode _http_request. Ici l'utilisation de responses me semblerait obscurcir les choses.
Mis à jour par Emmanuel Cazenave il y a plus d'un an
- Bloque Development #68337: Exploitation de APIClient ajouté
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
517 insertions, 550 deletions
un bilan final qui retire des lignes, ouf.
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
Poussé dès maintenant,
commit 5522a3f7b38fc21a57511048b08182ecf61c9db7
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Tue Aug 30 17:05:39 2022 +0200
tests: use more responses module to mock HTTP responses (#68527)
Replace every mock.patch('http_related') pattern.
commit 74f0c9e8f0502c5b688b15ccbfa009df4d06251f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Aug 29 20:01:05 2022 +0200
tests: use responses module to mock HTTP responses (#68527)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
tests: use responses module to mock HTTP responses (#68527)