Split destinations by commas to handle comma-delimited lists (#94)

Signed-off-by: Josh Russett <[email protected]>

models/security_groups.go

@@ -109,7 +109,12 @@ func (rule SecurityGroupRule) validateDestinations() error {
 
 	var validationError ValidationError
 
+	var destinations []string
 	for _, d := range rule.Destinations {
+		destinations = append(destinations, strings.Split(d, ",")...)
+	}
+
+	for _, d := range destinations {
 		n := strings.IndexAny(d, "-/")
 		if n == -1 {
 			if net.ParseIP(d) == nil {

models/security_groups_test.go

@@ -167,6 +167,7 @@ var _ = Describe("SecurityGroupRule", func() {
 						Expect(validationErr).To(MatchError(ContainSubstring("destination")))
 					})
 				})
+
 			})
 		}
 
@@ -262,6 +263,16 @@ var _ = Describe("SecurityGroupRule", func() {
 				})
 			})
 
+			Context("when it's a comma-delimited list of ips", func() {
+				BeforeEach(func() {
+					destination = "1.2.3.4,5.6.7.8"
+				})
+
+				It("passes validation and does not return an error", func() {
+					Expect(validationErr).NotTo(HaveOccurred())
+				})
+			})
+
 			Context("when its not valid", func() {
 				BeforeEach(func() {
 					destination = "8.8"