Fixed race condition test (#1866)

* Added go test `-race` option

Signed-off-by: yutachaos <[email protected]>

* Fixed the possibility of race condition when loading observedCount variable.

Signed-off-by: yutachaos <[email protected]>

Author: yutachaos

Makefile

@@ -15,7 +15,7 @@ all: controller
 
 # Run tests
 test: generate fmt vet manifests
-	go test ./pkg/... ./webhooks/... -coverprofile cover.out
+	go test -race ./pkg/... ./webhooks/... -coverprofile cover.out
 
 # Build controller binary
 controller: generate fmt vet

pkg/aws/throttle/throttler_test.go

@@ -195,7 +195,7 @@ func Test_throttler_beforeSign(t *testing.T) {
 				<-testQPSThrottle
 			}
 			elapsed := time.Since(start)
-			tt.validCallsCount(elapsed, observedCount)
+			tt.validCallsCount(elapsed, atomic.LoadInt64(&observedCount))
 			cancel()
 			wg.Wait()
 		})