Project

General

Profile

Développement #73330

testdef, support des champs donnée calculée

Added by Valentin Deniaud almost 2 years ago. Updated almost 2 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
12 January 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Associated revisions

Revision 90f85518 (diff)
Added by Valentin Deniaud almost 2 years ago

testdef: add support for computed fields (#73330)

History

#1

Updated by Valentin Deniaud almost 2 years ago

(trop lourd d'empiler autant de branches sur gitea)

#2

Updated by Valentin Deniaud almost 2 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.

#3

Updated by Frédéric Péters almost 2 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.

#4

Updated by Valentin Deniaud almost 2 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

#5

Updated by Valentin Deniaud almost 2 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.

#6

Updated by Robot Gitea almost 2 years ago

  • Status changed from En cours to Solution proposée

Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :

#7

Updated by Robot Gitea almost 2 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 :

#8

Updated by Robot Gitea almost 2 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 :

#9

Updated by Transition automatique almost 2 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#10

Updated by Transition automatique over 1 year ago

Automatic expiration

Also available in: Atom PDF