Scalingtest now always asserts on 10 documents instead of an estimate…

…d value
mesos · Feb 11, 2016 · 2ac7610 · 2ac7610
1 parent 50cfefb
commit 2ac7610
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/...test/src/systemTest/java/org/apache/mesos/elasticsearch/systemtest/ScalingSystemTest.java b/...test/src/systemTest/java/org/apache/mesos/elasticsearch/systemtest/ScalingSystemTest.java
@@ -93,11 +93,11 @@ public void shouldNotLoseDataWhenScalingDown() throws UnirestException {
                 return false;
             }
         });
-        Integer newNumberOfDocuments = Unirest.get("http://" + newAddress.get(0) + "/_count").asJson().getBody().getArray().getJSONObject(0).getInt("count");
+        int newNumberOfDocuments = Unirest.get("http://" + newAddress.get(0) + "/_count").asJson().getBody().getArray().getJSONObject(0).getInt("count");
 
         // Check that the data is still correct
         LOGGER.debug("New number of documents: " + newNumberOfDocuments);
-        assertEquals(correctNumberOfDocuments, newNumberOfDocuments);
+        assertEquals(DataPusherContainer.CORRECT_NUM_DOCS, newNumberOfDocuments);
     }
 
     public void scaleNumNodesTo(String ipAddress, Integer newNumNodes) throws UnirestException {

diff --git a/...emTest/java/org/apache/mesos/elasticsearch/systemtest/containers/DataPusherContainer.java b/...emTest/java/org/apache/mesos/elasticsearch/systemtest/containers/DataPusherContainer.java
@@ -12,7 +12,7 @@
  * Data Pusher container implementation
  */
 public class DataPusherContainer extends AbstractContainer {
-    public static final Integer CORRECT_NUM_DOCS = 10;
+    public static final int CORRECT_NUM_DOCS = 10;
 
     public String pusherImageName = "alexglv/es-pusher";