default interpreter sub-tests to run in parallel

Signed-off-by: Amir Alavi <amiralavi7@gmail.com>

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

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)

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)

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