add test to validate that NodeJS SDK times out when a client-side SDK key is provided

Author: EmilianoSanchez

package-lock.json

@@ -40,7 +40,7 @@
     "node": ">=6"
   },
   "dependencies": {
-    "@splitsoftware/splitio-commons": "1.10.1-rc.3",
+    "@splitsoftware/splitio-commons": "1.10.1-rc.4",
     "@types/google.analytics": "0.0.40",
     "@types/ioredis": "^4.28.0",
     "bloom-filters": "^3.0.0",

package.json

@@ -0,0 +1,51 @@
+import { SplitFactory } from '../../';
+
+import splitChangesMock1 from '../mocks/splitchanges.since.-1.json';
+import splitChangesMock2 from '../mocks/splitchanges.since.1457552620999.json';
+
+const readyTimeout = 0.1;
+
+const baseConfig = {
+  core: {
+    authorizationKey: '<fake-token>',
+  },
+  startup: {
+    readyTimeout,
+  },
+  streamingEnabled: false
+};
+
+export default function (fetchMock, assert) {
+
+  assert.test(t => { // Timeout test: we provide a client-side SDK key on server-side (403 error)
+    const testUrls = {
+      sdk: 'https://sdk.baseurl/readinessSuite1',
+      events: 'https://events.baseurl/readinessSuite1'
+    };
+
+    fetchMock.getOnce(testUrls.sdk + '/splitChanges?since=-1', { status: 200, body: splitChangesMock1 });
+    fetchMock.getOnce(testUrls.sdk + '/splitChanges?since=1457552620999', { status: 200, body: splitChangesMock2 });
+    fetchMock.get(new RegExp(testUrls.sdk + '/segmentChanges/*'), 403);
+    fetchMock.postOnce(testUrls.events + '/events/bulk', 200);
+
+    const splitio = SplitFactory({
+      ...baseConfig, urls: testUrls
+    });
+    const client = splitio.client();
+
+    t.true(client.track('some_key', 'some_tt', 'some_event_type'), 'since client is not destroyed, client.track returns true');
+
+    client.once(client.Event.SDK_READY, () => {
+      t.fail('### IS READY - NOT TIMED OUT when it should.');
+      t.end();
+    });
+    client.once(client.Event.SDK_READY_TIMED_OUT, () => {
+      t.pass('### SDK TIMED OUT - SegmentChanges requests with client-side SDK key should fail with 403. Timed out.');
+
+      t.false(client.track('some_key', 'some_tt', 'some_event_type'), 'since client is flagged as destroyed, client.track returns false');
+
+      client.destroy().then(() => { t.end(); });
+    });
+  });
+
+}

src/__tests__/nodeSuites/readiness.spec.js

@@ -14,6 +14,7 @@ import expectedTreatmentsSuite from '../nodeSuites/expected-treatments.spec';
 import managerSuite from '../nodeSuites/manager.spec';
 import ipAddressesSetting from '../nodeSuites/ip-addresses-setting.spec';
 import ipAddressesSettingDebug from '../nodeSuites/ip-addresses-setting.debug.spec';
+import readinessSuite from '../nodeSuites/readiness.spec';
 import readyPromiseSuite from '../nodeSuites/ready-promise.spec';
 import { fetchSpecificSplits, fetchSpecificSplitsForFlagSets } from '../nodeSuites/fetch-specific-splits.spec';
 
@@ -78,6 +79,9 @@ tape('## Node JS - E2E CI Tests ##', async function (assert) {
   assert.test('E2E / IP Addresses Setting', ipAddressesSetting.bind(null, fetchMock));
   assert.test('E2E / IP Addresses Setting Debug', ipAddressesSettingDebug.bind(null, fetchMock));
 
+  /* Validate readiness */
+  assert.test('E2E / Readiness', readinessSuite.bind(null, fetchMock));
+
   /* Validate readiness with ready promises */
   assert.test('E2E / Ready promise', readyPromiseSuite.bind(null, key, fetchMock));