github/weave-scope
1
0
Fork 0
mirror of https://github.com/weaveworks/scope.git synced 2026-07-21 22:36:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

Make TestSetsMerge() check both ways round

Browse Source
This commit is contained in:
Bryan Boreham
2018-07-09 20:20:13 +00:00
parent 8e305f1ce9
commit d37611a4a3
1 changed files with 15 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
report/sets_test.go
View File

@@ -1,6 +1,7 @@
package report_test
import (
"fmt"
"testing"
"github.com/weaveworks/scope/report"
@@ -29,15 +30,19 @@ func TestSetsMerge(t *testing.T) {
map[string][]string{"a": {"1"}, "b": {"2", "3"}, "c": {"3"}},
},
} {
haveSets := testcase.a.Merge(testcase.b)
have := map[string][]string{}
keys := haveSets.Keys()
for _, k := range keys {
have[k], _ = haveSets.Lookup(k)
}
if !reflect.DeepEqual(testcase.want, have) {
t.Errorf("%+v.Merge(%+v): want %+v, have %+v", testcase.a, testcase.b, testcase.want, have)
}
check(t, fmt.Sprintf("%+v.Merge(%+v)", testcase.a, testcase.b), testcase.a.Merge(testcase.b), testcase.want)
check(t, fmt.Sprintf("%+v.Merge(%+v)", testcase.b, testcase.a), testcase.b.Merge(testcase.a), testcase.want)
}
}
func check(t *testing.T, desc string, haveSets report.Sets, want map[string][]string) {
have := map[string][]string{}
keys := haveSets.Keys()
for _, k := range keys {
have[k], _ = haveSets.Lookup(k)
}
if !reflect.DeepEqual(want, have) {
t.Errorf("%s: want %+v, have %+v", desc, want, have)
}
}