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

testdef: use XML in import/export (#86587) #1096

Merged
vdeniaud merged 6 commits from wip/86587-testdef-inclure-les-tests-dans-l into main 2024-02-13 10:30:12 +01:00
Conversation 0 Commits 6 Files Changed 10 +254 -102
vdeniaud commented 2024-02-06 11:31:37 +01:00
Owner
Copy Link

Une PR indépendante du reste des devs en cours (test de wf, webservice).

Pas mal de trucs, il y a d'abord correction de la mécanique d'import/export existante :

  • 0001 : retrait d'un champ slug quasiment pas utilisé, il permettait une mise à jour automatique mais c'est un comportement trop particulier dans le cadre de wcs donc incompréhensible, et relou à maintenir. Si besoin on imaginera plus tard un bouton « Écraser » comme on a pour les formulaires, en attendant importer deux fois un export de test crée deux tests, là encore pareil que pour les formulaires.
  • 0002 : correction d'un bug où importer un test d'un formulaire A sur un formulaire B allait créer le test dans le formulaire A.

Ensuite avec cet import/export existant plus propre, on peut passer au sujet du ticket :

  • 0003 : switch de JSON à XML, au passage ça ajoute à l'export des nouveaux champs qui avaient été oublié, et les prochains seront inclus automatiquement.
  • 0004 : inclusion des tests dans l'export d'un formdef. Ici le truc un peu bancal c'est que l'import/export du formulaire est joué à plein d'endroits, notamment lors des snapshots, mais ça ne me semble pas utile d'avoir tout le temps les tests pris dedans, du coup restriction au seul cas d'export via l'UI.
  • 0005 : trivial
  • 0006 : inclusion des tests lors de la duplication.
Une PR indépendante du reste des devs en cours (test de wf, webservice). Pas mal de trucs, il y a d'abord correction de la mécanique d'import/export existante : * 0001 : retrait d'un champ slug quasiment pas utilisé, il permettait une mise à jour automatique mais c'est un comportement trop particulier dans le cadre de wcs donc incompréhensible, et relou à maintenir. Si besoin on imaginera plus tard un bouton « Écraser » comme on a pour les formulaires, en attendant importer deux fois un export de test crée deux tests, là encore pareil que pour les formulaires. * 0002 : correction d'un bug où importer un test d'un formulaire A sur un formulaire B allait créer le test dans le formulaire A. Ensuite avec cet import/export existant plus propre, on peut passer au sujet du ticket : * 0003 : switch de JSON à XML, au passage ça ajoute à l'export des nouveaux champs qui avaient été oublié, et les prochains seront inclus automatiquement. * 0004 : inclusion des tests dans l'export d'un formdef. Ici le truc un peu bancal c'est que l'import/export du formulaire est joué à plein d'endroits, notamment lors des snapshots, mais ça ne me semble pas utile d'avoir tout le temps les tests pris dedans, du coup restriction au seul cas d'export via l'UI. * 0005 : trivial * 0006 : inclusion des tests lors de la duplication.
vdeniaud added 1 commit 2024-02-06 11:31:37 +01:00
gitea/wcs/pipeline/head There was a failure building this commit Details
901f6f0a07 testdef: use XML in import/export (#86587)
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from 901f6f0a07 to a09fd81020 2024-02-06 12:20:02 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from a09fd81020 to 5f5da26708 2024-02-06 13:39:38 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from 5f5da26708 to 45b33c3162 2024-02-06 17:05:47 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from 45b33c3162 to 28e0abe8c8 2024-02-06 17:18:15 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from 28e0abe8c8 to 448c1798da 2024-02-06 17:22:42 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from 448c1798da to f33accbfb1 2024-02-06 17:25:11 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from f33accbfb1 to afd1daba1b 2024-02-06 17:42:58 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from afd1daba1b to 2f3ad0b646 2024-02-06 17:44:43 +01:00 Compare
vdeniaud changed title from WIP: testdef: use XML in import/export (#86587) to testdef: use XML in import/export (#86587) 2024-02-06 17:45:07 +01:00
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from 2f3ad0b646 to 1a1589c1a5 2024-02-06 17:56:12 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from 1a1589c1a5 to 9904ecb1f6 2024-02-07 15:14:12 +01:00 Compare
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from 9904ecb1f6 to dfa8293528 2024-02-07 15:24:39 +01:00 Compare
fpeters approved these changes 2024-02-13 10:04:50 +01:00
fpeters left a comment
Owner
Copy Link

Ok, conflit sur sql.py mais ça me semble juste être un numéro de migration à incrémenter.

Ok, conflit sur sql.py mais ça me semble juste être un numéro de migration à incrémenter.
vdeniaud force-pushed wip/86587-testdef-inclure-les-tests-dans-l from dfa8293528 to b9732bbf4f 2024-02-13 10:20:16 +01:00 Compare
vdeniaud merged commit 4479c301f8 into main 2024-02-13 10:30:12 +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#1096
No description provided.
Delete Branch "wip/86587-testdef-inclure-les-tests-dans-l"

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?