It actually boggles my mind a little bit that this works. I was thinking it could be interesting to extract this method, and ScheduleRule#percent_overlap and #discount_for into their own DiscountCalculator (or something like that) class with something like `#effective_discount(start,end)`. That would also be much easier to write fast specs for.