Bug #33224
tests sur hobo_deploy.py (côté agents)
Début:
17 mai 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Ticket pour ajouter des tests unitaires (idéalement) sur les appels des commandes hobo-deploy sur les agents :
- ./bijoe/bijoe/hobo_agent/management/commands/hobo_deploy.py (pas possible ici)
- ./hobo/hobo/agent/authentic2/management/commands/hobo_deploy.py (compliqué ici)
- ./hobo/hobo/agent/combo/management/commands/hobo_deploy.py
- ./hobo/hobo/agent/hobo/management/commands/hobo_deploy.py
- ./hobo/hobo/agent/common/management/commands/hobo_deploy.py
Fichiers
Demandes liées
Révisions associées
tests: add unit tests for hobo_deploy.py (#33224)
Historique
Mis à jour par Nicolas Roche il y a presque 5 ans
j'ai ajouté :
- dans tests_schemas/: des tests qui vérifient les surcharges des fonctions dans les agents
- dans tests/: des tests unitaires de couvertures
- dans hobo/agent/common/management/commands/hobo_deploy.py :
test_get_theme(self, hobo_environment): ... if not theme.get('module'): # <-- alway False def configure_template(self, hobo_environment, tenant): # <-- unused 'tenant' parameter ... if 'import_template' in get_commands() ... # <-- alway True
- dans hobo/agent/combo/management/commands/hobo_deploy.py :
def get_theme(self, hobo_environment): try: # <-- never raise me = [x for x in hobo_environment.get('services') if x.get('this')][0]
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0001-hobo_deploy-add-unit-tests-for-hobo_deploy.py-33224.patch 0001-hobo_deploy-add-unit-tests-for-hobo_deploy.py-33224.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution proposée à En cours
Test qui ne passe pas.
Mis à jour par Nicolas Roche il y a presque 5 ans
- Statut changé de En cours à Solution proposée
(désolé, j'ai oublié de commiter avant de pousser.)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Éventuellement poser la partie qui change le README dans un autre commit. (et pour info, avoir deux espaces en anglais derrière les points est juste une pratique un peu désuète, inutile de te lancer pas dans un grand nettoyage)
Mis à jour par Nicolas Roche il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Merci, je vais proposer les simplifications dans des tickets liés.
commit 7dcc824782c11c79d8c0f5157e2a638bded35588 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Fri May 31 10:41:39 2019 +0200 tests: add unit tests for hobo_deploy.py (#33224) commit cbd08f81b7933ac7546cef8eaf16d1a3eb665822 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Fri May 31 10:35:51 2019 +0200 doc: typo into README file (#33224)
Mis à jour par Nicolas Roche il y a presque 5 ans
- Lié à Bug #33590: simplification de la fonction configure_template dans hobo_deploy.py ajouté
Mis à jour par Nicolas Roche il y a presque 5 ans
- Lié à Bug #33592: simplification de la fonction get_theme dans hobo_deploy.py ajouté
Mis à jour par Nicolas Roche il y a presque 5 ans
- Lié à Bug #33594: plus besoin de surcharger get_theme lors des déploiements de combo ajouté
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
doc: typo into README file (#33224)