Fix: Display only unique changes (#8)

Co-authored-by: Vee <[email protected]>

Author: dpatsora

cmd/goci/changes.go

@@ -28,33 +28,35 @@ func actionChanges() error {
 		return err
 	}
 
-	var changedFiles []string
+	var paths []string
 
 	for _, stat := range stats {
-		changedFiles = append(changedFiles, stat.Name)
+		paths = append(paths, stat.Name)
 	}
 
-	if Depth == 0 {
-		fmt.Printf("%s\n", strings.Join(changedFiles, " "))
-	} else {
-		var res []string
-		keys := make(map[string]int)
+	if Depth > 0 {
+		paths = filterUnique(paths, Depth)
+	}
+	fmt.Printf("%s\n", strings.Join(paths, " "))
 
-		for _, f := range changedFiles {
-			if _, ok := keys[f]; !ok {
-				keys[f] = 1
-				filepath := strings.Split(f, "/")
+	return nil
+}
 
-				if len(filepath) >= Depth {
-					filepath = filepath[0:Depth]
-				}
+func filterUnique(changes []string, depth int) []string {
+	var paths []string
+	keys := make(map[string]int)
 
-				res = append(res, strings.Join(filepath, "/"))
-			}
+	for _, f := range changes {
+		filepath := strings.Split(f, "/")
+		if len(filepath) >= depth {
+			filepath = filepath[0:depth]
 		}
 
-		fmt.Printf("%s\n", strings.Join(res, " "))
+		path := strings.Join(filepath, "/")
+		if _, ok := keys[path]; !ok {
+			keys[path] = 1
+			paths = append(paths, path)
+		}
 	}
-
-	return nil
+	return paths
 }

cmd/goci/changes_test.go

@@ -0,0 +1,25 @@
+package main
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestFilterUnique(t *testing.T) {
+	scenarios := []struct {
+		name   string
+		input  []string
+		output []string
+	}{
+		{"unique values", []string{"example/values.yaml", "example/versions.yaml"}, []string{"example"}},
+		{"non-unique values", []string{"example1/test.yaml", "example2/test.yaml"}, []string{"example1", "example2"}},
+		{"unique and non-unique values", []string{"example1", "example1", "example2"}, []string{"example1", "example2"}},
+	}
+
+	for _, test := range scenarios {
+		t.Run(test.name, func(t *testing.T) {
+			assert.Equal(t, test.output, filterUnique(test.input, 1))
+		})
+	}
+}