luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

Don't pass in sm, server manager is already in scope 

Go closures are implicitly capturing lambdas.
This commit is contained in:
Sean Chittenden 2016-03-25 14:10:09 -07:00
parent 2713899a5b
commit b44554f882
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
consul/server_manager/server_manager_test.go
View File

@ -245,7 +245,7 @@ func TestServerManager_RemoveServer(t *testing.T) {
t.Fatalf("Expected %d servers", maxServers)
}
findServer := func(sm *server_manager.ServerManager, server *server_details.ServerDetails) bool {
findServer := func(server *server_details.ServerDetails) bool {
for i := sm.NumServers(); i > 0; i-- {
s := sm.FindServer()
if s == server {
@ -269,7 +269,7 @@ func TestServerManager_RemoveServer(t *testing.T) {
if sm.NumServers() != expectedNumServers {
t.Fatalf("Expected %d servers (got %d)", expectedNumServers, sm.NumServers())
}
if findServer(sm, server) == true {
if findServer(server) == true {
t.Fatalf("Did not expect to find server %s after removal from the front", server.Name)
}
removedServers = append(removedServers, server)
@ -284,7 +284,7 @@ func TestServerManager_RemoveServer(t *testing.T) {
if sm.NumServers() != expectedNumServers {
t.Fatalf("Expected %d servers (got %d)", expectedNumServers, sm.NumServers())
}
if findServer(sm, server) == true {
if findServer(server) == true {
t.Fatalf("Did not expect to find server %s", server.Name)
}
removedServers = append(removedServers, server)
@ -302,7 +302,7 @@ func TestServerManager_RemoveServer(t *testing.T) {
if sm.NumServers() != expectedNumServers {
t.Fatalf("Expected %d servers (got %d)", expectedNumServers, sm.NumServers())
}
if findServer(sm, server) == true {
if findServer(server) == true {
t.Fatalf("Did not expect to find server %s", server.Name)
}
removedServers = append(removedServers, server)