Projet

Général

Profil

Development #74289

testdef, gestion des sources de données

Ajouté par Valentin Deniaud il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 février 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Actuellement il y a un problème bloquant, si on crée un test sur un formulaire de prise de rendez-vous, il va planter très vite ensuite, parce que le créneau sélectionné au moment de la création du test sera à un moment réservé, ou plus proposé à cause du temps qui passe (le test échouera sur une erreur « Choix invalide »).

Une idée pour remédier à ça pourrait être d'avoir un paramétrage supplémentaire au champ liste en question, dire « sélectionner la première valeur valide », ça résoudrait le cas soulevé.

Resterait qu'en cas d'indispo/dysfonctionnement de la source de donnée, le test aurait toujours des chances de planter.

D'où une autre approche, plutôt rendre le test indépendant du réseau : lors de l'exécution on détecte que le champ est alimenté par une source de données, et on le remplit artificiellement à partir des données enregistrées lors de la création du test, c'est à dire en ajoutant à la saisie en cours les données _structured et compagnie qu'on récupérerait normalement après un appel WS.

On perd en dynamisme car en cas de modification de la source de donnée, on continuera à jouer le test sur de vieilles données et ça finira par planter le test aussi. Mais de manière moins systématique qu'actuellement (et de toute façon en cas de grosse modification à un formulaire on ne coupera jamais à devoir aller modifier les tests).

Il me semble que cette idée permet de lever le point bloquant sus-mentionné, assez facilement et sans complexifier l'UI : je fais le patch ?

Révisions associées

Révision 278058bf (diff)
Ajouté par Valentin Deniaud il y a environ un an

testdef: allow invalid value for item fields (#74289)

Révision 1796ece7 (diff)
Ajouté par Valentin Deniaud il y a environ un an

testdef: statically retrieve values for field with data source (#74289)

Historique

#1

Mis à jour par Valentin Deniaud il y a environ un an

Plus j'y pense plus je suis convaincu qu'il faut tout faire statiquement, ne jamais réinterroger la source de donnée (on a accès à un inspect pour voir les données du test, ça n'a pas de logique de jouer le test sur d'autres données que celles affichées sur cette page).

Donc ce qu'il y a à faire c'est 1/ ignorer qu'un choix est invalide (y compris pour les champs sans source de données, c'est une erreur qui n'a pas d'intérêt) et 2/ stocker les valeurs _display et _structured à la création du test puis les réutiliser.

#2

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à Solution proposée

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

#3

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 :

#4

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 :

#5

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#6

Mis à jour par Transition automatique il y a 12 mois

Automatic expiration

Formats disponibles : Atom PDF