Fix issue #120 - Not-fixed times recurring job bug

ajvb / kala

Modern Job Scheduler

MIT License

2.13k stars 188 forks source link

Fix issue #120 - Not-fixed times recurring job bug #121

Closed MarcoVigna17 closed 8 years ago

MarcoVigna17 commented 8 years ago

Issue: #120

Fixed the bug adding a function to check if a job has fixed repetitions before checking stats len and timesToRepeat to avoid problem in case of recurring job (timesToRepeat = -1). With tests.

Special thanks to @benwaine that help me on tests!

coveralls commented 8 years ago

Coverage increased (+1.4%) to 80.541% when pulling 6c7b0825842cf6e0ea2beead78557dec021dbb63 on flypay:fix-issue-120 into bc173ddaf1be46142935088fde62d729b81aea1d on ajvb:master.

ajvb commented 8 years ago

Looks good to me! Thanks @MarcoVigna17 and @benwaine