From 5425c7e31309fc6285e7b925aa5464ef0ecab052 Mon Sep 17 00:00:00 2001
From: John Roesler <johnrroesler@gmail.com>
Date: Wed, 26 Nov 2025 11:56:36 -0600
Subject: [PATCH] fix: calling start multiple times should no-op

---
 scheduler.go      |  5 +++++
 scheduler_test.go | 32 ++++++++++++++++++++++++++++++++
 2 files changed, 37 insertions(+)

diff --git a/scheduler.go b/scheduler.go
index 382c55f..c57f826 100644
--- a/scheduler.go
+++ b/scheduler.go
@@ -824,6 +824,11 @@ func (s *scheduler) RemoveJob(id uuid.UUID) error {
 }
 
 func (s *scheduler) Start() {
+	if s.started.Load() {
+		s.logger.Warn("gocron: scheduler already started")
+		return
+	}
+
 	select {
 	case <-s.shutdownCtx.Done():
 	case s.startCh <- struct{}{}:
diff --git a/scheduler_test.go b/scheduler_test.go
index e040576..302f8b2 100644
--- a/scheduler_test.go
+++ b/scheduler_test.go
@@ -578,6 +578,38 @@ func TestScheduler_Shutdown(t *testing.T) {
 	})
 }
 
+func TestScheduler_Start(t *testing.T) {
+	defer verifyNoGoroutineLeaks(t)
+
+	t.Run("calling start multiple times is a no-op", func(t *testing.T) {
+		s := newTestScheduler(t)
+
+		var counter int
+
+		_, err := s.NewJob(
+			DurationJob(
+				100*time.Millisecond,
+			),
+			NewTask(
+				func() {
+					counter++
+				},
+			),
+		)
+		require.NoError(t, err)
+
+		s.Start()
+		s.Start()
+		s.Start()
+
+		time.Sleep(1000 * time.Millisecond)
+
+		require.NoError(t, s.Shutdown())
+
+		assert.Contains(t, []int{9, 10}, counter)
+	})
+}
+
 func TestScheduler_NewJob(t *testing.T) {
 	defer verifyNoGoroutineLeaks(t)
 	tests := []struct {