Development #73330
testdef, 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.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-testdef-add-support-for-computed-fields-73330.patch 0001-testdef-add-support-for-computed-fields-73330.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
(trop lourd d'empiler autant de branches sur gitea)
Mis à jour par Valentin Deniaud il y a plus d'un an
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.
Mis à jour par Frédéric Péters il y a plus d'un an
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.
Mis à jour par Valentin Deniaud il y a plus d'un an
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
Mis à jour par Valentin Deniaud il y a environ un an
- Statut changé de Solution proposée à 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.
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de En cours à 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
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution proposée à Solution validée
Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution validée à 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
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
testdef: add support for computed fields (#73330)