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

plus d'actions de test (#88108) #1263

Merged
vdeniaud merged 6 commits from wip/88108-testdef-support-des-actions-manq into main 2024-03-21 11:07:16 +01:00
Conversation 0 Commits 6 Files Changed 10 +628 -20
vdeniaud commented 2024-03-14 11:04:36 +01:00
Owner
Copy Link

5 actions : anonymisation, redirection web, message dans l'historique, alerte, criticité.

Pour message dans l'historique et alerte ça sera intéressant d'avoir un champ pour indiquer qui doit voir le message, ça sera une évolution pour un autre ticket sur le modèle de ce qu'on aura pour indiquer qui clique sur un bouton (#87543).

5 actions : anonymisation, redirection web, message dans l'historique, alerte, criticité. Pour message dans l'historique et alerte ça sera intéressant d'avoir un champ pour indiquer qui doit voir le message, ça sera une évolution pour un autre ticket sur le modèle de ce qu'on aura pour indiquer qui clique sur un bouton (#87543).
vdeniaud force-pushed wip/88108-testdef-support-des-actions-manq from d93ddd144c to 6796b2381e 2024-03-14 11:17:12 +01:00 Compare
vdeniaud force-pushed wip/88108-testdef-support-des-actions-manq from 53e1e8e61b to c1195d8ead 2024-03-14 14:51:13 +01:00 Compare
vdeniaud changed title from WIP: plus d'actions de test (#88108) to plus d'actions de test (#88108) 2024-03-14 15:16:13 +01:00
fpeters reviewed 2024-03-15 14:04:55 +01:00
wcs/workflow_tests.py Outdated
@ -630,0 +825,4 @@
name = None
XML_NODES = WorkflowTestAction.XML_NODES + [
('name', 'str'),
fpeters commented 2024-03-15 14:04:55 +01:00
Owner
Copy Link

Je me suis demandé si on ne ferait pas mieux de stocker l'id de WorkflowCriticalityLevel plutôt que son nom, pour que les tests n'échouent pas sur le changement de libellé, je me dis que c'est mineur mais que ça serait quand même mieux, tu voyais un problème avec ça ?

Je me suis demandé si on ne ferait pas mieux de stocker l'id de WorkflowCriticalityLevel plutôt que son nom, pour que les tests n'échouent pas sur le changement de libellé, je me dis que c'est mineur mais que ça serait quand même mieux, tu voyais un problème avec ça ?
vdeniaud commented 2024-03-18 11:28:03 +01:00
Author
Owner
Copy Link

Si les ids étaient liés à un niveau, pas de problème, mais là puisqu'ils désignent l'index du niveau dans la liste des niveaux, je trouve ça trop fragile.

Pour moi si tu as les niveau Normal et Urgent, un test qui vérifie que la demande est dans le niveau Urgent, et que tu ajoutes un niveau Important entre Normal et Urgent, ce serait un comportement inattendu que l'action de test change de niveau à vérifier pour pointer sur Important.

Aussi c'est homogène avec ce qu'on fait pour les boutons où on stocke effectivement le libellé.

Si les ids étaient liés à un niveau, pas de problème, mais là puisqu'ils désignent l'index du niveau dans la liste des niveaux, je trouve ça trop fragile. Pour moi si tu as les niveau Normal et Urgent, un test qui vérifie que la demande est dans le niveau Urgent, et que tu ajoutes un niveau Important entre Normal et Urgent, ce serait un comportement inattendu que l'action de test change de niveau à vérifier pour pointer sur Important. Aussi c'est homogène avec ce qu'on fait pour les boutons où on stocke effectivement le libellé.
fpeters commented 2024-03-18 11:49:14 +01:00
Owner
Copy Link

Si les ids étaient liés à un niveau, pas de problème, mais là puisqu'ils désignent l'index du niveau dans la liste des niveaux, je trouve ça trop fragile.

Je parle bien de l'attribut "id", pas du numéro qui se trouve dans la db,

class WorkflowCriticalityLevel:
    id = None
    name = None
    colour = None

    def __init__(self, name=None, colour=None):
        self.name = name
        self.colour = colour
        self.id = str(random.randint(0, 100000))
> Si les ids étaient liés à un niveau, pas de problème, mais là puisqu'ils désignent l'index du niveau dans la liste des niveaux, je trouve ça trop fragile. Je parle bien de l'attribut "id", pas du numéro qui se trouve dans la db, ``` class WorkflowCriticalityLevel: id = None name = None colour = None def __init__(self, name=None, colour=None): self.name = name self.colour = colour self.id = str(random.randint(0, 100000)) ```
vdeniaud commented 2024-03-18 15:15:45 +01:00
Author
Owner
Copy Link

Ah oui merci j'avais raté ça, voilà c'est fait dans la branche

Ah oui merci j'avais raté ça, voilà c'est fait dans la branche
vdeniaud force-pushed wip/88108-testdef-support-des-actions-manq from c1195d8ead to 40dd08c603 2024-03-18 11:06:51 +01:00 Compare
vdeniaud force-pushed wip/88108-testdef-support-des-actions-manq from 40dd08c603 to 2b73cc2164 2024-03-18 15:08:04 +01:00 Compare
fpeters approved these changes 2024-03-21 10:29:17 +01:00
vdeniaud force-pushed wip/88108-testdef-support-des-actions-manq from 2b73cc2164 to 6fd4b87ff5 2024-03-21 10:44:07 +01:00 Compare
vdeniaud merged commit 6fd4b87ff5 into main 2024-03-21 11:07:16 +01:00
vdeniaud deleted branch wip/88108-testdef-support-des-actions-manq 2024-03-21 11:07:16 +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#1263
No description provided.
Delete Branch "wip/88108-testdef-support-des-actions-manq"

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?