karmada-io
/
karmada
mirror of https://github.com/karmada-io/karmada.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2996 from a7i/interpreter-parallel-tests 

default interpreter sub-tests to run in parallel
This commit is contained in:
karmada-bot 2022-12-29 09:58:51 +08:00 committed by GitHub
parent eda7814ffc 09ca7f506f
commit c34c443d2c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 45 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
pkg/resourceinterpreter/defaultinterpreter/dependencies_test.go
View File

@ -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)
}

28
pkg/resourceinterpreter/defaultinterpreter/healthy_test.go
View File

@ -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)

4
pkg/resourceinterpreter/defaultinterpreter/reflectstatus_test.go
View File

@ -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)

8
pkg/resourceinterpreter/defaultinterpreter/revisereplica_test.go
View File

@ -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")