month first weekday bug (#628)

go-co-op · Dec 7, 2023 · f5461a9 · f5461a9
1 parent 0512078
commit f5461a9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/scheduler.go b/scheduler.go
@@ -828,7 +828,7 @@ func (s *Scheduler) MonthFirstWeekday(weekday time.Weekday) *Scheduler {
 		return s.Cron(fmt.Sprintf("0 0 %d %d %d", day, month, weekday))
 	}
 
-	return s.Cron(fmt.Sprintf("0 0 %d %d %d", day, month+1, weekday))
+	return s.Cron(fmt.Sprintf("0 0 %d %d %d", day, (month+1)%12, weekday))
 }
 
 // LimitRunsTo limits the number of executions of this job to n.

diff --git a/scheduler_test.go b/scheduler_test.go
@@ -2263,6 +2263,11 @@ func TestScheduler_EveryMonthFirstWeekday(t *testing.T) {
 			expected: time.Date(2022, time.March, 7, 0, 0, 0, 0, time.UTC),
 			weekday:  time.Monday,
 		},
+		{
+			current:  time.Date(2022, time.December, 7, 0, 0, 0, 0, time.UTC),
+			expected: time.Date(2023, time.January, 2, 0, 0, 0, 0, time.UTC),
+			weekday:  time.Monday,
+		},
 	}
 
 	for _, tc := range testCases {