Développement #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
Files
Associated revisions
History
Updated by Valentin Deniaud over 2 years ago
- File 0001-api-ignore-guardian_external_id-when-child-has-no-cu.patch 0001-api-ignore-guardian_external_id-when-child-has-no-cu.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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 »).
Updated by Lauréline Guérin over 2 years ago
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 ?
Updated by Valentin Deniaud over 2 years ago
- File 0001-api-ignore-guardian_external_id-when-child-has-no-cu.patch 0001-api-ignore-guardian_external_id-when-child-has-no-cu.patch added
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.
Updated by Lauréline Guérin over 2 years ago
- Status changed from Solution proposée to Solution validée
Updated by Valentin Deniaud over 2 years ago
- Status changed from Solution validée to 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)Updated by Transition automatique over 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
api: ignore guardian_external_id when child has no custody agenda (#66828)