Développement #73330
closedtestdef, support des champs donnée calculée
0%
Description
Ces champs sont présents dans les données qu'on récupère pour jouer le test, mais il s'agit de les ignorer et de les recalculer dynamiquement au fur et à mesure qu'on les rencontre.
Files
Updated by Valentin Deniaud almost 3 years ago
- File 0001-testdef-add-support-for-computed-fields-73330.patch 0001-testdef-add-support-for-computed-fields-73330.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
(trop lourd d'empiler autant de branches sur gitea)
Updated by Valentin Deniaud almost 3 years ago
Note, j'ai chopé le code dans forms.root.FormPage.handle_computed_fields, en laissant de côté les with get_publisher().complex_data(): parce que je n'ai pas compris à quoi ils servaient, sûrement que je rate un truc.
Updated by Frédéric Péters almost 3 years ago
Note, j'ai chopé le code dans forms.root.FormPage.handle_computed_fields, en laissant de côté les with get_publisher().complex_data(): parce que je n'ai pas compris à quoi ils servaient, sûrement que je rate un truc.
On peut recevoir des données qui ne sont pas des chaines, par exemple on pourrait avoir en données calculées {{webservice.plop}} qui serait un dictionnaire; il y a un exemple de cette situation dans test_computed_field_complex_data.
Updated by Valentin Deniaud almost 3 years ago
Frédéric Péters a écrit :
On peut recevoir des données qui ne sont pas des chaines
Oui et j'ai écrit un test qui couvre ce cas et qui fonctionne, sans le with
Updated by Valentin Deniaud almost 3 years ago
- Status changed from Solution proposée to En cours
Valentin Deniaud a écrit :
Oui et j'ai écrit un test qui couvre ce cas et qui fonctionne, sans le
with
Mon test était tout à fait faux, le with était bien nécessaire, j'envoie une version corrigée sur gitea.
Updated by Robot Gitea almost 3 years ago
- Status changed from En cours to Solution proposée
Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/70
- Titre : testdef: add support for computed fields (#73330)
- Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/70/files
Updated by Robot Gitea almost 3 years ago
- Status changed from Solution proposée to Solution validée
Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :
Updated by Robot Gitea almost 3 years ago
- Status changed from Solution validée to Résolu (à déployer)
Valentin Deniaud (vdeniaud) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/70
- Titre : testdef: add support for computed fields (#73330)
- Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/70/files
Updated by Transition automatique almost 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée