Calendar availability incorrect

[orenyk]

This issue relates to the broken calendar view noted below; the issues with renewals were fixed in the #416 branch.

---

While writing tests for renewals for #416 I came across a weird bug. The calendar / Reservations doesn't consider the equipment occupied on the due date, and neither does the renewal code. Steps to replicate:

1. Create reservation from Time.zone.today until Time.zone.today + 1.day (for equipment model with a single equipment item)
2. Check out said reservation
3. Create reservation for same equipment model from Time.zone.today + 2.days until Time.zone.today + 3.days
4. Visit reservation page of first reservation
   • Expected: reservation cannot be renewed
   • Actual: reservation can be "renewed" until due date; clicking the link issues a valid renewal request but nothing changes
5. Visit the equipment model page
   • Expected: equipment has no availability from today until Time.zone.today + 4.days in the calendar
   • Actual: equipment is unavailable today, and for the duration of the upcoming reservation, but is considered available on its due date

This might be resolved in #462, or it might not 😛.

Screenshots

[orenyk]

Note that when I tried to create a reservation starting on the due date I'm pretty sure the cart validation for availability failed... I'd have to double-check though.

[orenyk]

Another note: manually editing the second reservation to start on the due date of the first fixes the calendar issue but not the renewal issue.

[orenyk]

Ok, so I've figured part of this out. The eligible_for_renew? method only checks based on how many times its been renewed and how close it is to the due date, not if there are actually days available to extend the reservation. I'm inclined to add a check to the method for equipment_model.available_count(due_date + 1.day) > 0 and hopefully that will fix things. I'll probably implement this in the #416 branch so I can write those specs.

[orenyk]

Verified - a reservation is no longer eligible_for_renew? if there is no equipment available on the day following its due date. This also prevents the renew method from happening if find_renewal_date simply returns the current due date (which is how it behaves if it can't find an eligible renewal date following the due date). I'll leave this issue open to resolve the JS weirdness.