Bug #48726
Test qui ne passe que le lundi, le mardi, le samedi et le dimanche
Début:
20 novembre 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
cf https://jenkins.entrouvert.org/job/chrono/961
Vu d'une part le code de get_time_slots qui me semble fonctionner par semaine
et d'autre part le test qui échoue lorsqu'il augmente les délais de reservation,
j'ai l'impression que le test n'est pas écrit pour fonctionner en milieu de semaine.
Enfin j'espère que c'est le cas mais c'est peut-être aussi un bug.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Lié à Development #48078: API datetimes : permettre de restreindre les résultats sur des bornes min/max de dates ajouté
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Fichier 0001-tests-freeze-date-on-a-test-48726.patch 0001-tests-freeze-date-on-a-test-48726.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Fichier 0001-tests-correct-modulo-on-a-test-48726.patch 0001-tests-correct-modulo-on-a-test-48726.patch ajouté
Non, en fait le test est bien écrit pour fonctionner tous les jours de la semaine,
mais il y avait un bug dans l'écriture des modulo qui a engendré l'effet constaté.
Mis à jour par Frédéric Péters il y a plus de 3 ans
Du patch il faudrait maintenant que le fonctionnement est validé retirer les deux fixtures, non ?
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Fichier 0001-tests-correct-modulo-on-a-test-48726.patch 0001-tests-correct-modulo-on-a-test-48726.patch ajouté
Oui.
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a007e425b513235fbbdb471027b5c236e84b6036
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Fri Nov 20 16:53:28 2020 +0100
tests: correct modulo on a test (#48726)
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
tests: correct modulo on a test (#48726)