Bug #48117
build jenkins cassé
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Lié à Development #47760: auth_saml: revoir le logging des erreurs de mapping ajouté
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Fichier 0001-tests-fix-auth_saml-tests-48117.patch 0001-tests-fix-auth_saml-tests-48117.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Je vois bien comment #47760 a cassé test_provision_attributes mais pas comment ça a cassé test_provision_add_role.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Je pense que c'est le rebase automatique au dessus des changements de "auth_saml: rename toggle-role action to add-role (#46857)" qui a foiré (deux fonctions qui se terminent par le même épilogue).
Mis à jour par Paul Marillonnet il y a plus de 3 ans
Benjamin Dauvergne a écrit :
Je pense que c'est le rebase automatique au dessus des changements de "auth_saml: rename toggle-role action to add-role (#46857)" qui a foiré (deux fonctions qui se terminent par le même épilogue).
C'est vert. Je retire ma délation dans le message de commit si tu veux :)
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Paul Marillonnet a écrit :
Benjamin Dauvergne a écrit :
Je pense que c'est le rebase automatique au dessus des changements de "auth_saml: rename toggle-role action to add-role (#46857)" qui a foiré (deux fonctions qui se terminent par le même épilogue).
C'est vert. Je retire ma délation dans le message de commit si tu veux :)
Non ce qui m'embête c'est que le code des tests n'a un peu aucun sens là, j'ai l'impression que ça fait plusieurs commits que tout se mélange.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-auth_saml-reorganize-and-fix-tests-48117.patch 0001-auth_saml-reorganize-and-fix-tests-48117.patch ajouté
- Tracker changé de Bug à Development
C'est un peu plus clair là.
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Tracker changé de Development à Bug
- Assigné à
Benjamin Dauvergnesupprimé
Benjamin Dauvergne a écrit :
Non ce qui m'embête c'est que le code des tests n'a un peu aucun sens là, j'ai l'impression que ça fait plusieurs commits que tout se mélange.
Ah bein zut j'ai vu un bloc doublonné d'un test à l'autre sans aucun rapport, je pensais que c'était un copié-collé et je l'ai retiré. Pas capté que c'était un micmac de git-merge.
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Assigné à mis à Benjamin Dauvergne
(Pardon, fausse manip')
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne a écrit :
C'est un peu plus clair là.
Maravilloso. Ça passe en local chez moi, mais à pousser quand Jenkins voit vert, sait-on jamais.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8d6b4653e3374f1988f0e7e43cb380e4bc6dae1f Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Thu Oct 29 16:11:19 2020 +0100 auth_saml: reorganize and fix tests (#48117)
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
auth_saml: reorganize and fix tests (#48117)