Project

General

Profile

Bug #33224

tests sur hobo_deploy.py (côté agents)

Added by Nicolas Roche 3 months ago. Updated 3 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
agent
Target version:
-
Start date:
17 May 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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

0001-hobo_deploy-add-unit-tests-for-hobo_deploy.py-33224.patch View (30.2 KB) Nicolas Roche, 24 May 2019 09:16 PM


Related issues

Related to Hobo - Bug #33590: simplification de la fonction configure_template dans hobo_deploy.py Solution proposée 31 May 2019
Related to Hobo - Bug #33592: simplification de la fonction get_theme dans hobo_deploy.py Solution déployée 31 May 2019
Related to Hobo - Bug #33594: plus besoin de surcharger get_theme lors des déploiements de combo Solution déployée 31 May 2019

Associated revisions

Revision cbd08f81 (diff)
Added by Nicolas Roche 3 months ago

doc: typo into README file (#33224)

Revision 7dcc8247 (diff)
Added by Nicolas Roche 3 months ago

tests: add unit tests for hobo_deploy.py (#33224)

History

#1 Updated by Nicolas Roche 3 months ago

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
4 endroits pourraient éventuellement être simplifiés :
  • 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]
    

#2 Updated by Nicolas Roche 3 months ago

#3 Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Solution proposée to En cours

Test qui ne passe pas.

#4 Updated by Nicolas Roche 3 months ago

  • Status changed from En cours to Solution proposée

(désolé, j'ai oublié de commiter avant de pousser.)

#5 Updated by Frédéric Péters 3 months ago

  • Status changed from Solution proposée to 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)

#6 Updated by Nicolas Roche 3 months ago

  • Status changed from Solution validée to 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)

#7 Updated by Nicolas Roche 3 months ago

  • Related to Bug #33590: simplification de la fonction configure_template dans hobo_deploy.py added

#8 Updated by Nicolas Roche 3 months ago

  • Related to Bug #33592: simplification de la fonction get_theme dans hobo_deploy.py added

#9 Updated by Nicolas Roche 3 months ago

  • Related to Bug #33594: plus besoin de surcharger get_theme lors des déploiements de combo added

#10 Updated by Frédéric Péters 3 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF