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

support actions globales dans les tests (#88311) #1285

Merged
vdeniaud merged 1 commits from wip/88311-testdef-support-des-actions-glob into main 2024-03-21 15:20:59 +01:00
Conversation 0 Commits 1 Files Changed 4 +109 -10
vdeniaud commented 2024-03-19 11:51:24 +01:00
Owner
Copy Link
No description provided.
vdeniaud added 1 commit 2024-03-19 11:51:25 +01:00
gitea/wcs/pipeline/head There was a failure building this commit Details
7c88ef8f1f wip
vdeniaud force-pushed wip/88311-testdef-support-des-actions-glob from 7c88ef8f1f to 08b9846f21 2024-03-19 12:08:49 +01:00 Compare
vdeniaud force-pushed wip/88311-testdef-support-des-actions-glob from 08b9846f21 to 4f83587ee8 2024-03-19 12:15:04 +01:00 Compare
vdeniaud reviewed 2024-03-19 13:49:31 +01:00
wcs/workflows.py Outdated
@ -2734,3 +2734,3 @@
raise ReplayException()
for action in filled.formdef.workflow.get_global_actions_for_user(filled, user):
if 'button-action-%s' % action.id in get_request().form:
if form.get_submit() == 'button-action-%s' % action.id:
vdeniaud commented 2024-03-19 13:49:31 +01:00
Author
Owner
Copy Link

Ce code était écrit comme cela depuis l'introduction des actions globales en 2015 (1dd5e103ba), changement peut-être un peu dangereux, en attendant dans l'espoir que ça soit équivalent ça m'évite une acrobatie pour mocker get_request().form

Ce code était écrit comme cela depuis l'introduction des actions globales en 2015 (1dd5e103ba397266c8b30574509db51ee09c14d2), changement peut-être un peu dangereux, en attendant dans l'espoir que ça soit équivalent ça m'évite une acrobatie pour mocker get_request().form
vdeniaud changed title from WIP: support actions globales dans les tests (#88311) to support actions globales dans les tests (#88311) 2024-03-19 13:49:50 +01:00
vdeniaud changed title from support actions globales dans les tests (#88311) to WIP: support actions globales dans les tests (#88311) 2024-03-19 17:03:08 +01:00
vdeniaud force-pushed wip/88311-testdef-support-des-actions-glob from 4f83587ee8 to c2af884f89 2024-03-20 15:36:37 +01:00 Compare
vdeniaud changed title from WIP: support actions globales dans les tests (#88311) to support actions globales dans les tests (#88311) 2024-03-20 15:38:28 +01:00
fpeters requested changes 2024-03-21 10:31:30 +01:00
Dismissed
wcs/workflow_tests.py Outdated
@ -275,0 +282,4 @@
x.name
for x in (formdef.workflow.global_actions or [])
if x.id == trace.event_args['global_action_id']
][0]
fpeters commented 2024-03-21 10:31:24 +01:00
Owner
Copy Link

J'excluerais les actions globales interactives (action.is_interactive() is False), qui demandent il me semble une gestion différente.

J'excluerais les actions globales interactives (action.is_interactive() is False), qui demandent il me semble une gestion différente.
vdeniaud commented 2024-03-21 10:43:51 +01:00
Author
Owner
Copy Link

Dac, à noter que actuellement puisqu'on passe par perform_items, on bénéficie pareil de l'exclusion des actions actuellement non supportées (formulaire, commentaire...)

Donc ici dans le cas action interactive ça permet juste de tester que le bouton est affiché, et l'action ne fera rien (sauf si bizarrement on a envoi de mail puis formulaire, alors on pourra tester l'envoi de mail, mais j'imagine que ce cas n'existe pas)

Dac, à noter que actuellement puisqu'on passe par perform_items, on bénéficie pareil de l'exclusion des actions actuellement non supportées (formulaire, commentaire...) Donc ici dans le cas action interactive ça permet juste de tester que le bouton est affiché, et l'action ne fera rien (sauf si bizarrement on a envoi de mail puis formulaire, alors on pourra tester l'envoi de mail, mais j'imagine que ce cas n'existe pas)
vdeniaud commented 2024-03-21 14:04:08 +01:00
Author
Owner
Copy Link

Ajouté le is_interactive

Ajouté le is_interactive
vdeniaud force-pushed wip/88311-testdef-support-des-actions-glob from c2af884f89 to 5b21f17e75 2024-03-21 11:16:34 +01:00 Compare
vdeniaud force-pushed wip/88311-testdef-support-des-actions-glob from 5b21f17e75 to eca7a12199 2024-03-21 11:30:23 +01:00 Compare
vdeniaud requested review from fpeters 2024-03-21 14:03:32 +01:00
fpeters approved these changes 2024-03-21 14:55:59 +01:00
vdeniaud force-pushed wip/88311-testdef-support-des-actions-glob from eca7a12199 to 9afbbccb13 2024-03-21 15:07:58 +01:00 Compare
vdeniaud merged commit 9afbbccb13 into main 2024-03-21 15:20:59 +01:00
vdeniaud deleted branch wip/88311-testdef-support-des-actions-glob 2024-03-21 15:20:59 +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#1285
No description provided.
Delete Branch "wip/88311-testdef-support-des-actions-glob"

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?