feat(utils): add robust SplitList utility with unit tests (Fixes litmuschaos#555)

This supersedes stale PRs litmuschaos#564 and litmuschaos#686 by introducing a centralized, tested utility.

- Added SplitList function to pkg/utils/stringutils to safely handle comma-separated strings
- Added unit tests covering edge cases (empty strings, trailing commas, spaces, only commas)
- Verified fix for issue where empty input returned slice of length 1

Author: xnaveenx

pkg/utils/stringutils/string.go

@@ -2,6 +2,7 @@ package stringutils
 
 import (
 	"math/rand"
+	"strings"
 	"time"
 )
 
@@ -12,6 +13,21 @@ const (
 	letterIdxMax  = 63 / letterIdxBits   // # of letter indices fitting in 63 bits
 )
 
+func SplitList(s string) []string {
+	if s == "" {
+		return []string{}
+	}
+	parts := strings.Split(s, ",")
+	var result []string
+	for _, p := range parts {
+		trimmed := strings.TrimSpace(p)
+		if trimmed != "" {
+			result = append(result, trimmed)
+		}
+	}
+	return result
+}
+
 func GetRunID() string {
 	return RandStringBytesMask(6, rand.NewSource(time.Now().UnixNano()))
 }

pkg/utils/stringutils/string_test.go

@@ -64,3 +64,53 @@ func TestGetRunID(t *testing.T) {
 		}
 	}
 }
+
+func TestSplitList(t *testing.T) {
+	tests := []struct {
+		name     string
+		input    string
+		expected []string
+	}{
+		{
+			name:     "simple comma seperation",
+			input:    "pod1,pod2,pod3",
+			expected: []string{"pod1", "pod2", "pod3"},
+		},
+		{
+			name:     "seperation with spaces",
+			input:    " pod1 , pod2 , pod3 ",
+			expected: []string{"pod1", "pod2", "pod3"},
+		},
+		{
+			name:     "empty string",
+			input:    "",
+			expected: []string{},
+		},
+		{
+			name:     "string with empty elements",
+			input:    "pod1,,pod2, ,",
+			expected: []string{"pod1", "pod2"},
+		},
+		{
+			name:	  "only commas input",
+			input:    ",,,",
+			expected: []string{},
+		},
+	}
+	
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			got := SplitList(tt.input)
+			if len(got) != len(tt.expected) {
+				t.Errorf("SplitList(%q) length =%d; want %d", tt.input, len(got), len(tt.expected))
+				return
+			}
+			
+			for i := range got {
+				if got[i] != tt.expected[i] {
+					t.Errorf("SplitList(%q)[%d] = %q; want %q", tt.input, i, got[i], tt.expected[i])
+				}
+			}
+		})
+	}	
+}