entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

testdef, workflow exécuté deux fois sur un changement de statut (#86955) #1140

Merged
fpeters merged 3 commits from wip/86955-testdef-workflow-execute-deux-fo into main 2024-02-16 11:11:21 +01:00
Conversation 0 Commits 3 Files Changed 5 +27 -21
vdeniaud commented 2024-02-14 15:17:22 +01:00
Owner
Copy Link

Mauvaise lecture du code de ma part, un changement de statut provoqué par un saut va taper dans formdata.perform_workflow, pas besoin de le faire par la suite.

Aussi en écrivant le test je me suis rendu compte que ma façon de vérifier pas d'email envoyé était mauvaise (ça part en afterjob mais l'afterjob jamais exécuté car attaché à une fausse réponse détruite à la fin du test).

Mauvaise lecture du code de ma part, un changement de statut provoqué par un saut va taper dans formdata.perform_workflow, pas besoin de le faire par la suite. Aussi en écrivant le test je me suis rendu compte que ma façon de vérifier pas d'email envoyé était mauvaise (ça part en afterjob mais l'afterjob jamais exécuté car attaché à une fausse réponse détruite à la fin du test).
vdeniaud force-pushed wip/86955-testdef-workflow-execute-deux-fo from fca34a97af to c4d940efd5 2024-02-14 15:29:46 +01:00 Compare
vdeniaud force-pushed wip/86955-testdef-workflow-execute-deux-fo from c4d940efd5 to b467a26d67 2024-02-14 15:40:25 +01:00 Compare
vdeniaud changed title from WIP: testdef, workflow exécuté deux fois sur un changement de statut (#86955) to testdef, workflow exécuté deux fois sur un changement de statut (#86955) 2024-02-14 15:45:36 +01:00
fpeters reviewed 2024-02-14 16:14:02 +01:00
wcs/workflow_tests.py Outdated
@ -66,3 +66,4 @@
formdata.workflow_test = True
formdata.workflow_tests_data = {
'now': formdata.receipt_time,
fpeters commented 2024-02-14 16:14:00 +01:00
Owner
Copy Link

Tu pourrais juste exploiter hasattr(formdata, 'workflow_tests_data') plutôt qu'introduire un nouvel attribut (si je lis bien), et peut-être ajouter ici un commentaire, type

# add context elements for workflow tests

ou (si tu préfères conserver l'ajout d'un nouvel attribut),

# mark formdata as running workflow tests

(comme tu le sens)

Tu pourrais juste exploiter hasattr(formdata, 'workflow_tests_data') plutôt qu'introduire un nouvel attribut (si je lis bien), et peut-être ajouter ici un commentaire, type ``` # add context elements for workflow tests ``` ou (si tu préfères conserver l'ajout d'un nouvel attribut), ``` # mark formdata as running workflow tests ``` (comme tu le sens)
vdeniaud commented 2024-02-15 17:17:29 +01:00
Author
Owner
Copy Link

En fait je n'aime plus trop ce dictionnaire workflow_tests_data, je le trouve trop verbeux, ça se voulait un genre de namespace pour indiquer que les attributs sont liés aux tests de workflow, mais puisqu'ils ne sont utilisés que dans ce cadre, c'est assez inutile.

J'ai ajouté un commit pour virer le dico et ajouter le commentaire que tu suggères.

En fait je n'aime plus trop ce dictionnaire `workflow_tests_data`, je le trouve trop verbeux, ça se voulait un genre de namespace pour indiquer que les attributs sont liés aux tests de workflow, mais puisqu'ils ne sont utilisés que dans ce cadre, c'est assez inutile. J'ai ajouté un commit pour virer le dico et ajouter le commentaire que tu suggères.
vdeniaud force-pushed wip/86955-testdef-workflow-execute-deux-fo from b467a26d67 to 8b4f236350 2024-02-15 17:14:27 +01:00 Compare
vdeniaud force-pushed wip/86955-testdef-workflow-execute-deux-fo from 8b4f236350 to 0daf0b454f 2024-02-15 17:16:34 +01:00 Compare
fpeters approved these changes 2024-02-15 18:14:30 +01:00
fpeters left a comment
Owner
Copy Link

ok mais attention typo dans un des messages de commit : "workfow_tests".

ok mais attention typo dans un des messages de commit : "workfow_tests".
fpeters force-pushed wip/86955-testdef-workflow-execute-deux-fo from 0daf0b454f to 5f2efc2c3c 2024-02-16 11:00:30 +01:00 Compare
fpeters merged commit f6725183d5 into main 2024-02-16 11:11:21 +01:00
fpeters deleted branch wip/86955-testdef-workflow-execute-deux-fo 2024-02-16 11:11:21 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#1140
No description provided.
Delete Branch "wip/86955-testdef-workflow-execute-deux-fo"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?