Projet

Général

Profil

Development #77483

tests saml TestAddRole

Ajouté par Frédéric Péters il y a 12 mois. Mis à jour il y a 12 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
11 mai 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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

Révision 85ce9631 (diff)
Ajouté par Valentin Deniaud il y a 12 mois

tests: remove mention of unused saml role action name (#77483)

Historique

#1

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.

#3

Mis à jour par Valentin Deniaud il y a 12 mois

  • Assigné à mis à Valentin Deniaud
#4

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 :

#5

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 :

#6

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 :

#7

Mis à jour par Transition automatique il y a 12 mois

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

Mis à jour par Transition automatique il y a 10 mois

Automatic expiration

Formats disponibles : Atom PDF