Project

General

Profile

Bug #45615

Build toujours cassé

Added by Valentin Deniaud 6 days ago. Updated 2 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
30 Jul 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Hier, #45556, on a été un peu rapide à se dire que le bug identifié dans pytest grâce aux tests passerelle était aussi en cause ici.

En fait ça n'a rien à voir, là l'histoire c'est que pytest 6 ajoute des nouveaux handlers, et donc les deux tests hobo qui regardent combien il y en a, plantent.

Je n'ai pas d'idée pour réparer.

0001-tests-remove-some-handlers-added-by-pytest-45615.patch View (2.52 KB) Nicolas Roche, 31 Jul 2020 01:39 PM

0001-tests-insert-log-filter-on-correct-handler-45615.patch View (2.89 KB) Nicolas Roche, 31 Jul 2020 02:34 PM

Associated revisions

Revision 7ce95390 (diff)
Added by Nicolas Roche 5 days ago

tests: insert log filter on correct handler (#45615)

History

#1 Updated by Nicolas Roche 5 days ago

Ce patch qui permet de passer les tests sur l'ancienne et la nouvelle version de pytest.
Il ignore/retire les handlers qui posent problème.

#2 Updated by Frédéric Péters 5 days ago

Mmm, j'aurais plutôt imaginer vérifier la présence/l'absence de l'handler qu'on veut, et ignorer le nombre d'handlers qui est une info dont on se contrefiche, en fait, il me semble. (?)

#3 Updated by Nicolas Roche 5 days ago

Oui, j'espère que ce patch sera plus lisible :
  • sur le premier test, on insert le filtre sur le bon handler
  • sur la fixture du second test, on insert notre handler en début de liste

#4 Updated by Frédéric Péters 5 days ago

  • Status changed from Solution proposée to Solution validée

#5 Updated by Nicolas Roche 5 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 7ce95390ac8584ed0af2bee45909eda147b6e3b3
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri Jul 31 11:47:46 2020 +0200

    tests: insert log filter on correct handler (#45615)

#6 Updated by Frédéric Péters 2 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF