Merge pull request #78 from utilitywarehouse/dl-unique-dirs

Remove helper functions

Author: DTLP

cmd/kustomize-build-dirs/main.go

@@ -127,35 +127,6 @@ func checkKustomizeInstalled() error {
 	return nil
 }
 
-// splitPath takes a full file path string and returns a slice of its directory components.
-// It extracts the directory portion of the path and splits it by forward slashes (/).
-// Example: "aaa/bbb/ccc/file.yaml" => ["aaa", "bbb", "ccc"]
-func splitPath(path string) []string {
-	cleaned := filepath.ToSlash(filepath.Dir(path))
-	if cleaned == "." {
-		return []string{}
-	}
-	return strings.Split(cleaned, "/")
-}
-
-// commonPrefix compares two string slices (representing directory segments)
-// and returns their longest shared prefix.
-// Example: ["aaa", "bbb", "ccc"], ["aaa", "bbb", "ddd"] => ["aaa", "bbb"]
-func commonPrefix(a, b []string) []string {
-	minLen := len(a)
-	if len(b) < minLen {
-		minLen = len(b)
-	}
-	out := make([]string, 0, minLen)
-	for i := 0; i < minLen; i++ {
-		if a[i] != b[i] {
-			break
-		}
-		out = append(out, a[i])
-	}
-	return out
-}
-
 // deepestCommonDirs takes a list of file paths and returns a list of directory paths
 // that represent the deepest common directory for each group of similarly prefixed files
 // with a minimum directory depth enforced.

cmd/kustomize-build-dirs/main_test.go

@@ -477,45 +477,6 @@ resources:
 	compareResults(t, outDir, expectedContents, readOutDir(t, outDir))
 }
 
-func TestSplitPath(t *testing.T) {
-	tests := []struct {
-		input    string
-		expected []string
-	}{
-		{"file.yaml", []string{}},
-		{"aaa/bbb/file.yaml", []string{"aaa", "bbb"}},
-		{"aaa/bbb/ccc/file.yaml", []string{"aaa", "bbb", "ccc"}},
-		{"./aaa/bbb/ccc/file.yaml", []string{"aaa", "bbb", "ccc"}},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.input, func(t *testing.T) {
-			got := splitPath(tt.input)
-			require.Equal(t, tt.expected, got)
-		})
-	}
-}
-
-func TestCommonPrefix(t *testing.T) {
-	tests := []struct {
-		a, b     []string
-		expected []string
-	}{
-		{[]string{}, []string{"aaa"}, []string{}},
-		{[]string{"aaa"}, []string{}, []string{}},
-		{[]string{"aaa", "bbb"}, []string{"aaa", "bbb"}, []string{"aaa", "bbb"}},
-		{[]string{"aaa", "bbb"}, []string{"ccc", "ddd"}, []string{}},
-		{[]string{"aaa", "bbb", "ccc"}, []string{"aaa", "bbb", "ddd"}, []string{"aaa", "bbb"}},
-	}
-
-	for _, tt := range tests {
-		t.Run("", func(t *testing.T) {
-			got := commonPrefix(tt.a, tt.b)
-			require.Equal(t, tt.expected, got)
-		})
-	}
-}
-
 func TestDeepestCommonDirs(t *testing.T) {
 	tests := []struct {
 		name     string