[WIP] Bugfix : indispo qui termine avant de commencer

[francois-ferrandis]

Note : le nom de la branche n'est pas pertinent, ignorez-le. 😉

Contexte

En travaillant sur une première version de #4976, je tentais de factoriser la validation des champs horaires des plages et absences dans le concern partagé RecurrenceConcern. La spec models/rdv_spec.rb:821 est alors passée au rouge. 🤔 J'ai alors découvert que cette spec crée une absence récurrente qui commence à 10h et finit à 10h, et que cette absence est valide ! 🤯

Comportement actuel

Je peux créer une indispo dont l'heure de début est après l'heure de fin (seulement si j'active l'option de récurrence). Cette indispo est considérée valide et a 0 occurrences.

Comportement attendu

Si je tente de créer une indispo avec une heure de fin antérieure à l'heure de début, le fomulaire affiche une erreur et m'empêche de créer l'indispo.

Solution

Le bug est causé par le fait que RecurrenceConcern#ends_at retourne nil lorsque l'indispo est récurrente. Or, ends_at est utilisée dans la validation sur les heures : si ends_at est nil, alors on ne valide pas les heures. 😉

[francois-ferrandis]

Rubocop m’empêchait d'appeler ma variable absence_from_8_to_9, alors que franchement c'est un super nom ! 😤