Skip to content

let agents bypass booking delays in trouver un creneau interface

Vincent Agnano requested to merge feature/let-agents-bypass-booking-delays into master

Created by: adipasquale

cf https://trello.com/c/zc0mkTqD/682-agenttrouver-un-cr%C3%A9neau-rendre-le-d%C3%A9lai-mini-et-maxi-inactif-c%C3%B4t%C3%A9-agent

J'ai ajouté des specs pour la méthode que je modifie et un peu refacto les existantes:

  • moins de create, plus de build (= moins d'acces DB)
  • introduction d'une factory Creneau. ca eloigne un peu le contexte du test @yaf mais ca me parait quand meme mieux pour pas répéter 50 fois les memes choses, et permettre de reutiliser un creneau qui ne respecte pas les booking delays ailleurs ... c'est discutable
  • refacto les fonctions too_late? et too_soon? en respects_max/min_booking_delays (a l'envers donc). plus explicite et moins de doubles negations.
  • ajouté une methode qui regroupe les deux respects_booking_delays

cette PR peut etre lue commit par commit

Merge request reports