Write tests for multiple year schedules

ryanjm / rails-event-manager

Repeating schedule manager Rails.

MIT License

0 stars 0 forks source link

Open ryanjm opened 10 years ago

ryanjm commented 10 years ago

None of the tests currently cross the year. Need to write tests to make sure that works properly.

Matt said it currently doesn't.

ryanjm commented 9 years ago

This line should be:

DateTime.new(start.year, start.month, month_days.first,0,0,0, start.offset).next_month

ryanjm commented 9 years ago

That actually won't be right since it could return a date with the day being different than requested.

Need to write a few tests:

Test the case where the following month has less than the requested month_days (Feb has 28). Should return the following month.
Test to make sure that crossing years works properly
Test to make sure that monthly intervals work (test 2 and 3 intervals), especially crossing months that might not match (Feb)