Bug #59511
build jenkins cassé sur 2 tests
0%
Description
Le premier test HS arrive le 4/12 et le second le 7/12.
Fichiers
Révisions associées
test: correct endpoint URL on cryptor test (#59511)
Historique
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Statut changé de Nouveau à En cours
Pour info,
- 2565 -> 2568 (tests/test_mdph13.py::test_dossier_http_error)
charset-normalizer : 2.0.8 -> 2.0.9
pycryptodomex : 3.11.0 -> 3.12.0
pylint : 2.12.1 -> 2.12.2
typed-ast : 1.5.0 -> 1.5.1
A priori c'est lié à charset-normalizer comme déjà vu avec #58995
- 2568 -> 2569 (tests/test_cryptor.py::test_cryptor_encrypt_decrypt)
Django: 2.2.24 -> 2.2.25
pytest-django: 4.5.1 -> 4.5.2
Une 200 qui devient une 400.
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier 0002-test-correct-endpoint-URL-on-cryptor-test-59511.patch 0002-test-correct-endpoint-URL-on-cryptor-test-59511.patch ajouté
- Fichier 0001-tests-adapt-logs-check-on-charset-normalizer-upgrade.patch 0001-tests-adapt-logs-check-on-charset-normalizer-upgrade.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
Le deuxième patch est louche, tu ne testes plus le même URL qu'avant.
Mis à jour par Nicolas Roche il y a plus de 2 ans
Oui. Avant on avait :
endpoint = utils.generic_endpoint_url('cryptor', 'file-decrypt', slug=cryptor.slug) endpoint = endpoint + '/' + str(cfile.uuid) endpoint = endpoint + '/' + str(cfile.uuid)
Et ça fonctionnait quand même.
(j'ai cru que c'était un oubli dans le code, mais bon ici c'est Thomas qui le connaît).
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Nicolas Roche a écrit :
Et ça fonctionnait quand même.
(j'ai cru que c'était un oubli dans le code, mais bon ici c'est Thomas qui le connaît).
Good catch ; ack sur ce patch 0002. 0001 est ok aussi.
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a795d36b3e9bf02f8e916de0da6a0d67aa761013 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Thu Dec 9 10:30:41 2021 +0100 test: correct endpoint URL on cryptor test (#59511) commit 9ec2738c951ce040ef324174630ec5fdbf4cb1a8 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Thu Dec 9 10:18:05 2021 +0100 tests: adapt logs check on charset-normalizer upgrade (#59511)
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
tests: adapt logs check on charset-normalizer upgrade (#59511)