Projet

Général

Profil

Development #88176

Test de formulaire - devrait détecter quand une condition n'est plus ok car l'item a été modifié

Ajouté par Anaïs Ecuvillon → en congés, retour le 30/04 il y a environ un mois. Mis à jour il y a environ un mois.

Statut:
Solution déployée
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
14 mars 2024
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

testé avec un client que je formais à ça, et bam, je tombe sur le mauvais exemple car ce n'était pas pris en compte.
Pourtant, ledit client m'indique que c'est parmi les dysfonctionnements qu'ils rencontrent, ce type d'erreur représente la majorité de leur cas.

Et donc, j'explique, j'ai un champ de type liste avec une liste simple : poire, pomme, autre
J'ai une condition d'affichage du champ autre sur le champ de type liste, genre

form_var_liste == "autre" 

Je modifie ma liste simple, je mets désormais des majuscules Poire, Pomme, Autre
La condition d'affichage de mon champ est fausse, pourtant le test ne le détecte pas. Ce serait utile qu'il le détecte.

Bien évidemment, ça voudrait dire que ça serait également utile qu'il détecte les données d'une source de données.

Révisions associées

Révision 17ae2751 (diff)
Ajouté par Valentin Deniaud il y a environ un mois

testdef: stop ignoring item selection error when no data source (#88176)

Historique

#1

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

  • Assigné à changé de Valentin Deniaud à Anaïs Ecuvillon → en congés, retour le 30/04

Anaïs Ecuvillon a écrit :

La condition d'affichage de mon champ est fausse, pourtant le test ne le détecte pas. Ce serait utile qu'il le détecte.

Pour moi le test doit planter sur une erreur du type « Tentative de remplir le champ xxx mais celui-ci est caché », ce n'est pas le cas ?

Oui ce qu'il se passe c'est que le test continue à saisir la valeur « autre » et donc la condition passe.

Ce qu'il faudrait ici c'est planter plus en amont, sur une erreur genre « Choix invalide ».

Ce n'est pas le cas actuellement car pour les sources de données, on ne veut pas être trop strict sur le choix qui peut disparaître.

Mais on pourrait avoir le fonctionnement voulu sur le cas liste simple, je vais voir ce qui est faisable.

#2

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

  • Assigné à changé de Anaïs Ecuvillon → en congés, retour le 30/04 à Valentin Deniaud
#3

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

  • Statut changé de Nouveau à En cours

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

#4

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

  • Statut changé de En cours à Solution proposée
#5

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

  • 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 :

#6

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

  • Statut changé de Solution validée à Résolu (à déployer)

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

#7

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

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

Formats disponibles : Atom PDF