Development #66828
garde partagée, ignorer le paramètre guardian_external_id si il n'y a pas d'agenda de garde
0%
Description
J'ai été vite en besogne en écrivant que c'était bon dans #66332, certes on ne lève pas d'erreur mais on ne renvoie aucun créneau...
Comportement souhaité :- ?guardian_external_id=xxx&user_external_id=yyy ne doit pas avoir d'impact si yyy n'est pas dans un agenda de garde
- dès qu'un agenda de garde existe pour yyy, une valeur inconnue pour guardian_external_id ne doit renvoyer aucun créneau
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-api-ignore-guardian_external_id-when-child-has-no-cu.patch 0001-api-ignore-guardian_external_id-when-child-has-no-cu.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Valentin Deniaud a écrit :
- dès qu'un agenda de garde existe pour yyy, une valeur inconnue pour guardian_external_id ne doit renvoyer aucun créneau
Le test pour ça existe déjà plus bas (« # unknown guardian »).
Mis à jour par Valentin Deniaud il y a presque 2 ans
Mis à jour par Lauréline Guérin il y a presque 2 ans
le test modifié dans tests/api/fillslot/test_events_multiple_agendas.py, test_api_events_fillslots_multiple_agendas_shared_custody, supprime le child, et donc le SharedCustodyAgenda est supprimé en cascade.
Le bon test serait pas plutôt de changer le user_external_id de child, et voir ce qui se passe ?
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-api-ignore-guardian_external_id-when-child-has-no-cu.patch 0001-api-ignore-guardian_external_id-when-child-has-no-cu.patch ajouté
Lauréline Guerin a écrit :
le test modifié dans tests/api/fillslot/test_events_multiple_agendas.py,
test_api_events_fillslots_multiple_agendas_shared_custody, supprime le child, et donc le SharedCustodyAgenda est supprimé en cascade.
Le bon test serait pas plutôt de changer le user_external_id de child, et voir ce qui se passe ?
Oui tu pointes à raison que le test de base était pas clair, mais si je change le principe du test on ne comprend plus trop ce qu'il se passe à la lecture du diff. Je tente un mix des deux en recopiant le commentaire pour bien montrer qu'on teste le même cas côté datetimes et fillslot.
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 7ba7ace9de3e5e50d3d31431393a22ca811185bd
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Jun 30 11:24:25 2022 +0200
api: ignore guardian_external_id when child has no custody agenda (#66828)Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api: ignore guardian_external_id when child has no custody agenda (#66828)