rescuedcode/StaffPlan — comment on commit 685f7c0f5dfad1faaf5090e5e47fabdd5bdf3f5e
File:  line 0
By fermion:

Ahh, gotcha. IIRC it should take the current date, shift it to the Monday of that week and pull the two weeks prior + 3.months from that Monday. Depending on they day of the week that could pull a varying number of weeks. Make sense?

On Friday, March 9, 2012 at 9:59 AM, Bertrand Chardon wrote:

> Note that the test on date_range was introduced by my previous commit.
> I thought that the array was always 13-long (one week in the past and three months in the future give or take) but after executing the tests for a few times I got a failure telling me that RSpec was expecting 13 and got 14.
>
> I guess it's always finicky with dates, depending on lots of factors like odd years, alignment on week days etc.
> Maybe something like .size > 12 would be a better fit.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rescuedcode/StaffPlan/commit/685f7c0f5dfad1faaf5090e5e47fabdd5bdf3f5e#commitcomment-1065884
>
>