feature/converter : return converted messages as a list#372
Conversation
|
|
|
|
…into feature/converter/return-converted-messages-as-list
|
| @@ -8,8 +8,9 @@ | |||
|
|
|||
| def conversion_strategy( | |||
| edxl_json, source_version: str, target_version: str, is_cisu_conversion: bool | |||
There was a problem hiding this comment.
Est ce qu'on peut rajouter un test sur conversion strategy qui mock l'appel à cisu_conversion_strategy ou health_conversion_strategy pour retourner soit une liste soit un seul object pour tester le comportement dans les deux cas ?
|
|
1 similar comment
|
|
🔎 Détails
Cette pull request modifie le format de réponse du point d’entrée /convert du service converter. Dorénavant, l’endpoint retourne la ou les messages convertis sous forme de liste dans la clé "converted_messages", au lieu d’un seul message sous la clé "edxl". Cela Prépare l’API à gérer des réponses multiples par requête dans le futur.
L’API renvoie désormais :
{ "converted_messages": [ ... ] }au lieu de :
{ "edxl": { ... } }Mise à jour de tous les tests unitaires concernés pour :
📄 Documentation
📸 Captures d'écran
🔗 Ticket associé
6. Mise à jour du converter pour retourner une liste de messages en réponse à l’endpoint POST /convert