diff --git a/pkg/resourceinterpreter/defaultinterpreter/dependencies_test.go b/pkg/resourceinterpreter/defaultinterpreter/dependencies_test.go index af809b841..62f701cfd 100644 --- a/pkg/resourceinterpreter/defaultinterpreter/dependencies_test.go +++ b/pkg/resourceinterpreter/defaultinterpreter/dependencies_test.go @@ -44,8 +44,10 @@ func TestGetSecretNames(t *testing.T) { }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() res := getSecretNames(tt.pod) if !reflect.DeepEqual(res, tt.expected) { t.Errorf("getSecretNames() = %v, want %v", res, tt.expected) @@ -91,8 +93,10 @@ func TestGetConfigMapNames(t *testing.T) { }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() res := getConfigMapNames(tt.pod) if !reflect.DeepEqual(res, tt.expected) { t.Errorf("getConfigMapNames() = %v, want %v", res, tt.expected) @@ -136,8 +140,10 @@ func TestGetPVCNames(t *testing.T) { }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() res := getPVCNames(tt.pod) if !reflect.DeepEqual(res, tt.expected) { t.Errorf("getPVCNames() = %v, want %v", res, tt.expected) @@ -194,8 +200,10 @@ func TestGetDependenciesFromPodTemplate(t *testing.T) { }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() res, _ := getDependenciesFromPodTemplate(tt.pod) if !reflect.DeepEqual(res, tt.expected) { t.Errorf("getDependenciesFromPodTemplate() = %v, want %v", res, tt.expected) @@ -224,8 +232,10 @@ func Test_getServiceAccountNames(t *testing.T) { want: sets.NewString(), }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() if got := getServiceAccountNames(tt.args.pod); !reflect.DeepEqual(got, tt.want) { t.Errorf("getServiceAccountNames() = %v, want %v", got, tt.want) } diff --git a/pkg/resourceinterpreter/defaultinterpreter/healthy_test.go b/pkg/resourceinterpreter/defaultinterpreter/healthy_test.go index 01204cf89..76d66ab84 100644 --- a/pkg/resourceinterpreter/defaultinterpreter/healthy_test.go +++ b/pkg/resourceinterpreter/defaultinterpreter/healthy_test.go @@ -106,8 +106,10 @@ func Test_interpretDeploymentHealth(t *testing.T) { wantErr: false, }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() got, err := interpretDeploymentHealth(tt.object) if (err != nil) != tt.wantErr { t.Errorf("interpretDeploymentHealth() err = %v, wantErr %v", err, tt.wantErr) @@ -218,8 +220,10 @@ func Test_interpretStatefulSetHealth(t *testing.T) { wantErr: false, }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() got, err := interpretStatefulSetHealth(tt.object) if (err != nil) != tt.wantErr { t.Errorf("interpretStatefulSetHealth() err = %v, wantErr %v", err, tt.wantErr) @@ -294,8 +298,10 @@ func Test_interpretReplicaSetHealth(t *testing.T) { }, }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() got, err := interpretReplicaSetHealth(tt.object) if (err != nil) != tt.wantErr { t.Errorf("interpretReplicaSetHealth() error = %v, wantErr %v", err, tt.wantErr) @@ -372,8 +378,10 @@ func Test_interpretDaemonSetHealth(t *testing.T) { wantErr: false, }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() got, err := interpretDaemonSetHealth(tt.object) if (err != nil) != tt.wantErr { t.Errorf("interpretDaemonSetHealth() error = %v, wantErr %v", err, tt.wantErr) @@ -445,8 +453,10 @@ func Test_interpretServiceHealth(t *testing.T) { wantErr: false, }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() got, err := interpretServiceHealth(tt.object) if (err != nil) != tt.wantErr { t.Errorf("interpretServiceHealth() error = %v, wantErr %v", err, tt.wantErr) @@ -495,8 +505,10 @@ func Test_interpretIngressHealth(t *testing.T) { wantErr: false, }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() got, err := interpretIngressHealth(tt.object) if (err != nil) != tt.wantErr { t.Errorf("interpretIngressHealth() error = %v, wantErr %v", err, tt.wantErr) @@ -541,8 +553,10 @@ func Test_interpretPersistentVolumeClaimHealth(t *testing.T) { wantErr: false, }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() got, err := interpretPersistentVolumeClaimHealth(tt.object) if (err != nil) != tt.wantErr { t.Errorf("interpretPersistentVolumeClaimHealth() error = %v, wantErr %v", err, tt.wantErr) diff --git a/pkg/resourceinterpreter/defaultinterpreter/reflectstatus_test.go b/pkg/resourceinterpreter/defaultinterpreter/reflectstatus_test.go index 823744c30..7915dfc38 100644 --- a/pkg/resourceinterpreter/defaultinterpreter/reflectstatus_test.go +++ b/pkg/resourceinterpreter/defaultinterpreter/reflectstatus_test.go @@ -57,8 +57,10 @@ func Test_getEntireStatus(t *testing.T) { false, }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() got, err := reflectWholeStatus(tt.args.object) if (err != nil) != tt.wantErr { t.Errorf("reflectWholeStatus() error = %v, wantErr %v", err, tt.wantErr) diff --git a/pkg/resourceinterpreter/defaultinterpreter/revisereplica_test.go b/pkg/resourceinterpreter/defaultinterpreter/revisereplica_test.go index c7499ff11..458c1be70 100644 --- a/pkg/resourceinterpreter/defaultinterpreter/revisereplica_test.go +++ b/pkg/resourceinterpreter/defaultinterpreter/revisereplica_test.go @@ -63,8 +63,10 @@ func TestReviseDeploymentReplica(t *testing.T) { }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() res, err := reviseDeploymentReplica(tt.object, tt.replica) if err == nil && tt.expectError == true { t.Fatal("expect an error but got none") @@ -137,8 +139,10 @@ func TestReviseJobReplica(t *testing.T) { }, } - for _, tt := range tests { + for i := range tests { + tt := tests[i] t.Run(tt.name, func(t *testing.T) { + t.Parallel() res, err := reviseJobReplica(tt.object, tt.replica) if err == nil && tt.expectError == true { t.Fatal("expect an error but got none")