tests: fix workflow tests failing at midnight (#87143) #1164

vdeniaud · 2024-02-19T14:19:02+01:00

vdeniaud commented

2024-02-19 14:19:02 +01:00

Je botte en touche, ça veut dire que oui le succès d'un test de workflow qui utilise des sauts dans le temps + des conditions age_in n'est pas stable en fonction du moment où il est exécuté.

L'alternative c'est de permettre de spécifier un receipt_time fixe plutôt que de prendre l'heure courante à chaque exécution du test, mais ça nécessite d'aller mocker tous les appels à now() ensuite, je vais peut-être attendre que quelqu'un se plaigne du fonctionnement actuel avant d'aller dans cette direction.

Je botte en touche, ça veut dire que oui le succès d'un test de workflow qui utilise des sauts dans le temps + des conditions age_in n'est pas stable en fonction du moment où il est exécuté. L'alternative c'est de permettre de spécifier un receipt_time fixe plutôt que de prendre l'heure courante à chaque exécution du test, mais ça nécessite d'aller mocker tous les appels à now() ensuite, je vais peut-être attendre que quelqu'un se plaigne du fonctionnement actuel avant d'aller dans cette direction.

vdeniaud added 1 commit 2024-02-19 14:19:02 +01:00

gitea/wcs/pipeline/head This commit looks good Details

40518093bc tests: fix workflow tests failing at midnight (#87143 )

fpeters approved these changes 2024-02-19 14:19:55 +01:00

vdeniaud merged commit 40518093bc into main

2024-02-19 14:31:51 +01:00

vdeniaud deleted branch wip/87143-testdef-un-test-de-wf-en-echec-a

2024-02-19 14:31:51 +01:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1164