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

avoir des utilisateurs de test (#88951) #1355

Merged
vdeniaud merged 5 commits from wip/88951-testdef-avoir-des-utilisateurs-d into main 2024-04-08 12:03:45 +02:00
Conversation 2 Commits 5 Files Changed 16 +543 -60
vdeniaud commented 2024-04-03 14:53:26 +02:00
Owner
Copy Link

Voilà tout le nécessaire pour faire en sorte que les tests ne dépendent plus de vrais utilisateurs.

À noter :

  • À la création d'un test depuis une demande existante, les utilisateurs de test correspondant aux utilisateurs réels (usager et agent) sont créés automatiquement : mais pour qu'une création répétée à partir d'une même demande ne crée pas plusieurs utilisateurs identiques, petite heuristique basée sur le mail pour plutôt récupérer un utilisateur de test existant.
  • Niveau parcours, les vues se trouvent à /backoffice/forms/test-users/, mais l'accès se fait depuis la page des tests d'un formulaire, /backoffice/forms/1/tests/.
  • Il y a une migration pour l'usager lié au test mais pas pour l'agent ni pour l'action clic sur un bouton, car ces bouts sont derrière feature flag, tant pis ça va casser.
  • Il restera à permettre l'import/export des utilisateurs de test, et bloquer l'import d'un test sur un utilisateur manquant (ticket à ouvrir).
Voilà tout le nécessaire pour faire en sorte que les tests ne dépendent plus de vrais utilisateurs. À noter : * À la création d'un test depuis une demande existante, les utilisateurs de test correspondant aux utilisateurs réels (usager et agent) sont créés automatiquement : mais pour qu'une création répétée à partir d'une même demande ne crée pas plusieurs utilisateurs identiques, petite heuristique basée sur le mail pour plutôt récupérer un utilisateur de test existant. * Niveau parcours, les vues se trouvent à /backoffice/forms/test-users/, mais l'accès se fait depuis la page des tests d'un formulaire, /backoffice/forms/1/tests/. * Il y a une migration pour l'usager lié au test mais pas pour l'agent ni pour l'action clic sur un bouton, car ces bouts sont derrière feature flag, tant pis ça va casser. * Il restera à permettre l'import/export des utilisateurs de test, et bloquer l'import d'un test sur un utilisateur manquant (ticket à ouvrir).
vdeniaud added 8 commits 2024-04-03 14:53:27 +02:00
gitea/wcs/pipeline/head There was a failure building this commit Details
58078e2bfd use test_uuid in views
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from 58078e2bfd to 0e903df3ce 2024-04-03 15:38:48 +02:00 Compare
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from 0e903df3ce to 7b35fa3496 2024-04-03 15:52:14 +02:00 Compare
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from 7b35fa3496 to 45319e77b5 2024-04-03 16:11:09 +02:00 Compare
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from 45319e77b5 to 55c23bcd97 2024-04-03 16:48:38 +02:00 Compare
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from 55c23bcd97 to be38df4126 2024-04-03 16:57:05 +02:00 Compare
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from be38df4126 to d1938d460d 2024-04-04 10:22:08 +02:00 Compare
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from d1938d460d to 6ff884f9e4 2024-04-04 10:51:10 +02:00 Compare
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from 6ff884f9e4 to 6c01a892ef 2024-04-04 10:57:00 +02:00 Compare
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from 6c01a892ef to 8d2ddca418 2024-04-04 11:09:17 +02:00 Compare
vdeniaud changed title from WIP: avoir des utilisateurs de test (#88951) to avoir des utilisateurs de test (#88951) 2024-04-04 11:18:39 +02:00
vdeniaud force-pushed wip/88951-testdef-avoir-des-utilisateurs-d from 8d2ddca418 to 647cc0226a 2024-04-04 15:02:34 +02:00 Compare
fpeters reviewed 2024-04-08 10:22:11 +02:00
fpeters left a comment
Owner
Copy Link

Il ne manque pas la partie pour intégrer dans l'export XML d'un test les utilisateurs associés ?

Il ne manque pas la partie pour intégrer dans l'export XML d'un test les utilisateurs associés ?
vdeniaud commented 2024-04-08 11:01:41 +02:00
Author
Owner
Copy Link

Il ne manque pas la partie pour intégrer dans l'export XML d'un test les utilisateurs associés ?

C'est le dernier point de la description, je pense que ça peut être séparé puisque cette PR est déjà conséquente

(actuellement dans l'export on a juste l'uuid de l'utilisateur, ça restera comme ça, ce qu'on va ajouter ensuite c'est un export des utilisateurs à part)

> Il ne manque pas la partie pour intégrer dans l'export XML d'un test les utilisateurs associés ? C'est le dernier point de la description, je pense que ça peut être séparé puisque cette PR est déjà conséquente (actuellement dans l'export on a juste l'uuid de l'utilisateur, ça restera comme ça, ce qu'on va ajouter ensuite c'est un export des utilisateurs à part)
fpeters commented 2024-04-08 11:27:24 +02:00
Owner
Copy Link

C'est le dernier point de la description, je pense que ça peut être séparé puisque cette PR est déjà conséquente

Ok désolé je suis passé à côté.

> C'est le dernier point de la description, je pense que ça peut être séparé puisque cette PR est déjà conséquente Ok désolé je suis passé à côté.
fpeters approved these changes 2024-04-08 11:27:31 +02:00
vdeniaud merged commit 310fb84f0f into main 2024-04-08 12:03:45 +02:00
vdeniaud deleted branch wip/88951-testdef-avoir-des-utilisateurs-d 2024-04-08 12:03:45 +02: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#1355
No description provided.
Delete Branch "wip/88951-testdef-avoir-des-utilisateurs-d"

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?