From 49753a9798e111f253345c8976d540164ec013a2 Mon Sep 17 00:00:00 2001
From: Matous Jobanek <mjobanek@redhat.com>
Date: Thu, 3 Aug 2023 17:42:20 +0200
Subject: [PATCH] comments from PR #767

---
 test/e2e/parallel/serviceaccount_test.go | 33 +++++++++++++-----------
 1 file changed, 18 insertions(+), 15 deletions(-)

diff --git a/test/e2e/parallel/serviceaccount_test.go b/test/e2e/parallel/serviceaccount_test.go
index dfdf74afb..864822126 100644
--- a/test/e2e/parallel/serviceaccount_test.go
+++ b/test/e2e/parallel/serviceaccount_test.go
@@ -18,8 +18,6 @@ import (
 
 func TestDoNotOverrideServiceAccount(t *testing.T) {
 	// given
-	// Skipping the TestDoNotOverrideServiceAccount test instead of deleting it because we will need to create SAs as part
-	// of the environment sub-workspaces so the test & logic will be useful to keep.
 	t.Parallel()
 	awaitilities := WaitForDeployments(t)
 	member := awaitilities.Member1()
@@ -68,26 +66,31 @@ func TestDoNotOverrideServiceAccount(t *testing.T) {
 		sa, err := member.WaitForServiceAccount(t, fmt.Sprintf("%s-env", mur.Name), "namespace-manager")
 		require.NoError(t, err)
 		assert.Equal(t, "stay", sa.Annotations["should"])
-	secrets:
-		for j := 0; j < i; j++ {
-			expName := fmt.Sprintf("dummy-secret-%d", j)
+
+		for j := 0; j <= i; j++ {
+			expSecretName := fmt.Sprintf("dummy-secret-%d", j)
+			secretFound := false
 			for _, secretRef := range sa.Secrets {
-				if secretRef.Name == fmt.Sprintf("dummy-secret-%d", j) {
-					continue secrets
+				if secretRef.Name == expSecretName {
+					secretFound = true
+					break
 				}
 			}
-			assert.Fail(t, fmt.Sprintf("secret '%s' not found", expName))
-		}
+			if !secretFound {
+				assert.Fail(t, fmt.Sprintf("secret '%s' not found", expSecretName))
+			}
 
-	pullSecrets:
-		for j := 0; j < i; j++ {
-			expName := fmt.Sprintf("dummy-pull-secret-%d", j)
+			expPullSecretName := fmt.Sprintf("dummy-pull-secret-%d", j)
+			pullSecretFound := false
 			for _, pullSecretRef := range sa.ImagePullSecrets {
-				if pullSecretRef.Name == fmt.Sprintf("dummy-pull-secret-%d", j) {
-					continue pullSecrets
+				if pullSecretRef.Name == expPullSecretName {
+					pullSecretFound = true
+					break
 				}
 			}
-			assert.Fail(t, fmt.Sprintf("pull secret '%s' not found", expName))
+			if !pullSecretFound {
+				assert.Fail(t, fmt.Sprintf("pull secret '%s' not found", expPullSecretName))
+			}
 		}
 
 		// verify that the secrets created for SA is the same