added tests

Author: chillaq

client/src/test/java/io/split/engine/experiments/ParsedRuleBasedSegmentTest.java

@@ -46,5 +46,37 @@ public void worksWithoutExcluded() {
         RuleBasedSegmentsToUpdate toUpdate = processRuleBasedSegmentChanges(parser, change.ruleBasedSegments.d);
         Assert.assertTrue(toUpdate.getToAdd().get(0).excludedKeys().isEmpty());
         Assert.assertTrue(toUpdate.getToAdd().get(0).excludedSegments().isEmpty());
+
+        load = "{\"ff\":{\"s\":-1,\"t\":-1,\"d\":[]},\"rbs\":{\"s\":-1,\"t\":1457726098069,\"d\":[{ \"changeNumber\": 123, \"trafficTypeName\": \"user\", \"name\": \"some_name\","
+                + "\"status\": \"ACTIVE\",\"excluded\":{\"segments\":[\"segment1\"]},\"conditions\": [{\"contitionType\": \"ROLLOUT\","
+                + "\"label\": \"some_label\", \"matcherGroup\": { \"matchers\": [{ \"matcherType\": \"ALL_KEYS\", \"negate\": false}],"
+                + "\"combiner\": \"AND\"}}]}]}}";
+        change = Json.fromJson(load, SplitChange.class);
+        toUpdate = processRuleBasedSegmentChanges(parser, change.ruleBasedSegments.d);
+        Assert.assertTrue(toUpdate.getToAdd().get(0).excludedKeys().isEmpty());
+
+        load = "{\"ff\":{\"s\":-1,\"t\":-1,\"d\":[]},\"rbs\":{\"s\":-1,\"t\":1457726098069,\"d\":[{ \"changeNumber\": 123, \"trafficTypeName\": \"user\", \"name\": \"some_name\","
+                + "\"status\": \"ACTIVE\",\"excluded\":{\"segments\":[\"segment1\"], \"keys\":null},\"conditions\": [{\"contitionType\": \"ROLLOUT\","
+                + "\"label\": \"some_label\", \"matcherGroup\": { \"matchers\": [{ \"matcherType\": \"ALL_KEYS\", \"negate\": false}],"
+                + "\"combiner\": \"AND\"}}]}]}}";
+        change = Json.fromJson(load, SplitChange.class);
+        toUpdate = processRuleBasedSegmentChanges(parser, change.ruleBasedSegments.d);
+        Assert.assertTrue(toUpdate.getToAdd().get(0).excludedKeys().isEmpty());
+
+        load = "{\"ff\":{\"s\":-1,\"t\":-1,\"d\":[]},\"rbs\":{\"s\":-1,\"t\":1457726098069,\"d\":[{ \"changeNumber\": 123, \"trafficTypeName\": \"user\", \"name\": \"some_name\","
+                + "\"status\": \"ACTIVE\",\"excluded\":{\"keys\":[\"key1\"]},\"conditions\": [{\"contitionType\": \"ROLLOUT\","
+                + "\"label\": \"some_label\", \"matcherGroup\": { \"matchers\": [{ \"matcherType\": \"ALL_KEYS\", \"negate\": false}],"
+                + "\"combiner\": \"AND\"}}]}]}}";
+        change = Json.fromJson(load, SplitChange.class);
+        toUpdate = processRuleBasedSegmentChanges(parser, change.ruleBasedSegments.d);
+        Assert.assertTrue(toUpdate.getToAdd().get(0).excludedSegments().isEmpty());
+
+        load = "{\"ff\":{\"s\":-1,\"t\":-1,\"d\":[]},\"rbs\":{\"s\":-1,\"t\":1457726098069,\"d\":[{ \"changeNumber\": 123, \"trafficTypeName\": \"user\", \"name\": \"some_name\","
+                + "\"status\": \"ACTIVE\",\"excluded\":{\"segments\":null, \"keys\":[\"key1\"]},\"conditions\": [{\"contitionType\": \"ROLLOUT\","
+                + "\"label\": \"some_label\", \"matcherGroup\": { \"matchers\": [{ \"matcherType\": \"ALL_KEYS\", \"negate\": false}],"
+                + "\"combiner\": \"AND\"}}]}]}}";
+        change = Json.fromJson(load, SplitChange.class);
+        toUpdate = processRuleBasedSegmentChanges(parser, change.ruleBasedSegments.d);
+        Assert.assertTrue(toUpdate.getToAdd().get(0).excludedSegments().isEmpty());
     }
 }