copy var instead of removing Parallel

Signed-off-by: zachaller <[email protected]>
argoproj · Jul 28, 2023 · 8e028f0 · 8e028f0
1 parent 07ebff2
commit 8e028f0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/pkg/apis/rollouts/validation/validation_references_test.go b/pkg/apis/rollouts/validation/validation_references_test.go
@@ -484,7 +484,9 @@ func TestValidateRolloutReferencedResourcesNginxIngress(t *testing.T) {
 	}
 
 	for _, test := range tests {
+		test := test
 		t.Run(test.name, func(t *testing.T) {
+			t.Parallel()
 			var ingresses []ingressutil.Ingress
 			for i, service := range test.services {
 				ingress := extensionsIngress(test.ingresses[i], 80, service)
@@ -612,7 +614,9 @@ func TestValidateRolloutReferencedResourcesAlbIngress(t *testing.T) {
 	}
 
 	for _, test := range tests {
+		test := test
 		t.Run(test.name, func(t *testing.T) {
+			t.Parallel()
 			var ingresses []ingressutil.Ingress
 			for i, service := range test.services {
 				ingress := extensionsIngress(test.ingresses[i], 80, service)

diff --git a/rollout/trafficrouting/nginx/nginx_test.go b/rollout/trafficrouting/nginx/nginx_test.go
@@ -577,7 +577,9 @@ func TestCanaryIngressAdditionalAnnotations(t *testing.T) {
 func TestReconciler_canaryIngress(t *testing.T) {
 	tests := generateMultiIngressTestData()
 	for _, test := range tests {
+		test := test
 		t.Run(test.name, func(t *testing.T) {
+			t.Parallel()
 			// given
 			r := Reconciler{
 				cfg: ReconcilerConfig{