Revert "test: avoid data race"

This reverts commit f372969.

Author: wolfogre

modules/globallock/locker_test.go

@@ -29,6 +29,11 @@ func TestLocker(t *testing.T) {
 		oldExpiry := redisLockExpiry
 		redisLockExpiry = 5 * time.Second // make it shorter for testing
 		defer func() {
+			// Avoid data race.
+			// The startExtend goroutine may still be running and reading redisLockExpiry.
+			// Wait for a while since it will be stopped soon after Close is called.
+			time.Sleep(time.Second)
+
 			redisLockExpiry = oldExpiry
 		}()
 

modules/globallock/redis_locker.go

@@ -26,9 +26,8 @@ var redisLockExpiry = 30 * time.Second
 type redisLocker struct {
 	rs *redsync.Redsync
 
-	mutexM   sync.Map
-	closed   atomic.Bool
-	extendWg sync.WaitGroup
+	mutexM sync.Map
+	closed atomic.Bool
 }
 
 var _ Locker = &redisLocker{}
@@ -41,8 +40,6 @@ func NewRedisLocker(connection string) Locker {
 			),
 		),
 	}
-
-	l.extendWg.Add(1)
 	l.startExtend()
 
 	return l
@@ -71,7 +68,6 @@ func (l *redisLocker) TryLock(ctx context.Context, key string) (bool, context.Co
 // But it's useful in tests to release resources.
 func (l *redisLocker) Close() error {
 	l.closed.Store(true)
-	l.extendWg.Wait()
 	return nil
 }
 
@@ -123,7 +119,6 @@ func (l *redisLocker) lock(ctx context.Context, key string, tries int) (context.
 
 func (l *redisLocker) startExtend() {
 	if l.closed.Load() {
-		l.extendWg.Done()
 		return
 	}