DevFW-CICD/ingress-nginx-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Refactor isIterable

Browse source
This commit is contained in:
Manuel Alejandro de Brito Fontes 2019-04-08 10:11:44 -04:00
parent 4efe549502
commit 6d02a9e475
No known key found for this signature in database
GPG key ID: 786136016A8BA02A
1 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
internal/sets/match.go
View file

@ -24,18 +24,19 @@ type equalFunction func(e1, e2 interface{}) bool
// Compare checks if the parameters are iterable and contains the same elements
func Compare(listA, listB interface{}, eq equalFunction) bool {
a := reflect.ValueOf(listA)
ok := isIterable(a)
ok := isIterable(listA)
if !ok {
return false
}
b := reflect.ValueOf(listB)
ok = isIterable(b)
ok = isIterable(listB)
if !ok {
return false
}
a := reflect.ValueOf(listA)
b := reflect.ValueOf(listB)
if a.IsNil() && b.IsNil() {
return true
}
@ -91,9 +92,9 @@ func StringElementsMatch(a, b []string) bool {
return Compare(a, b, compareStrings)
}
func isIterable(obj reflect.Value) bool {
func isIterable(obj interface{}) bool {
switch reflect.TypeOf(obj).Kind() {
case reflect.Struct, reflect.Slice, reflect.Array:
case reflect.Slice, reflect.Array:
return true
default:
return false