Projet

Général

Profil

Bug #61262

erreur sur test_dossier_http_error

Ajouté par Frédéric Péters il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Nicolas Roche
Version cible:
-
Début:
31 janvier 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://jenkins.entrouvert.org/job/passerelle/2653/testReport/junit/py3-django22-codestyle-coverage.tests/test_mdph13/test_dossier_http_error/

    def test_dossier_http_error(app, mdph13, mock_http, caplog):
        url = '/mdph13/test1/link/?NameID=%s&numero_dossier=%s&date_de_naissance=%s&secret=%s&email=%s&ip=%s'
        url = url % (NAME_ID, FILE_NUMBER, DOB, SECRET, EMAIL, IP)
        mock_http.add_response({'status_code': 401, 'content': 'wtf', 'reason': 'Authentication required'})
        response = app.post(url, status=500)
        assert response.json['err_class'] == 'requests.exceptions.HTTPError'
>       assert caplog.records[3].levelno == logging.ERROR
E       assert 30 == 40
E        +  where 30 = <LogRecord: django.request, 30, /tmp/tox-jenkins/passerelle/py3-django22-codestyle-coverage/lib/python3.9/site-packages/django/utils/log.py, 222, "%s: %s">.levelno
E        +  and   40 = logging.ERROR

après un commit sans rapport.

Fichiers

0001-tests-adapt-logs-check-on-charset-normalizer-upgrade.patch (1,68 ko) 0001-tests-adapt-logs-check-on-charset-normalizer-upgrade.patch Nicolas Roche, 31 janvier 2022 11:50
0001-tests-adapt-logs-check-on-charset-normalizer-upgrade.patch (2,4 ko) 0001-tests-adapt-logs-check-on-charset-normalizer-upgrade.patch Nicolas Roche, 31 janvier 2022 16:04

Demandes liées

Lié à Passerelle - Bug #58995: Build cassé suite au passage de charset-normalizer de 2.0.7 à 2.0.8Fermé25 novembre 2021

Actions

Révisions associées

Révision b8cd7dd1 (diff)
Ajouté par Nicolas Roche il y a plus de 2 ans

tests: adapt logs check on charset-normalizer upgrade (#61262)

Historique

#1

Mis à jour par Nicolas Roche il y a plus de 2 ans

  • Lié à Bug #58995: Build cassé suite au passage de charset-normalizer de 2.0.7 à 2.0.8 ajouté
#2

Mis à jour par Nicolas Roche il y a plus de 2 ans 

<<<
charset-normalizer==2.0.10
---
charset-normalizer==2.0.11

Vu avec #58995 la 2.0.8 ajoutait 2 messages dans les logs.
Vu avec #59511 la 2.0.9 retirait un message.
À présent la 2.0.11 retire le second message.
(retour au test initial)

#3

Mis à jour par Frédéric Péters il y a plus de 2 ans

Ok mais sur l'idée que dans deux mois une autre modification viendra péter les choses, on pourrait modifier le test pour qu'il vérifie que le message qu'on cherche soit présent, sans se soucier de la position ?

#5

Mis à jour par Frédéric Péters il y a plus de 2 ans

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

Mis à jour par Nicolas Roche il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit b8cd7dd130d9196e6d2f15d7fd30c964300e572c
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Mon Jan 31 11:24:32 2022 +0100

    tests: adapt logs check on charset-normalizer upgrade (#61262)
#7

Mis à jour par Frédéric Péters il y a plus de 2 ans

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

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF