Refactor notification mailers and extract services
Created by: adipasquale
beaucoup de changements mais ces parties du code sont bien testées donc ca devrait être assez peu risqué.
- mailers scopés dans un namespace
users
pour anticiper leagents
parallele - le nom des methodes de mails est harmonisé pour correspondre à un événement :
-
send_ics_to_user
->rdv_created
-
send_notification
->new_creneau_available
- ...
-
- vues des mailers regroupés dans un repertoire parents
views/mailers
- extraction des appels aux Mailers et au job d'envoi de SMS dans des services de
Notifications
- logique de la méthode
rdv#notify?
déplacée dans un service de baseNotifications::Rdv::BaseService
. Cette logique dépend du temps donc je trouve ça bizarre de l'avoir dans le modèle, et ça extrait de la logique du modèle déjà assez costaud. - refacto de quelques tests pour qu'ils ne testent pas au dela de leur scope et mocke bien les objets dépendants