feat(api): do not allow areas, links or thermals deletion when referenced in a binding constraint

[mabw-rte]

context:
When an object is deleted while being referenced in a binding constraint, this may cause the study to become corrupt.

Acceptance tests

List of endpoints affected by the changes and to be tested:

• PUT /v1/studies/{uuid}/table-mode/binding-constraints: Update binding constraints from Table Mode,
• PUT /v1/studies/{uuid}/bindingconstraints/{binding_constraint_id}: Update binding constraint,
• POST /v1/studies/{uuid}/bindingconstraints: Create a binding constraint,
• DELETE /v1/studies/{uuid}/areas/{area_id}/clusters/thermal: Remove thermal clusters for a given area,
• DELETE /v1//studies/{uuid}/links/{area_from}/{area_to}: Delete a link,
• DELETE /v1/studies/{uuid}/areas/{area_id}: Delete an area.

[MartinBelthle]

This code prevents the front-end to remove links/areas/clusters but not the scripts as they use the commands.

Looking at the code I don't think there's a nice way to implement it (PUT /commands on a variant only add the command to the variant commands list without any verification for instance).

I think we can keep this how it is and not support the verification for /commands, what do you think @laurent-laporte-pro ?

[laurent-laporte-pro]

@mabw-rte

J'ai effectué un refactoring au niveau de la gestion des exceptions pour n'avoir qu'une seule classe d'exception BindingConstraintDeletionNotAllowed : il y avait de la duplication de code. Aussi, j'ai un peu changé le message d'erreur.

Au niveau des fonctions qui contrôlent les objets référencés par des contraintes couplantes, j'ai amélioré ou ajouté une docstring. Le raise a aussi été adapté en fonction des paramètres du constructeur de l'exception BindingConstraintDeletionNotAllowed (ajout du paramètre object_type).

J'ai ajouté des tests unitaires pour avoir une meilleure couverture de tests, notamment dans le cas de la suppression d'une zone.

J'ai fait aussi quelques corrections annexes qui m'ont été remontées par SonarCloud.