test step fix

Author: rajguptavm

tests/e2e/multiSvc/multi_svc_test.go

@@ -443,22 +443,6 @@ var _ = ginkgo.Describe("[csi-multi-svc-refactor] Multi-SVC", func() {
 			for _, datastorePath := range datastores {
 				// roleForUser array to store roles for both svc account user based on datastore
 				var roleForUser []string
-
-				// cluster1Group := []string{"vsanDatastore (2)"}
-				// cluster2Group := []string{"vsanDatastore (1)"}
-				// mixedGroup := []string{"nfs0-1", "sharedVmfs_0"}
-				// // remainGroup := []string{"local-0 (8)", "local-0 (9)", "local-0 (10)", "local-0 (11)", "local-0 (12)", "local-0 (13)", "vsanDatastore", "vsanDatastore (3)"}
-				// parts := strings.Split(datastorePath, "/")
-				// datastoreName := parts[len(parts)-1]
-				// if helper_util.ContainsItem(cluster1Group, datastoreName) {
-				// 	roleForUser = []string{constants.RoleCnsDatastore, ""}
-				// } else if helper_util.ContainsItem(cluster2Group, datastoreName) {
-				// 	roleForUser = []string{"", constants.RoleCnsDatastore}
-				// } else if helper_util.ContainsItem(mixedGroup, datastoreName) {
-				// 	roleForUser = []string{constants.RoleCnsDatastore, constants.RoleCnsDatastore}
-				// } else {
-				// 	roleForUser = []string{"", ""}
-				// }
 				switch {
 				case strings.Contains(datastorePath, "vsanDatastore (2)"):
 					roleForUser = []string{constants.RoleCnsDatastore, ""}
@@ -548,7 +532,6 @@ var _ = ginkgo.Describe("[csi-multi-svc-refactor] Multi-SVC", func() {
 		}()
 
 		ginkgo.By("Verify an alarm is raised for unmounted datastore and host in the supervisor cluster 1")
-		// alarm := "Datastore not accessible to all hosts under the cluster"
 		alarm := "Datastore no longer accessible to all hosts in the cluster compute resource"
 		alarmPresent, err = IsAlarmPresentOnDatacenter(ctx, e2eTestConfig, dataCenter, alarm, true)
 		gomega.Expect(err).NotTo(gomega.HaveOccurred())
@@ -673,7 +656,7 @@ var _ = ginkgo.Describe("[csi-multi-svc-refactor] Multi-SVC", func() {
 			gomega.Expect(err).NotTo(gomega.HaveOccurred())
 			isDsMountedOnSvc2 = false
 			ginkgo.By("Verify datastore has permission for storage service account from the svc1 but not from svc2")
-			roleForSvcUser = []string{"", constants.RoleCnsSearchAndSpbm}
+			roleForSvcUser = []string{constants.RoleCnsDatastore, ""}
 			// iterating through service account users
 			for j, user := range wcpServiceAccUsers {
 				userPermission, err := VerifyPermissionForWcpStorageUser(ctx, e2eTestConfig, "Datastore",