Feat/datasets distributions

[EmmanuelDemey]

No description provided.

[EmmanuelDemey]

@FBibonne que penses tu de créer un ControllerAdvice pour la gestion des exceptions dans les controllers ? plutot que la classe parent GenericResources. Cela permet d'éviter le try catch à chaque fois.
Je peux le supprimer si cela ne convient pas ;)

d532c6c

[FBibonne]

@FBibonne que penses tu de créer un ControllerAdvice pour la gestion des exceptions dans les controllers ? plutot que la classe parent GenericResources. Cela permet d'éviter le try catch à chaque fois. Je peux le supprimer si cela ne convient pas ;)

d532c6c

Ok pour moi : c'est une bonne idée qui pourrait être généralisée.

Par contre, il faudrait faire des tests plus globaux (tests d'intégration avec une serveur web émulé) pour vérifier que lorsqu'une erreur se produit, le serveur renvoie bien une 500 via le mécanisme du ControllerAdvice.

En fait, comme le code pour le retour dans le cas du 200 se répète également, on pourrait presque faire une seule méthode par controller !

Pour ces deux points, je propose de regarder la semaine prochaine pour proposer quelque chose en sus de la PR afin de ne pas la bloquer.