Merge pull request #216 from therealmitchconnors/elegant

Implement SliceValue for better list semantics

1 files changed, 23 insertions, 0 deletions

diff --git a/string_array_test.go b/string_array_test.go
index 1ceac8c..3c6d595 100644
--- a/string_array_test.go
+++ b/string_array_test.go
@@ -193,6 +193,29 @@ func TestSAWithSpecialChar(t *testing.T) {
 	}
 }
 
+func TestSAAsSliceValue(t *testing.T) {
+	var sa []string
+	f := setUpSAFlagSet(&sa)
+
+	in := []string{"1ns", "2ns"}
+	argfmt := "--sa=%s"
+	arg1 := fmt.Sprintf(argfmt, in[0])
+	arg2 := fmt.Sprintf(argfmt, in[1])
+	err := f.Parse([]string{arg1, arg2})
+	if err != nil {
+		t.Fatal("expected no error; got", err)
+	}
+
+	f.VisitAll(func(f *Flag) {
+		if val, ok := f.Value.(SliceValue); ok {
+			_ = val.Replace([]string{"3ns"})
+		}
+	})
+	if len(sa) != 1 || sa[0] != "3ns" {
+		t.Fatalf("Expected ss to be overwritten with '3ns', but got: %v", sa)
+	}
+}
+
 func TestSAWithSquareBrackets(t *testing.T) {
 	var sa []string
 	f := setUpSAFlagSet(&sa)