Projet

Général

Profil

Bug #32687

regression sur cook avec plusieurs variables

Ajouté par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans. Mis à jour il y a presque 5 ans.

Statut:
Fermé
Priorité:
Haut
Catégorie:
agent
Version cible:
-
Début:
29 avril 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

L'action est appelée autant de fois qu'elle a de variables.
Mis en évidence par :

@mock.patch('hobo.environment.management.commands.cook.open')
def test_having_several_action_args(mocked_open):
    """load variables from a file""" 
    receipe = """{
  "steps": [
    {"create-authentic": {
      "url": "https://entrouvert.org/",
      "title": "Connexion" 
    }}
  ]
}""" 
    expected_a2 = "[call(title=u'Connexion', url=u'https://foo1/')]" 

    command = Command()
    command.permissive = True
    command.create_authentic = mock.MagicMock()

    mocked_open.side_effect = [StringIO.StringIO(receipe)]
    command.run_cook('recipe_me.json')
    assert str(command.create_authentic.mock_calls) == expected_a2

---8<---
E       assert "[call(title=...uvert.org/')]" == "[call(title=u...ps://foo1/')]" 
E         - [call(title=u'Connexion', url=u'https://entrouvert.org/'),
E         -  call(title=u'Connexion', url=u'https://entrouvert.org/')]
E         ? ^                                       ^^^^ ------ ^^
E         + [call(title=u'Connexion', url=u'https://foo1/')]
E         ? ^                              

C'est ma faute, j'ai laissé passer une erreur d'indentation quand j'ai validé #16599


Fichiers


Demandes liées

Lié à Hobo - Development #16599: cook : vérification des noms de domaine avant de lancer le déploiementFermé29 mai 2017

Actions

Révisions associées

Révision f3feefca (diff)
Ajouté par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans

cook: correct reression on cook action using several args (#32687)

Historique

#1

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans

  • Lié à Development #16599: cook : vérification des noms de domaine avant de lancer le déploiement ajouté
#2

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans

#3

Mis à jour par Christophe Siraut il y a presque 5 ans

  • Statut changé de Solution proposée à Solution validée

merci.

#4

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit f3feefca2b2fc1694dc5dfbda50cc9a5104c4cc0
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Mon Apr 29 19:26:01 2019 +0200

    cook: correct reression on cook action using several args (#32687)
#6

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

Formats disponibles : Atom PDF