remove empty string from target parsing

Santiago-Labs · Apr 19, 2024 · 6ac9fd8 · 6ac9fd8
1 parent 1054947
commit 6ac9fd8
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 4 deletions.
diff --git a/cmd/deploy.go b/cmd/deploy.go
@@ -38,12 +38,13 @@ var deployCmd = &cobra.Command{
 			return
 		}
 		var consoleUI runner.ConsoleUI
+		parsedTargets := filterEmptyStrings(strings.Split(targets, ","))
 		if useTUI {
 			consoleUI = runner.NewTUI()
-			go ProcessOrgEndToEnd(consoleUI, resourceoperation.Deploy, strings.Split(targets, ","))
+			go ProcessOrgEndToEnd(consoleUI, resourceoperation.Deploy, parsedTargets)
 		} else {
 			consoleUI = runner.NewSTDOut()
-			ProcessOrgEndToEnd(consoleUI, resourceoperation.Deploy, strings.Split(targets, ","))
+			ProcessOrgEndToEnd(consoleUI, resourceoperation.Deploy, parsedTargets)
 		}
 
 		consoleUI.Start()

diff --git a/cmd/diff.go b/cmd/diff.go
@@ -29,12 +29,14 @@ var diffCmd = &cobra.Command{
 			return
 		}
 		var consoleUI runner.ConsoleUI
+		parsedTargets := filterEmptyStrings(strings.Split(targets, ","))
+
 		if useTUI {
 			consoleUI = runner.NewTUI()
-			go ProcessOrgEndToEnd(consoleUI, resourceoperation.Diff, strings.Split(targets, ","))
+			go ProcessOrgEndToEnd(consoleUI, resourceoperation.Diff, parsedTargets)
 		} else {
 			consoleUI = runner.NewSTDOut()
-			ProcessOrgEndToEnd(consoleUI, resourceoperation.Diff, strings.Split(targets, ","))
+			ProcessOrgEndToEnd(consoleUI, resourceoperation.Diff, parsedTargets)
 		}
 
 		consoleUI.Start()

diff --git a/cmd/root.go b/cmd/root.go
@@ -166,3 +166,13 @@ func resolveMgmtAcct(
 	}
 	return fetchedMgmtAcct, nil
 }
+
+func filterEmptyStrings(slice []string) []string {
+	var result []string
+	for _, str := range slice {
+		if str != "" {
+			result = append(result, str)
+		}
+	}
+	return result
+}