Project

General

Profile

Bug #48726

Test qui ne passe que le lundi, le mardi, le samedi et le dimanche

Added by Nicolas Roche 15 days ago. Updated 9 days ago.

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

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files


Related issues

Related to Chrono - Development #48078: API datetimes : permettre de restreindre les résultats sur des bornes min/max de datesSolution déployée28 Oct 202004 Dec 2020

Actions

Associated revisions

Revision a007e425 (diff)
Added by Nicolas Roche 12 days ago

tests: correct modulo on a test (#48726)

History

#1

Updated by Nicolas Roche 15 days ago

  • Related to Development #48078: API datetimes : permettre de restreindre les résultats sur des bornes min/max de dates added
#2

Updated by Nicolas Roche 15 days ago

#3

Updated by Nicolas Roche 15 days ago

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é.

#4

Updated by Frédéric Péters 15 days ago

Du patch il faudrait maintenant que le fonctionnement est validé retirer les deux fixtures, non ?

#6

Updated by Emmanuel Cazenave 13 days ago

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

Updated by Nicolas Roche 12 days ago

  • Status changed from Solution validée to 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)
#8

Updated by Frédéric Péters 9 days ago

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

Also available in: Atom PDF