Development #42706
Supprimer w.c.s. des dépendances de test
0%
Description
Pour tester le module tests/test_wcs.py, il faut installer w.c.s. et y injecter des données pendant le setup du module.
Pour gagner du temps d'init, on pourrait supprimer cette dépendance et mocker les appels à l'api w.c.s.
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Fichier 0001-tests-remove-w.c.s.-from-test-dependencies-42706.patch 0001-tests-remove-w.c.s.-from-test-dependencies-42706.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En local, je suis à 6s de tests vs 16s avant - 10s dans le setup)
Mis à jour par Lauréline Guérin il y a presque 4 ans
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
Ça a peut-être été validé par ailleurs, et ma question tombera à plat; je ne doute pas que ça aille bien plus vite mais tout l'intérêt n'est-il pas de vérifier que ça marche toujours avec le vrai w.c.s. ? (sachant qu'on pas super formel en matière de contrat d'interface, si encore on mockait à part d'un swagger ou wsdl, si tant est que ce soit possible).
Ça ne me gênerait pas d'avoir en général du mock et de temps en temps des runs sur un vrai w.c.s. (mais ça revient à faire deux fois le boulot chaque fois qu'on fera un test qui interagit avec w.c.s.).
Mis à jour par Frédéric Péters il y a presque 4 ans
On en a discuté préalablement avec Lauréline et ma position était qu'il y a aujourd'hui suffisamment de tests (et d'attention) pour qu'on puisse considérer qu'un changement inattendu à l'API soit détecté, que le coût de démarrage/assemblage de w.c.s. pouvait du coup être éliminé.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
On en a discuté préalablement avec Lauréline et ma position était qu'il y a aujourd'hui suffisamment de tests (et d'attention) pour qu'on puisse considérer qu'un changement inattendu à l'API soit détecté, que le coût de démarrage/assemblage de w.c.s. pouvait du coup être éliminé.
#42429 me laisse penser que sur certains coins obscurs ça peut être utile d'avoir un vrai w.c.s. en face, mais c'est bien aussi de pouvoir itérer plus vite (j'ai fait mes remarques) ; pour moi étant donné que ça ne concerne que les tests, c'est validé d'office.
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8b0e96ef16a580d3954b15cd300cbf792ca459ab
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Thu May 7 08:47:50 2020 +0200
tests: remove w.c.s. from test dependencies (#42706)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests: remove w.c.s. from test dependencies (#42706)