try atomic.AddUint32() to fix data race in test

coupergateway · Feb 15, 2023 · b0f2a68 · b0f2a68
1 parent 4837cfe
commit b0f2a68
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/server/http_oauth2_test.go b/server/http_oauth2_test.go
@@ -2195,11 +2195,11 @@ func TestTokenRequest_Runtime_Errors(t *testing.T) {
 func Test_OAuth2_Introspection_Caching(t *testing.T) {
 	helper := test.New(t)
 
-	introspectCalled := 0
+	var introspectCalled uint32
 	active := true
 	asOrigin := httptest.NewServer(http.HandlerFunc(func(rw http.ResponseWriter, req *http.Request) {
 		if req.URL.Path == "/introspect" {
-			introspectCalled++
+			atomic.AddUint32(&introspectCalled, 1)
 			rw.Header().Set("Content-Type", "application/json")
 			rw.WriteHeader(http.StatusOK)
 
@@ -2310,11 +2310,11 @@ func Test_OAuth2_Introspection_Caching(t *testing.T) {
 func Test_OAuth2_Introspection_NonCaching(t *testing.T) {
 	helper := test.New(t)
 
-	introspectCalled := 0
+	var introspectCalled uint32
 	active := true
 	asOrigin := httptest.NewServer(http.HandlerFunc(func(rw http.ResponseWriter, req *http.Request) {
 		if req.URL.Path == "/introspect" {
-			introspectCalled++
+			atomic.AddUint32(&introspectCalled, 1)
 			rw.Header().Set("Content-Type", "application/json")
 			rw.WriteHeader(http.StatusOK)