Produits Entr'ouvert » Hobo

Du bout "je clique sur activate debug et ça ajoute mon ip cliente dans le truc" de la description, je me dis que ça peut être bien d'avoir ainsi l'ajout de son IP, sans avoir à la chercher. (peut-être que ça peut juste être un truc d'affichage, sur la page "votre adresse IP est : ").

Souci nouveau à cause des tests sur cook, une des mes modifs non essentielles :

# 0001-environment-coding-style-29240.patch
    def as_dict(self):
         as_dict = dict([(x, y) for (x, y) in self.__dict__.items()
-                        if type(y) in (int, str, unicode)])-
+                        if isinstance(y, six.integer_types + six.string_types)])
         as_dict['base_url'] = self.get_base_url_path()
         as_dict['service-id'] = self.Extra.service_id
         as_dict['service-label'] = force_text(self.Extra.service_label)

fait que des attributs non exposés dans le hobo.json des services le sont maintenant :

>       assert dump == expected_dump
E       assert '{\n    "prof...89"\n    }\n}' == '{\n    "profi...89"\n    }\n}'
E         Skipping 5532 identical leading characters in diff, use -v to show
E         -          "use_as_idp_for_self": true,
E         -             "variables": {}
E         ? ---
E         +          "variables": {}
E                   },
E                   {...
E         
E         ...Full output truncated (66 lines hidden), use '-vv' to show

Ici "use_as_idp_for_self" qui est booléen et désormais pris en compte par six.integer_types (les booléens sont des entiers en python pour des raisons historiques), je défais ça ?

Juste pour commenter le point ci-dessus avec les tests sur cook :
le test ne fait rien de plus que d'avertir quand l'environnement (hobo.json) évolue.

S'il évolue et que c'est souhaité, alors le patch 2 fait l'affaire :
- soit modifier le json attendu à la main
- soit le régénérer comme indiqué dans tests_schema/test_cook.py::test_cook_example

Je répond à côté car je n'ai pas d'avis sur le fait d'avoir une nouvelle entrée "use_as_idp_for_self".

Nicolas Roche a écrit :

Je répond à côté car je n'ai pas d'avis sur le fait d'avoir une nouvelle entrée "use_as_idp_for_self".

Je pense que c'est juste la bonne chose à faire, hobo.json doit exposer tout ce qui est stocké dans les objets de service.

Voici 2 tests pour améliorer la couverture.
Le second montre un petit problème dans l'IHM avec les IP non valables.

/tests/test_manager.py::test_debug_home() :

    # toggle-current-ip button
    page = logged_app.get('/debug/')
    page.form['debug_ips'] = IPS
    page = page.form.submit(name='toggle-current-ip').follow()  # click
    assert get_setting_variable('INTERNAL_IPS').json == IP_LIST + ['127.0.0.1']
    page = page.form.submit(name='toggle-current-ip').follow()  # click again
    assert get_setting_variable('INTERNAL_IPS').json == IP_LIST

    # wrong ips are not returned as a list
    page = logged_app.get('/debug/')
    page.form['debug_ips'] = 'not_an_IP'
    page = page.form.submit()
    assert 'Enter a valid IPv4 or IPv6 address' in page.content
    assert page.form['debug_ips']._value == 'not_an_IP'  # get 'n o t _ a n _ I P'

Pour moi l'IHM fait le job, pas de problème.

Par contre, le taf attendu, logguer, sur l'environnement de dev je ne vois rien, et c'est normal ce ticket n'avait aucun sens à être poussé avant #29240.

Frédéric Péters a écrit :

Pour moi l'IHM fait le job, pas de problème.

Par contre, le taf attendu, logguer, sur l'environnement de dev je ne vois rien, et c'est normal ce ticket n'avait aucun sens à être poussé avant #29240.

#29149, on peut déjà vérifier que les settings sont bien là.