Development #71867
Simplifier le test de la commande import_template
0%
Description
Introduit par un ticket à 35 notes (#14630), le test va exporter aveuglément tout authentic et regarder si ça correspond à un export hardcodé dans un fichier.
En trois ans il y a eu 10 commits rien que pour éditer ce fichier, toujours en relative urgence parce qu'une modif dans authentic était venue casser le test de manière inattendue.
Il est nécessaire de revoir ça (parce que je suis en train de faire #71844 et ça va repéter ce test).
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Tu pourrais simplement tester que des objets ont été créé en les comptant, à mon avis c'est tout ce que cherche à prouver ce test (que la commande 'import_site' est bien appelée et que ça a bien des conséquences).
L'autre possibilité c'est de charger le dump, le ré-exporter créer un deuxième tenant et faire le test entre ce deuxième export et l'export du deuxième tenant (comme ça pas de variation sur les clés présentes à un moment donné).
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-test_authentic-restrict-scope-of-import_template-com.patch 0001-test_authentic-restrict-scope-of-import_template-com.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Benjamin Dauvergne a écrit :
Tu pourrais simplement tester que des objets ont été créé en les comptant
Pas suffisant, ça casserait toujours à l'ajout d'un rôle d'admin côté a2.
à mon avis c'est tout ce que cherche à prouver ce test (que la commande 'import_site' est bien appelée et que ça a bien des conséquences).
Tout à fait d'accord, du coup réécriture du test pour s'appuyer seulement sur des données explicites et pas des trucs générés d'ailleurs.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
- Assigné à changé de Valentin Deniaud à Benjamin Dauvergne
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Assigné à changé de Benjamin Dauvergne à Valentin Deniaud
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 02df3a7a818a363c9eb3cb73f8491eb3eed7f0d8 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Nov 30 10:36:01 2022 +0100 test_authentic: restrict scope of import_template command test (#71867)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
test_authentic: restrict scope of import_template command test (#71867)