Development #77483
tests saml TestAddRole
0%
Description
Pour un autre ticket j'étais parti à la recherche de ce qui existait spécifique authentification SAML dans l'attribution automatique de rôle (à la {"action": "toggle-role", "role": {"name": "Connexion ADFS"}}) et à ne pas les trouver (normal ça a été unifié/supprimé dans #67025) malgré des traces dans les tests, dans tests/auth_saml/test_adapter.py :
@pytest.mark.parametrize('action_name', ['add-role', 'toggle-role']) class TestAddRole: @pytest.fixture def idp(self, action_name, simple_role): authenticator = SAMLAuthenticator.objects.create( enabled=True, metadata='meta1.xml', slug='idp1', ) AddRoleAction.objects.create(authenticator=authenticator, role=simple_role) return authenticator.settings @pytest.fixture def saml_attributes(self): return { 'issuer': 'https://idp.com/', 'name_id_content': 'xxx', 'name_id_format': lasso.SAML2_NAME_IDENTIFIER_FORMAT_PERSISTENT, } def test_lookup_user_success(self, adapter, simple_role, idp, saml_attributes): user = adapter.lookup_user(idp, saml_attributes) assert simple_role in user.roles.all()
comme action_name n'est pas utilisé, le test ne va pas varier une fois add-role une fois toggle-role.
Mais à regarder dans les résultats des tests jenkins, ce bloc TestAddRole ne produit en fait rien. (?)
Si ce code doit servir mais que c'est une évolution quelconque qui fait qu'on ne passe plus dessus, corriger, mais plus vraisemblablement, si ce code ne doit plus servir autant le supprimer. (?)
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a 12 mois
Je pense que ce test est un vrai test de la fonctionnalité d'ajout de rôle pour SAML.
L'adaptation du test suite à la migration du paramétrage en base de donnée a en effet laissé derrière elle le pytest.mark.parametrize qui devrait être supprimé.
ce bloc TestAddRole ne produit en fait rien. (?)
Je ne comprends pas ce passage, qu'est-ce qui te permet d'affirmer ça ? (perso si je vais voir sur jenkins je ne vois pas la liste des tests qui sont exécutés)
En local si je lance le test test_lookup_user_success il est bien collecté et exécuté, ça serait bizarre que ce soit différent sur jenkins.
Mis à jour par Frédéric Péters il y a 12 mois
C'est parce que je ne le voyais pas ici https://jenkins.entrouvert.org/job/authentic/3457/testReport/py3-stable-backports.tests.auth_saml/test_adapter/ mais c'est parce qu'il est en fait ici https://jenkins.entrouvert.org/job/authentic/3457/testReport/py3-stable-backports.tests.auth_saml.test_adapter/TestAddRole/ il est bel et bien exécuté.
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de Nouveau à Solution proposée
Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/55
- Titre : tests: remove mention of unused saml role action name (#77483)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/55/files
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de Solution proposée à Solution validée
Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/55
- Titre : tests: remove mention of unused saml role action name (#77483)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/55/files
Mis à jour par Transition automatique il y a 12 mois
- Statut changé de Résolu (à déployer) à Solution déployée
tests: remove mention of unused saml role action name (#77483)