add a test for correctly checking the final minute of a skipped DST r…

…ange.

lib/time.js

@@ -536,6 +536,14 @@ function CronTime(luxon) {
  /**
  * Component of checking if a CronJob time existed in a DateTime range skipped by DST.
  * This subroutine assumes the jump touches at least 2 hours, but the jump does not necessarily fully contain these hours.
+ *
+ * @see _checkTimeInSkippedRange
+ *
+ * This is done by defining the minutes to check for the first and last hour,
+ * and checking all 60 minutes for any hours in between them.
+ *
+ * If any hour x minute combination is a valid time, true is returned.
+ * The endMinute x endHour combination is only checked with the 0th second, since the rest would be out of the range.
  */
  _checkTimeInSkippedRangeMultiHour: function (
  startMinute,

tests/crontime.test.js

@@ -439,6 +439,20 @@ describe('crontime', () => {
  jobInRange = cronTime._checkTimeInSkippedRange(startDate, endDate);
  expect(jobInRange).toEqual(true);
  });
+ it('Should not include seconds in the minute after the DST jump as part of the jump scan', () => {
+ const endDate = luxon.DateTime.fromISO('2023-01-01T16:00:00.000', {
+ zone: 'Europe/Amsterdam'
+ });
+ // 1 hour jump case
+ let startDate = endDate.minus({ hour: 1, second: 1 });
+ const cronTime = new cron.CronTime('1 5 16 * * *'); // at 16:00:01.
+ let jobInRange = cronTime._checkTimeInSkippedRange(startDate, endDate);
+ expect(jobInRange).toEqual(false);
+ // 'quirky' jump case
+ startDate = endDate.minus({ hour: 1, minute: 45, second: 1 });
+ jobInRange = cronTime._checkTimeInSkippedRange(startDate, endDate);
+ expect(jobInRange).toEqual(false);
+ });
  it('Should correctly scan time periods as if they are DST jumps, full hour jumps', () => {
  let endDate = luxon.DateTime.fromISO('2023-01-01T16:00:00.000', {
  zone: 'Europe/Amsterdam'