Merge pull request #42 from bcwaldon/values

feat(set): Add set.Values() method

Author: bcwaldon

dbus/set.go

@@ -21,6 +21,13 @@ func (s *set) Length() (int) {
 	return len(s.data)
 }
 
+func (s *set) Values() (values []string) {
+	 for val, _ := range s.data {
+		values = append(values, val)
+	 }
+	 return
+}
+
 func newSet() (*set) {
 	return &set{make(map[string] bool)}
 }

dbus/set_test.go

@@ -18,9 +18,22 @@ func TestBasicSetActions(t *testing.T) {
 		t.Fatal("set should contain 'foo'")
 	}
 
+	v := s.Values()
+	if len(v) != 1 {
+		t.Fatal("set.Values did not report correct number of values")
+	}
+	if v[0] != "foo" {
+		t.Fatal("set.Values did not report value")
+	}
+
 	s.Remove("foo")
 
 	if s.Contains("foo") {
 		t.Fatal("set should not contain 'foo'")
 	}
+
+	v = s.Values()
+	if len(v) != 0 {
+		t.Fatal("set.Values did not report correct number of values")
+	}
 }