Bug #70851
toulouse-maelis: corrections
0%
Description
Correction des incohérences sur le connecteur.
Fichiers
Révisions associées
toulouse-maelis: remove child payload check (#70851)
toulouse-maelis: correct family payload check (#70851)
toulouse-maelis: invert oneOf schema order to get explicit error messages (#70851)
toulouse-maelis: do not allow empty payload (#70851)
Historique
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier 0005-toulouse-maelis-do-not-allow-empty-payload-70851.patch 0005-toulouse-maelis-do-not-allow-empty-payload-70851.patch ajouté
- Fichier 0004-toulouse-maelis-invert-oneOf-schema-order-to-get-exp.patch 0004-toulouse-maelis-invert-oneOf-schema-order-to-get-exp.patch ajouté
- Fichier 0003-toulouse-maelis-correct-family-payload-check-70851.patch 0003-toulouse-maelis-correct-family-payload-check-70851.patch ajouté
- Fichier 0002-toulouse-maelis-correct-child-payload-check-70851.patch 0002-toulouse-maelis-correct-child-payload-check-70851.patch ajouté
- Fichier 0001-toulouse-maelis-re-add-update_family-deleted-test-70.patch 0001-toulouse-maelis-re-add-update_family-deleted-test-70.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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
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 :)
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)
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.
- via updateChildAutorization (chemin implémenté dans le connecteur) :
personList[] -> personInfo -> Person3Bean (num non requis)
https://demo-toulouse.sigec.fr/maelisws-toulouse/doc/family.html#bean-Person3Bean
- via CreateFamily ou updateFamily (chemin non pris en compte actuellement par le connecteur) :
childList -> authorizedPersonList[] -> personInfo -> PersonInfoBean (num requis)
https://demo-toulouse.sigec.fr/maelisws-toulouse/doc/family.html#bean-PersonInfoBean
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)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
toulouse-maelis: re-add update_family deleted test (#70851)