Projet

Général

Profil

Bug #48117

build jenkins cassé

Ajouté par Paul Marillonnet il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

c'est rouge depuis #47760


Fichiers


Demandes liées

Lié à Authentic 2 - Development #47760: auth_saml: revoir le logging des erreurs de mappingFermé15 octobre 2020

Actions

Révisions associées

Révision 8d6b4653 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 3 ans

auth_saml: reorganize and fix tests (#48117)

Historique

#1

Mis à jour par Paul Marillonnet il y a plus de 3 ans

#2

Mis à jour par Paul Marillonnet il y a plus de 3 ans

#3

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.

#4

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).

#5

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 :)

#6

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.

#7

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

  • Assigné à mis à Benjamin Dauvergne
#8

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

C'est un peu plus clair là.

#9

Mis à jour par Paul Marillonnet il y a plus de 3 ans

  • Tracker changé de Development à Bug
  • Assigné à Benjamin Dauvergne supprimé

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.

#10

Mis à jour par Paul Marillonnet il y a plus de 3 ans

  • Assigné à mis à Benjamin Dauvergne

(Pardon, fausse manip')

#11

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.

#12

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)
#13

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