admin: allow creating workflow tests from formdata (#87545) #1231

Merged
fpeters merged 1 commits from wip/87545-testdef-alimenter-les-tests-de-w into main 2024-03-08 11:07:43 +01:00
Owner

Création à partir de ce qui est stocké dans les traces d’exécution du workflow, ce qui permet de remplir les actions clic sur un bouton, saut dans le temps, et vérif du statut.

Pour les autres on ne remplit rien, l'action est ajoutée mais elle sera non configurée. Pour faire mieux il faudrait aller chercher les infos dans les évolutions (j'imagine) mais on a rien pour lier une trace à une évolution (je crois), donc on verra plus tard.

Création à partir de ce qui est stocké dans les traces d’exécution du workflow, ce qui permet de remplir les actions clic sur un bouton, saut dans le temps, et vérif du statut. Pour les autres on ne remplit rien, l'action est ajoutée mais elle sera non configurée. Pour faire mieux il faudrait aller chercher les infos dans les évolutions (j'imagine) mais on a rien pour lier une trace à une évolution (je crois), donc on verra plus tard.
vdeniaud added 3 commits 2024-03-06 11:44:04 +01:00
vdeniaud force-pushed wip/87545-testdef-alimenter-les-tests-de-w from e4a898a0c6 to 4737d0794f 2024-03-06 11:56:19 +01:00 Compare
vdeniaud changed title from WIP: admin: allow creating workflow tests from formdata (#87545) to admin: allow creating workflow tests from formdata (#87545) 2024-03-06 11:56:25 +01:00
fpeters reviewed 2024-03-06 12:52:53 +01:00
@ -347,0 +350,4 @@
]
if get_publisher().has_site_option('enable-workflow-tests'):
creation_options.append(
('formdata-wf', _('Import data from form (and initialise workflow tests)'), 'formdata-wf')
Owner

Il restera un intérêt à créer à test depuis une demande mais ne pas initialiser les tests de workflow ?

Je me dis que si le cas très commun est l'initialisation des tests de workflow, on pourrait rester sur deux options ici et libre à la personne qui ne voudrait pas de test de workflow d'aller l'effacer ?

(je n'ai pas d'opinion forte ici, ça me va tout à fait de partir ainsi et de voir)

Il restera un intérêt à créer à test depuis une demande mais ne pas initialiser les tests de workflow ? Je me dis que si le cas très commun est l'initialisation des tests de workflow, on pourrait rester sur deux options ici et libre à la personne qui ne voudrait pas de test de workflow d'aller l'effacer ? (je n'ai pas d'opinion forte ici, ça me va tout à fait de partir ainsi et de voir)
Author
Owner

libre à la personne qui ne voudrait pas de test de workflow d'aller l'effacer ?

Pour le moment on ne peut pas effacer les tests d'un coup, si la création automatique des tests ajoute 50 actions ce sera 50 clics sur supprimer si on en veut pas, donc plutôt utile de permettre de ne pas les créer

> libre à la personne qui ne voudrait pas de test de workflow d'aller l'effacer ? Pour le moment on ne peut pas effacer les tests d'un coup, si la création automatique des tests ajoute 50 actions ce sera 50 clics sur supprimer si on en veut pas, donc plutôt utile de permettre de ne pas les créer
fpeters approved these changes 2024-03-08 09:37:50 +01:00
fpeters left a comment
Owner

Ok (ça me semble ensuite une évolution utile de pouvoir supprimer une série de tests/actions sur les workflows, pour par exemple partir d'une demande existante, puis tout effacer à partir d'un point, pour correspondre à par exemple une erreur webservice).

Ok (ça me semble ensuite une évolution utile de pouvoir supprimer une série de tests/actions sur les workflows, pour par exemple partir d'une demande existante, puis tout effacer à partir d'un point, pour correspondre à par exemple une erreur webservice).
fpeters merged commit 16e844a049 into main 2024-03-08 11:07:43 +01:00
fpeters deleted branch wip/87545-testdef-alimenter-les-tests-de-w 2024-03-08 11:07:43 +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#1231
No description provided.