Project

General

Profile

Development #66828

garde partagée, ignorer le paramètre guardian_external_id si il n'y a pas d'agenda de garde

Added by Valentin Deniaud about 2 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
30 June 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 7ba7ace9 (diff)
Added by Valentin Deniaud about 1 month ago

api: ignore guardian_external_id when child has no custody agenda (#66828)

History

#1

Updated by Valentin Deniaud about 1 month ago

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 »).

#3

Updated by Lauréline Guérin (de retour le 29/8) about 1 month 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 ?

#4

Updated by Valentin Deniaud about 1 month ago

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.

#5

Updated by Lauréline Guérin (de retour le 29/8) about 1 month ago

  • Status changed from Solution proposée to Solution validée
#6

Updated by Valentin Deniaud about 1 month 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)
#7

Updated by Transition automatique about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF