v4.14.1 (#157)

* v4.14.1 * bump version
statsig-io · May 6, 2022 · 9b12b4b · 9b12b4b
1 parent fe5c6b4
commit 9b12b4b
Show file tree

Hide file tree

Showing 4 changed files with 47 additions and 6 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "statsig-node",
-  "version": "4.14.0",
+  "version": "4.14.1",
   "description": "Statsig Node.js SDK for usage in multi-user server environments.",
   "main": "dist/src/index.js",
   "scripts": {

diff --git a/src/LogEventProcessor.js b/src/LogEventProcessor.js
@@ -177,7 +177,7 @@ function LogEventProcessor(options, secretKey) {
 
     let metadataKey = '';
     if (metadata && typeof(metadata) === 'object') {
-      customIdKey = Object.values(metadata).join();
+      metadataKey = Object.values(metadata).join();
     }
 
     const keyList = [ 

diff --git a/src/__tests__/index.test.js b/src/__tests__/index.test.js
@@ -420,7 +420,7 @@ describe('Verify behavior of top level index functions', () => {
     expect(spy).toHaveBeenCalledWith(configExposure);
   });
 
-  test('Verify that getConfig() and getExperiment() are deduped with same metadata', async () => {
+  test('Verify that checkGate() getConfig() and getExperiment() are deduped with same metadata', async () => {
     expect.assertions(1);
 
     const statsig = require('../index');
@@ -431,6 +431,9 @@ describe('Verify behavior of top level index functions', () => {
         number: 12345,
       });
     });
+    jest.spyOn(Evaluator, 'checkGate').mockImplementation((_, configName) => {
+      return new ConfigEvaluation(true, 'rule_id_config', [], {});
+    });
     await statsig.initialize(secretKey);
 
     let user = { userID: 123, privateAttributes: { secret: 'do not log' } };
@@ -439,9 +442,47 @@ describe('Verify behavior of top level index functions', () => {
     const spy = jest.spyOn(statsig._logger, 'log');
     for (let ii = 0 ; ii < 10000; ii++) {
       await statsig.getConfig(user, configName);
+      await statsig.checkGate(user, "test_gate");
     }
 
-    expect(spy).toHaveBeenCalledTimes(1);
+    expect(spy).toHaveBeenCalledTimes(2);
+  });
+
+  test('Verify that checkGate() getConfig() and getExperiment() are not deduped with different customIDs', async () => {
+    expect.assertions(2);
+
+    const statsig = require('../index');
+    const { Evaluator, ConfigEvaluation } = require('../Evaluator');
+    jest.spyOn(Evaluator, 'getConfig').mockImplementation((_, configName) => {
+      return new ConfigEvaluation(true, 'rule_id_config', [], {
+        string: '12345',
+        number: 12345,
+      });
+    });
+    jest.spyOn(Evaluator, 'checkGate').mockImplementation((_, configName) => {
+      return new ConfigEvaluation(true, 'rule_id_config', [], {});
+    });
+    await statsig.initialize(secretKey);
+
+    let user = { userID: 123, customIDs: {"project": "def", company: "abc"}, privateAttributes: { secret: 'do not log' } };
+    let configName = 'config_downloaded';
+    const gateName = "test";
+
+    const spy = jest.spyOn(statsig._logger, 'log');
+    for (let ii = 0 ; ii < 10000; ii++) {
+      await statsig.getConfig(user, configName);
+      await statsig.checkGate(user, gateName);
+    }
+
+    expect(spy).toHaveBeenCalledTimes(2);
+
+    for (let ii = 0 ; ii < 10000; ii++) {
+      user.customIDs.project = ii + "";
+      await statsig.getConfig(user, configName);
+      await statsig.checkGate(user, gateName);
+    }
+
+    expect(spy).toHaveBeenCalledTimes(20002);
   });
 
   test('Verify that getConfig() and getExperiment() are not deduped with different user', async () => {