Open tokuhirom opened 7 years ago
The bug is around this patch. At this pr, it divides elapsed seconds by seconds in the current month. https://github.com/mesos/chronos/pull/800
The number of days in a month is different for each month. February is until 28th or 29th. January is until the 31st. As a result, the current implementation may not be the expected behavior.
Current test case uses new DateTime()
. It gets current date and time.
In my opinion, that such a test case is not good because the result will be different depending on the date and time of execution.
Please look this code.
The output is following. This output seems strange.
scheduledTime
should after thannow
but it's not in some case.