Add tests

Author: EmilianoSanchez

src/__tests__/browserSuites/evaluations.spec.js

@@ -362,7 +362,7 @@ export default function (config, fetchMock, assert) {
 
   };
 
-  const evaluationsWithRuleBasedSegments = async (splitio) => {
+  const evaluationsWithRuleBasedSegmentsAndPrerequisites = async (splitio) => {
     fetchMock.getOnce('https://sdk.split.io/api/memberships/emi%40split.io', { status: 200, body: { ms: { k: [{ n: 'segment_excluded_by_rbs' }] } } });
     fetchMock.getOnce('https://sdk.split.io/api/memberships/mauro%40split.io', { status: 200, body: { ms: {} } });
     fetchMock.getOnce('https://sdk.split.io/api/memberships/bilal%40split.io', { status: 200, body: { ms: {} } });
@@ -372,24 +372,28 @@ export default function (config, fetchMock, assert) {
     await client1.ready();
     assert.equal(client1.getTreatment('rbs_test_flag'), 'v2', 'key in excluded segment');
     assert.equal(client1.getTreatment('rbs_test_flag_negated'), 'v1', 'key in excluded segment');
+    assert.equal(client1.getTreatment('always_on_if_prerequisite'), 'off', 'prerequisite not satisfied (key in excluded segment)');
     await client1.destroy();
 
     const client2 = splitio.client('[email protected]');
     await client2.ready();
     assert.equal(client2.getTreatment('rbs_test_flag'), 'v2', 'excluded key');
     assert.equal(client2.getTreatment('rbs_test_flag_negated'), 'v1', 'excluded key');
+    assert.equal(client2.getTreatment('always_on_if_prerequisite'), 'off', 'prerequisite not satisfied (excluded key)');
     await client2.destroy();
 
     const client3 = splitio.client('[email protected]');
     await client3.ready();
     assert.equal(client3.getTreatment('rbs_test_flag'), 'v1', 'key satisfies the rbs condition');
     assert.equal(client3.getTreatment('rbs_test_flag_negated'), 'v2', 'key satisfies the rbs condition');
+    assert.equal(client3.getTreatment('always_on_if_prerequisite'), 'on', 'prerequisite satisfied (key satisfies the rbs condition)');
     await client3.destroy();
 
     const client4 = splitio.client('other_key');
     await client4.ready();
     assert.equal(client4.getTreatment('rbs_test_flag'), 'v2', 'key not in segment');
     assert.equal(client4.getTreatment('rbs_test_flag_negated'), 'v1', 'key not in segment');
+    assert.equal(client4.getTreatment('always_on_if_prerequisite'), 'off', 'prerequisite not satisfied (key not in segment)');
     await client4.destroy();
   };
 
@@ -408,7 +412,7 @@ export default function (config, fetchMock, assert) {
       getTreatmentsWithConfigTests(client);
       getTreatmentsWithInMemoryAttributes(client);
 
-      evaluationsWithRuleBasedSegments(splitio).then(() => {
+      evaluationsWithRuleBasedSegmentsAndPrerequisites(splitio).then(() => {
         clientTABucket1.destroy();
         client.destroy();
         tested++;

src/__tests__/browserSuites/ready-from-cache.spec.js

@@ -818,7 +818,7 @@ export default function (fetchMock, assert) {
     });
 
     await client.ready();
-    t.equal(manager.names().sort().length, 35, 'active splits should be present for evaluation');
+    t.equal(manager.names().sort().length, 36, 'active splits should be present for evaluation');
 
     await splitio.destroy();
     t.equal(localStorage.getItem('readyFromCache_10.SPLITIO.splits.till'), '1457552620999', 'splits.till must correspond to the till of the last successfully fetched Splits');
@@ -835,7 +835,7 @@ export default function (fetchMock, assert) {
 
     await new Promise(res => client.once(client.Event.SDK_READY_FROM_CACHE, res));
 
-    t.equal(manager.names().sort().length, 35, 'active splits should be present for evaluation');
+    t.equal(manager.names().sort().length, 36, 'active splits should be present for evaluation');
     t.false(console.log.calledWithMatch('clearOnInit was set and cache was not cleared in the last 24 hours. Cleaning up cache'), 'It should log a message about cleaning up cache');
 
     await splitio.destroy();
@@ -856,7 +856,7 @@ export default function (fetchMock, assert) {
 
     await new Promise(res => client.once(client.Event.SDK_READY, res));
 
-    t.equal(manager.names().sort().length, 35, 'active splits should be present for evaluation');
+    t.equal(manager.names().sort().length, 36, 'active splits should be present for evaluation');
     t.true(console.log.calledWithMatch('clearOnInit was set and cache was not cleared in the last 24 hours. Cleaning up cache'), 'It should log a message about cleaning up cache');
 
     await splitio.destroy();

src/__tests__/browserSuites/telemetry.spec.js

@@ -76,7 +76,7 @@ export default async function telemetryBrowserSuite(fetchMock, t) {
 
       // @TODO check if iDe value is correct
       assert.deepEqual(data, {
-        mE: {}, hE: { sp: { 500: 1 }, ms: { 500: 1 } }, tR: 0, aR: 0, iQ: 4, iDe: 1, iDr: 0, spC: 35, seC: 1, skC: 1, eQ: 1, eD: 0, sE: [], t: [], ufs: {}
+        mE: {}, hE: { sp: { 500: 1 }, ms: { 500: 1 } }, tR: 0, aR: 0, iQ: 4, iDe: 1, iDr: 0, spC: 36, seC: 1, skC: 1, eQ: 1, eD: 0, sE: [], t: [], ufs: {}
       }, 'metrics/usage JSON payload should be the expected');
 
       finish.next();
@@ -96,7 +96,7 @@ export default async function telemetryBrowserSuite(fetchMock, t) {
       // @TODO check if iDe value is correct
       assert.deepEqual(data, {
         mL: {}, mE: {}, hE: {}, hL: {}, // errors and latencies were popped
-        tR: 0, aR: 0, iQ: 4, iDe: 1, iDr: 0, spC: 35, seC: 1, skC: 1, eQ: 1, eD: 0, sE: [], t: [], ufs: {}
+        tR: 0, aR: 0, iQ: 4, iDe: 1, iDr: 0, spC: 36, seC: 1, skC: 1, eQ: 1, eD: 0, sE: [], t: [], ufs: {}
       }, '2nd metrics/usage JSON payload should be the expected');
       return 200;
     });

src/__tests__/mocks/splitchanges.since.-1.json

@@ -9,8 +9,13 @@
         "status": "ACTIVE",
         "trafficTypeName": "user",
         "excluded": {
-          "keys": [ "[email protected]", "[email protected]" ],
-          "segments": [ "segment_excluded_by_rbs" ]
+          "keys": [
+            "[email protected]",
+            "[email protected]"
+          ],
+          "segments": [
+            "segment_excluded_by_rbs"
+          ]
         },
         "conditions": [
           {
@@ -1730,6 +1735,52 @@
         "configurations": {},
         "sets": [],
         "impressionsDisabled": false
+      },
+      {
+        "orgId": null,
+        "environment": null,
+        "trafficTypeId": null,
+        "trafficTypeName": null,
+        "name": "always_on_if_prerequisite",
+        "seed": -790401604,
+        "status": "ACTIVE",
+        "killed": false,
+        "defaultTreatment": "off",
+        "prerequisites": [
+          {
+            "n": "rbs_test_flag",
+            "ts": [
+              "v1"
+            ]
+          }
+        ],
+        "conditions": [
+          {
+            "matcherGroup": {
+              "combiner": "AND",
+              "matchers": [
+                {
+                  "keySelector": {
+                    "trafficType": "user",
+                    "attribute": null
+                  },
+                  "matcherType": "ALL_KEYS",
+                  "negate": false,
+                  "userDefinedSegmentMatcherData": null,
+                  "whitelistMatcherData": null,
+                  "unaryNumericMatcherData": null,
+                  "betweenMatcherData": null
+                }
+              ]
+            },
+            "partitions": [
+              {
+                "treatment": "on",
+                "size": 100
+              }
+            ]
+          }
+        ]
       }
     ],
     "s": -1,

src/__tests__/nodeSuites/evaluations.spec.js

@@ -175,6 +175,10 @@ export default async function (config, key, assert) {
     assert.equal(client.getTreatment('[email protected]', 'rbs_test_flag_negated'), 'v1', 'excluded key');
     assert.equal(client.getTreatment('[email protected]', 'rbs_test_flag_negated'), 'v2', 'key satisfies the rbs condition');
     assert.equal(client.getTreatment('other_key', 'rbs_test_flag_negated'), 'v1', 'key not in segment');
+
+    // Prerequisites
+    assert.equal(client.getTreatment('[email protected]', 'always_on_if_prerequisite'), 'on', 'prerequisite satisfied (key satisfies the rbs condition)');
+    assert.equal(client.getTreatment('[email protected]', 'always_on_if_prerequisite'), 'off', 'prerequisite not satisfied (key in excluded segment)');
   };
 
   const getTreatmentsTests = (client, sdkInstance) => {

src/__tests__/nodeSuites/telemetry.spec.js

@@ -66,7 +66,7 @@ export default async function telemetryNodejsSuite(key, fetchMock, assert) {
 
     // @TODO check if iDe value is correct
     assert.deepEqual(data, {
-      mE: {}, hE: { sp: { 500: 1 } }, tR: 0, aR: 0, iQ: 4, iDe: 1, iDr: 0, spC: 35, seC: 4, skC: 4, eQ: 1, eD: 0, sE: [], t: [], ufs: {}
+      mE: {}, hE: { sp: { 500: 1 } }, tR: 0, aR: 0, iQ: 4, iDe: 1, iDr: 0, spC: 36, seC: 4, skC: 4, eQ: 1, eD: 0, sE: [], t: [], ufs: {}
     }, 'metrics/usage JSON payload should be the expected');
 
     finish.next();
@@ -85,7 +85,7 @@ export default async function telemetryNodejsSuite(key, fetchMock, assert) {
     // @TODO check if iDe value is correct
     assert.deepEqual(data, {
       mL: {}, mE: {}, hE: {}, hL: {}, // errors and latencies were popped
-      tR: 0, aR: 0, iQ: 4, iDe: 1, iDr: 0, spC: 35, seC: 4, skC: 4, eQ: 1, eD: 0, sE: [], t: [], ufs: {}
+      tR: 0, aR: 0, iQ: 4, iDe: 1, iDr: 0, spC: 36, seC: 4, skC: 4, eQ: 1, eD: 0, sE: [], t: [], ufs: {}
     }, '2nd metrics/usage JSON payload should be the expected');
     return 200;
   });