Merge pull request #17 from devjoes/master

fix: operator precedence issue results in invalid index

Author: saschat

client.go

@@ -124,7 +124,7 @@ func (c *Client) getServer(key string) (*server, error) {
 	}
 	nServers := uint(len(c.servers))
 	for i := uint(0); i < nServers; i++ {
-		s := c.servers[idx+i%nServers]
+		s := c.servers[(idx+i)%nServers]
 		if s.isAlive {
 			return s, nil
 		}

client_test.go

@@ -1606,3 +1606,12 @@ func TestStatsReset(t *testing.T) {
 		break // in case there are multiple servers
 	}
 }
+
+func TestGetServer(t *testing.T) {
+	c := NewMC(mcAddr+","+badAddr, user, pass)
+	c.servers[1].isAlive = false
+
+	s, e := c.getServer("ok")
+	assertEqualf(t, nil, e, "err was not nil")
+	assertEqualf(t, mcAddr, s.address, "address of incorrect server returned")
+}