Projet

Général

Profil

Bug #47120

mettre à jour les tests pour authentic

Ajouté par Frédéric Péters il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Nicolas Roche
Catégorie:
-
Version cible:
-
Début:
29 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://jenkins.entrouvert.org/job/hobo/3414/ en erreur suite à a priori #46182.

Fichiers

0001-tests-add-manage-authorizations-to-expected-tests-re.patch (1,68 ko) 0001-tests-add-manage-authorizations-to-expected-tests-re.patch Nicolas Roche, 30 septembre 2020 16:18
0002-tox-remove-python2-test-for-passerelle-47120.patch (1,18 ko) 0002-tox-remove-python2-test-for-passerelle-47120.patch Nicolas Roche, 30 septembre 2020 19:26
0001-tests-add-manage-authorizations-to-expected-tests-re.patch (1,69 ko) 0001-tests-add-manage-authorizations-to-expected-tests-re.patch Nicolas Roche, 30 septembre 2020 19:26

Demandes liées

Lié à Authentic 2 - Development #47197: Problème sur les tests hobo lancés en python 3.5Rejeté30 septembre 2020

Actions

Révisions associées

Révision b063c9cc (diff)
Ajouté par Nicolas Roche il y a plus de 3 ans

tests: add manage authorizations to expected tests results (#47120)

Révision c9eb0266 (diff)
Ajouté par Nicolas Roche il y a plus de 3 ans

tox: remove python2 test for passerelle (#47120)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 3 ans

Nicolas, tu peux confirmer que tu as vu ce ticket ?

#2

Mis à jour par Nicolas Roche il y a plus de 3 ans

Non, je n'avais pas fait le lien avec authentic, merci.

#3

Mis à jour par Nicolas Roche il y a plus de 3 ans

#46182 ajoute effectivement la permission "manage_authorizations" au groupe "Manager of users",
qui est la raison de la première régression sanctionnée par les tests.

Mais ensuite les test plantent plus loin (en python 2 uniquement), parce que le nombre de messages renvoyés est différent suivant les versions de python.

>               assert notify_agents.call_count == 2
E               AssertionError: assert 1 == 2
E                +  where 1 = <MagicMock name='notify_agents' id='140519727015632'>.call_count

Là je n'arrive pas à comprendre d'où ça vient.

#4

Mis à jour par Nicolas Roche il y a plus de 3 ans

#5

Mis à jour par Frédéric Péters il y a plus de 3 ans

Note que les tests hobo/py2/authentic ne doivent plus tourner, ça sert à rien de te casser la tête sur cet aspect.

#6

Mis à jour par Nicolas Roche il y a plus de 3 ans

Oui, j'ai tenté de faire passer ce patch derrière #47185 (qui retire les tests en python2), et sans #47084 (qui introduit une autre erreur), mais j'ai encore un autre soucis avec jenkins, que je n'arrive pas à comprendre :

FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tox-jenkins/hobo/wip/47120-add-manage-authorizations-role-into-tests/py3-coverage-multitenant-oldstylemiddleware/bin/pip'

#7

Mis à jour par Nicolas Roche il y a plus de 3 ans

  • Statut changé de En cours à Solution proposée

J'ai l'impression que sur la branche hobo, jenkins n'a noté qu'un seul test défaillant et que le patch proposé plus haut le corrige.
https://jenkins.entrouvert.org/job/hobo/3423/

Je ne pense pas que ce soit la bonne façon de procéder, mais je n'arrive pas à trouver quelle configuration d'authentic et de hobo je dois tester (en tout cas j'ai beau m'acharner, j'ai l'impression que je n'aurais pas les tests au vert sur cette branche).

Je veux bien de l'aide pour m'en sortir.

#8

Mis à jour par Frédéric Péters il y a plus de 3 ans

que sur la branche hobo

Tu fais référence à quoi ?

J'ai créé ce ticket quand il y avait juste je pense manage_authorizations; après c'est bien le problème de continuer à avoir eu des commits alors que c'était cassé ici.

mais je n'arrive pas à trouver quelle configuration d'authentic et de hobo je dois tester

master et master, c'est ça que jenkins doit faire tourner. Ou je ne pige pas la question.

#9

Mis à jour par Frédéric Péters il y a plus de 3 ans

Pour la branche faut juste se calmer sur les noms à rallonge; ça génère un pip avec :

#!/tmp/tox-jenkins/hobo/wip/47120-add-manage-authorizations-role-into-tests/py3-coverage-multitenant-oldstylemiddleware/bin/python3

131 caractères, c'est au-dessus de la limite à 128.

#10

Mis à jour par Frédéric Péters il y a plus de 3 ans

(je viens de pousser une branche wip/47120-adapt-tests identique à la tienne, qui va normalement passer jenkins).

#11

Mis à jour par Nicolas Roche il y a plus de 3 ans

Merci !
Il me reste encore une dernière erreur à comprendre puis corriger :

[Checks API] No suitable checks publisher found.
Finished: FAILURE

#12

Mis à jour par Nicolas Roche il y a plus de 3 ans

Ce dernier bug, c'était une typo que je viens d'introduire.

J'avais un doute mais il faut aussi désactiver les tests python2 sur l'environnement de test passerelle (comme Benjamin me l'a conseillé) :

File "/tmp/tox-jenkins/hobo/wip/47120-add-perm-to-test-file/py2-passerelle/local/lib/python2.7/site-packages/passerelle/apps/maelis/models.py", line 20, in <module>
    from urllib.parse import urljoin
ImportError: No module named parse

(Là pour que ça fonctionne la branche est rebasée sur #47185)

#13

Mis à jour par Benjamin Dauvergne il y a plus de 3 ans

  • Statut changé de Solution proposée à Résolu (à déployer)
commit c9eb0266829b6f4b1b5b72a134f4bdb61f4f2a89
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Sep 30 18:16:11 2020 +0200

    tox: remove python2 test for passerelle (#47120)

commit b063c9cc0f7b77c5ee71330166b4dd170345b553
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Sep 30 14:37:24 2020 +0200

    tests: add manage authorizations to expected tests results (#47120)
#14

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF