Merge pull request #2165 from idealhack/tests

test: fix ineffectual assignments
This commit is contained in:
Chris Telfer 2018-05-29 10:51:23 -04:00 committed by GitHub
commit 55567d88e4

View file

@ -48,6 +48,9 @@ func TestAddedCallback(t *testing.T) {
if !added {
t.Fatal("Expecting an Added callback notification. But none received")
}
if removed {
t.Fatal("Not expecting a Removed callback notification. But received a callback")
}
}
func TestRemovedCallback(t *testing.T) {
@ -58,6 +61,9 @@ func TestRemovedCallback(t *testing.T) {
added := false
removed := false
hd.processCallback(update, func() {}, func(hosts []net.IP) { added = true }, func(hosts []net.IP) { removed = true })
if added {
t.Fatal("Not expecting an Added callback notification. But received a callback")
}
if !removed {
t.Fatal("Expecting a Removed callback notification. But none received")
}