Développement #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).
Files
Associated revisions
History
Updated by Benjamin Dauvergne about 2 years ago
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é).
Updated by Valentin Deniaud about 2 years ago
- File 0001-test_authentic-restrict-scope-of-import_template-com.patch 0001-test_authentic-restrict-scope-of-import_template-com.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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.
Updated by Benjamin Dauvergne about 2 years ago
- Status changed from Solution proposée to Solution validée
- Assignee changed from Valentin Deniaud to Benjamin Dauvergne
Updated by Benjamin Dauvergne about 2 years ago
- Assignee changed from Benjamin Dauvergne to Valentin Deniaud
Updated by Valentin Deniaud about 2 years ago
- Status changed from Solution validée to 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)
Updated by Transition automatique about 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
test_authentic: restrict scope of import_template command test (#71867)