Projet

Général

Profil

Bug #70851

toulouse-maelis: corrections

Ajouté par Nicolas Roche il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
30 octobre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Correction des incohérences sur le connecteur.


Fichiers

Révisions associées

Révision c2631bf6 (diff)
Ajouté par Nicolas Roche il y a plus d'un an

toulouse-maelis: re-add update_family deleted test (#70851)

Révision 82136bbc (diff)
Ajouté par Nicolas Roche il y a plus d'un an

toulouse-maelis: remove child payload check (#70851)

Révision c2500791 (diff)
Ajouté par Nicolas Roche il y a plus d'un an

toulouse-maelis: correct family payload check (#70851)

Révision 2a60072e (diff)
Ajouté par Nicolas Roche il y a plus d'un an

toulouse-maelis: invert oneOf schema order to get explicit error messages (#70851)

Révision 87055032 (diff)
Ajouté par Nicolas Roche il y a plus d'un an

toulouse-maelis: do not allow empty payload (#70851)

Historique

#1

Mis à jour par Nicolas Roche il y a plus d'un an

C'est un peu en vrac désolé, mais ça n'ajoute rien de nouveau.
0001: j'ai supprimé ce test par erreur dans #70075
0002: bug passé à l'as (dans #70075), mais pas testable
0003: bug passé à l'as (toujours #70075), parce qu'il manquait un test
0004: inversion de l'ordre sur les "oneOf", pour avoir des messages d'erreurs consistants
0005: remonter les "oneOf" sur les schémas parents pour interdire les payload vides sur les WS qui les utilisent directement

#2

Mis à jour par Thomas Noël il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée

Let's go !

Je note qu'en fait, quand on voit le test :

assert resp.json['err_desc'] == "42 is not of type 'string'"

... on se dit que le bogue c'est ce message en lui même, ça ne devrait pas parler de 42 mais de "description" (le nom de la clé, pas sa valeur), et ça devrait dire « string or null » lors d'un oneOf. Mais c'est pas dans ce connecteur que ça se joue, plutôt dans l'analyseur de schéma, et je vais pas regarder ni même faire de ticket :)

#3

Mis à jour par Benjamin Dauvergne il y a plus d'un an

J'ai un peu du mal avec ce commentaire :

# dead code as not manage on updateFamily that need another schema (having num person field)

#4

Mis à jour par Nicolas Roche il y a plus d'un an

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

Oui, tu as raison, j'ai retiré ce code.
---8<---
J'arrive sur 2 schémas différents suivant le WS utilisé pour spécifier les personnes autorisées à venir chercher les enfants,
et je n'ai pas codé un des 2 chemins (alors que par chance ça aurait pu fonctionner).
Là, je n'ose pas trop introduire ou tester quoi que ce soit parce que Sigec modifie beaucoup son code.

commit 87055032a084e7d65003529266ec35bddf926365
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Sun Oct 23 15:33:38 2022 +0200

    toulouse-maelis: do not allow empty payload (#70851)

commit 2a60072e53fc87c574cf62b2f341e30cfb26ce54
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Sun Oct 23 15:25:46 2022 +0200

    toulouse-maelis: invert oneOf schema order to get explicit error messages (#70851)

commit c25007919b4ac6ea9aaed811a13f0bfb8732d76d
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Sun Oct 23 14:59:16 2022 +0200

    toulouse-maelis: correct family payload check (#70851)

commit 82136bbce68ff748ea8a2279ea41bc1700371f24
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Sun Oct 23 14:27:40 2022 +0200

    toulouse-maelis: remove child payload check (#70851)

commit c2631bf67246484711ce3181a6021622d3daaf3c
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Sun Oct 23 13:35:34 2022 +0200

    toulouse-maelis: re-add update_family deleted test (#70851)
#5

Mis à jour par Transition automatique il y a plus d'un an

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

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF