Make featureflag.MockClient return enabled flags in AllEnabledFlags (#190)

kitop · web-flow · commit 8f699f8c1e3c · 2020-10-09T16:33:05.000-03:00
The MockClient should respect the flags set for it.
diff --git a/featureflag/mock.go b/featureflag/mock.go
@@ -32,5 +32,11 @@ func (c MockClient) VariationUser(key string, defaultVal string, _ ld.User) stri
 }
 
 func (c MockClient) AllEnabledFlags(string) []string {
-	return []string{}
+	var res []string
+	for key, value := range c.BoolVars {
+		if value {
+			res = append(res, key)
+		}
+	}
+	return res
 }

Original file line number	Diff line number	Diff line change
`@@ -32,5 +32,11 @@ func (c MockClient) VariationUser(key string, defaultVal string, _ ld.User) stri`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`func (c MockClient) AllEnabledFlags(string) []string {`
`35`		`- return []string{}`
	`35`	`+ var res []string`
	`36`	`+ for key, value := range c.BoolVars {`
	`37`	`+ if value {`
	`38`	`+ res = append(res, key)`
	`39`	`+ }`
	`40`	`+ }`
	`41`	`+ return res`
`36`	`42`	`}`