Merge pull request #216 from therealmitchconnors/elegant

Implement SliceValue for better list semantics

1 files changed, 23 insertions, 0 deletions

diff --git a/string_slice_test.go b/string_slice_test.go
index c41f3bd..9692461 100644
--- a/string_slice_test.go
+++ b/string_slice_test.go
@@ -251,3 +251,26 @@ func TestSSWithSquareBrackets(t *testing.T) {
 		}
 	}
 }
+
+func TestSSAsSliceValue(t *testing.T) {
+	var ss []string
+	f := setUpSSFlagSet(&ss)
+
+	in := []string{"one", "two"}
+	argfmt := "--ss=%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{"three"})
+		}
+	})
+	if len(ss) != 1 || ss[0] != "three" {
+		t.Fatalf("Expected ss to be overwritten with 'three', but got: %s", ss)
+	}
+}