Merge pull request #85 from arangodb/bug-fix/version-check-fix

fix - /api/version will answer on all servers (not leader only)
arangodb · Mar 28, 2018 · 3f35016 · 3f35016
2 parents e8d594d + 983a072
commit 3f35016
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/tests/test_util.go b/tests/test_util.go
@@ -379,8 +379,10 @@ func waitUntilArangoDeploymentHealthy(deployment *api.ArangoDeployment, DBClient
 				}
 			}
 
-			if goodResults != 1 || noLeaderResults != *deployment.Spec.Single.Count-1 {
-				return maskAny(fmt.Errorf("Wrong number of results: good %d - noleader %d", goodResults, noLeaderResults))
+			expectedGood := *deployment.Spec.Single.Count
+			expectedNoLeader := 0
+			if goodResults != expectedGood || noLeaderResults != expectedNoLeader {
+				return maskAny(fmt.Errorf("Wrong number of results: good %d (expected: %d)- noleader %d (expected %d)", goodResults, expectedGood, noLeaderResults, expectedNoLeader))
 			}
 		}
 	default: