too many loops in #next_time/previous_time for invalid dates

[hspazio]

Issue description

This is again a known issue reporting as per error message. Although the input date is incorrect (e.g. Feb 31st) the :parse method returns a valid object and the loop :next_time reaches the max number of iterations.

How to reproduce

require 'fugit'
c = Fugit.parse('0 12 31 2 *')
p c.next_time

Same issue occurs with all inputs that have month within 1-12 and day within 1-31 but are not valid dates:

• Feb 30th
• Feb 31st
• Apr 31st
• Jun 31st
• Sep 31st
• Nov 31st

Suggestions

• Could the parse method return nil directly when invalid combination of month-day is detected?

[jmettraux]

gh-20

[jmettraux]

https://crontab.guru/#*_*_31_2_* is valid, crond probably never schedules (and doesn't fail with a RuntimeError).

[jmettraux]

Thanks!

[hspazio]

Thank you @jmettraux! much appreciated your quick turnaround! :)

[jmettraux]

You're welcome :-)