Merge pull request #273 from mgrisole/fix/immediate-interval-start

fix no immediate execution with interval

Author: Hexagon

src/croner.ts

@@ -284,8 +284,6 @@ class Cron {
    * @param prev Starting date, defaults to now - minimum interval
    */
   public msToNext(prev?: CronDate | Date | string): number | null {
-    prev = prev || new Date();
-
     // Get next run time
     const next = this._next(prev);
 

test/croner.test.ts

@@ -1020,3 +1020,11 @@ test("Fire-once should be supported by date, future and .nextRun() should return
   assertEquals(nextRun && nextRun?.getTime() > refTime.getTime(), true);
   assertEquals(nextRun && nextRun?.getTime() < refTime.getTime() + 4000, true);
 });
+
+test('Number of milliseconds to next run should be < 1s when interval is paired with a PerSecondJob', () => {
+  const cron = new Cron("* * * * * *", {
+    interval: 5,
+  })
+  assertEquals(cron.msToNext()! < 1000, true);
+});
+