Projet

Général

Profil

Bug #84843

échec du build après merge #84710

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

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

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

build qui était ok dans la PR :/

py3-django32-authentic.tests_authentic.test_provisionning.test_middleware

        assert notify_agents.call_count == 0
        resp = resp.form.submit().follow()
>       assert notify_agents.call_count == 1
E       AssertionError: assert 0 == 1
E        +  where 0 = <MagicMock name='notify_agents' id='140479618804320'>.call_count

tests_authentic/test_provisionning.py:676: AssertionError

Demandes liées

Lié à Hobo - Development #84864: Arrêter de mettre des choses spécifiques aux briques dans hobo (et debian_config_common.py)Nouveau18 décembre 2023

Actions

Révisions associées

Révision 2de52d9e (diff)
Ajouté par Benjamin Dauvergne il y a 4 mois

misc: fix checking for authentic2 context (#84843)

INSTALLED_APPS in authentic2.settings does not contain the string
'authentic2' anymore, we should not use it as a flag for the authentic2
context.

Historique

#1

Mis à jour par Benjamin Dauvergne il y a 4 mois

  • Assigné à mis à Benjamin Dauvergne
#2

Mis à jour par Robot Gitea il y a 4 mois

  • Statut changé de Nouveau à Solution proposée

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#3

Mis à jour par Benjamin Dauvergne il y a 4 mois

Bug tordu a cause du code moche et fragile dans debian_config_common.py, en changeant INSTALLED_APPS dans #84019 ça a cassé tous les tests qui vérifie si on est dans authentic2 ou pas, ici c'est le ProvisionningMiddleware spécifique à authentic qui n'était plus configuré.

#4

Mis à jour par Benjamin Dauvergne il y a 4 mois

  • Lié à Development #84864: Arrêter de mettre des choses spécifiques aux briques dans hobo (et debian_config_common.py) ajouté
#5

Mis à jour par Robot Gitea il y a 4 mois

  • Statut changé de Solution proposée à Solution validée

Emmanuel Cazenave (ecazenave) a approuvé une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Robot Gitea il y a 4 mois

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

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Transition automatique il y a 4 mois

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

Mis à jour par Transition automatique il y a 2 mois

Automatic expiration

Formats disponibles : Atom PDF