style: add mutex to avoid warning on race condition

ha/lock_test.go

@@ -3,6 +3,7 @@ package ha
 import (
 	"context"
 	"errors"
+	"sync"
 	"testing"
 	"time"
 
@@ -16,11 +17,15 @@ type mockLock struct {
 	locked        bool
 	acquiresCalls map[string]int
 	renewsCounter int
+	mu            sync.Mutex
 
 	leaseStartTime time.Time
 }
 
 func (ml *mockLock) Acquire(_ context.Context, callerID string) (string, error) {
+	ml.mu.Lock()
+	defer ml.mu.Unlock()
+
 	ml.acquiresCalls[callerID] += 1
 	if ml.locked {
 		return "", nil
@@ -32,6 +37,9 @@ func (ml *mockLock) Acquire(_ context.Context, callerID string) (string, error)
 }
 
 func (ml *mockLock) Release(_ context.Context) error {
+	ml.mu.Lock()
+	defer ml.mu.Unlock()
+
 	if !ml.locked {
 		return errors.New("error")
 	}
@@ -45,6 +53,9 @@ func (ml *mockLock) Release(_ context.Context) error {
 // the lock work, its intended to test the behavior of the
 // multiple instances running.
 func (ml *mockLock) Renew(_ context.Context) error {
+	ml.mu.Lock()
+	defer ml.mu.Unlock()
+
 	if !ml.locked {
 		return errors.New("error")
 	}
@@ -104,7 +115,7 @@ func TestAcquireLock_MultipleInstances(t *testing.T) {
 		logger:        testlog.HCLogger(t),
 		renewalPeriod: time.Duration(float64(lease) * renewalFactor),
 		waitPeriod:    time.Duration(float64(lease) * waitFactor),
-		randomDelay:   5 * time.Millisecond,
+		randomDelay:   6 * time.Millisecond,
 	}
 
 	must.False(t, l.locked)