github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #117110 from pohly/test-integration-race-detection-auth

Browse Source 
test/integration/auth: fix data race
This commit is contained in:
Kubernetes Prow Robot
2023-04-23 22:11:13 -07:00
committed by GitHub
parent 4d70d1df7c a2fb32870f
commit 08755fe249
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
test/integration/auth/accessreview_test.go
View File

@@ -21,6 +21,7 @@ import (
"errors" "errors"
"net/http" "net/http"
"strings" "strings"
"sync"
"testing" "testing"
authorizationapi "k8s.io/api/authorization/v1" authorizationapi "k8s.io/api/authorization/v1"
@@ -148,8 +149,12 @@ func TestSubjectAccessReview(t *testing.T) {
} }
func TestSelfSubjectAccessReview(t *testing.T) { func TestSelfSubjectAccessReview(t *testing.T) {
var mutex sync.Mutex
username := "alice" username := "alice"
authenticatorFunc := func(req *http.Request) (*authenticator.Response, bool, error) { authenticatorFunc := func(req *http.Request) (*authenticator.Response, bool, error) {
mutex.Lock()
defer mutex.Unlock()
return &authenticator.Response{ return &authenticator.Response{
User: &user.DefaultInfo{ User: &user.DefaultInfo{
Name: username, Name: username,
@@ -216,7 +221,9 @@ func TestSelfSubjectAccessReview(t *testing.T) {
} }
for _, test := range tests { for _, test := range tests {
mutex.Lock()
username = test.username username = test.username
mutex.Unlock()
response, err := clientset.AuthorizationV1().SelfSubjectAccessReviews().Create(context.TODO(), test.sar, metav1.CreateOptions{}) response, err := clientset.AuthorizationV1().SelfSubjectAccessReviews().Create(context.TODO(), test.sar, metav1.CreateOptions{})
switch { switch {