Fix data race in syncer test

syncer_test.go

@@ -5,11 +5,11 @@
 package asynq
 
 import (
+	"fmt"
 	"sync"
 	"testing"
 	"time"
 
-	"github.com/go-redis/redis/v7"
 	h "github.com/hibiken/asynq/internal/asynqtest"
 	"github.com/hibiken/asynq/internal/base"
 	"github.com/hibiken/asynq/internal/rdb"
@@ -50,54 +50,43 @@ func TestSyncer(t *testing.T) {
 }
 
 func TestSyncerRetry(t *testing.T) {
-	inProgress := []*base.TaskMessage{
-		h.NewTaskMessage("send_email", nil),
-		h.NewTaskMessage("reindex", nil),
-		h.NewTaskMessage("gen_thumbnail", nil),
-	}
-	goodClient := setup(t)
-	h.SeedInProgressQueue(t, goodClient, inProgress)
-
-	// Simulate the situation where redis server is down
-	// by connecting to a wrong port.
-	badClient := redis.NewClient(&redis.Options{
-		Addr: "localhost:6390",
-	})
-	rdbClient := rdb.NewRDB(badClient)
-
 	const interval = time.Second
 	syncRequestCh := make(chan *syncRequest)
 	syncer := newSyncer(syncRequestCh, interval)
+
 	var wg sync.WaitGroup
 	syncer.start(&wg)
 	defer syncer.terminate()
 
-	for _, msg := range inProgress {
+	var (
-		m := msg
+		mu      sync.Mutex
-		syncRequestCh <- &syncRequest{
+		counter int
-			fn: func() error {
+	)
-				return rdbClient.Done(m)
+
-			},
+	// Increment the counter for each call.
+	// Initial call will fail and second call will succeed.
+	requestFunc := func() error {
+		mu.Lock()
+		defer mu.Unlock()
+		if counter == 0 {
+			counter++
+			return fmt.Errorf("zero")
 		}
+		counter++
+		return nil
 	}
 
-	time.Sleep(2 * interval) // ensure that syncer runs at least once
+	syncRequestCh <- &syncRequest{
-
+		fn:     requestFunc,
-	// Sanity check to ensure that message was not successfully deleted
+		errMsg: "error",
-	// from in-progress list.
-	gotInProgress := h.GetInProgressMessages(t, goodClient)
-	if l := len(gotInProgress); l != len(inProgress) {
-		t.Errorf("%q has length %d; want %d", base.InProgressQueue, l, len(inProgress))
 	}
 
-	// FIXME: This assignment introduces data race and running the test with -race will fail.
+	// allow syncer to retry
-	// simualate failover.
+	time.Sleep(3 * interval)
-	rdbClient = rdb.NewRDB(goodClient)
 
-	time.Sleep(2 * interval) // ensure that syncer runs at least once
+	mu.Lock()
-
+	if counter != 2 {
-	gotInProgress = h.GetInProgressMessages(t, goodClient)
+		t.Errorf("counter = %d, want 2", counter)
-	if l := len(gotInProgress); l != 0 {
-		t.Errorf("%q has length %d; want 0", base.InProgressQueue, l)
 	}
+	mu.Unlock()
 }