Minor changes to tests

maksred · Sep 6, 2012 · 604324d · 604324d
1 parent da04c85
commit 604324d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/test/unit/voldemort/server/EndToEndTest.java b/test/unit/voldemort/server/EndToEndTest.java
@@ -120,8 +120,7 @@ public void testSanity() {
 
         assertEquals("getAll works as expected", "Moscow", capitals.get("Russia").getValue());
         assertEquals("getAll works as expected", "Kiev", capitals.get("Ukraine").getValue());
-        assertTrue("getAll works as expected",
-                   !capitals.containsKey("Japan") || capitals.get("Japan") == null);
+        assertTrue("getAll works as expected", capitals.get("Japan") == null);
 
         storeClient.delete("Ukraine");
         assertNull("delete works as expected", storeClient.get("Ukraine"));

diff --git a/test/unit/voldemort/store/routed/GetallNodeReachTest.java b/test/unit/voldemort/store/routed/GetallNodeReachTest.java
@@ -249,19 +249,13 @@ public void testGetall_211_zoneCountRead_1() throws Exception {
     public void testGetall_322() throws Exception {
         cluster = getEightNodeClusterWithZones();
         HashMap<Integer, Integer> zoneReplicationFactor = new HashMap<Integer, Integer>();
-        zoneReplicationFactor.put(0, 4);
-        zoneReplicationFactor.put(1, 4);
-        /*
-         * First n nodes on the preference list will be one node from each
-         * remote n zones, where n=zoneCountReads, therefore preferred read
-         * should be set > n if want to include local zone node results in
-         * parallel request
-         */
+        zoneReplicationFactor.put(0, 3);
+        zoneReplicationFactor.put(1, 3);
         storeDef = new StoreDefinitionBuilder().setName("test")
                                                .setType(InMemoryStorageConfiguration.TYPE_NAME)
                                                .setRoutingPolicy(RoutingTier.CLIENT)
                                                .setRoutingStrategyType(RoutingStrategyType.ZONE_STRATEGY)
-                                               .setReplicationFactor(8)
+                                               .setReplicationFactor(6)
                                                .setZoneReplicationFactor(zoneReplicationFactor)
                                                .setKeySerializer(new SerializerDefinition("string"))
                                                .setValueSerializer(new SerializerDefinition("string"))