Merge pull request #47 from Flagsmith/release/2.0.4

Release 2.0.4

Author: matthewelwell

example/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "flagsmith-nodejs",
-  "version": "2.0.3",
+  "version": "2.0.4",
   "description": "Flagsmith lets you manage features flags and remote config across web, mobile and server side applications. Deliver true Continuous Integration. Get builds out faster. Control who has access to new features.",
   "main": "build/index.js",
   "repository": {

package-lock.json

@@ -49,8 +49,8 @@ export class AnalyticsProcessor {
         this.lastFlushed = Date.now();
     }
 
-    trackFeature(featureId: number) {
-        this.analyticsData[featureId] = (this.analyticsData[featureId] || 0) + 1;
+    trackFeature(featureName: string) {
+        this.analyticsData[featureName] = (this.analyticsData[featureName] || 0) + 1;
         if (Date.now() - this.lastFlushed > ANALYTICS_TIMER * 1000) {
             this.flush();
         }

package.json

@@ -129,7 +129,7 @@ export class Flags {
         }
 
         if (this.analyticsProcessor && flag.featureId) {
-            this.analyticsProcessor.trackFeature(flag.featureId);
+            this.analyticsProcessor.trackFeature(flag.featureName);
         }
 
         return flag;

sdk/analytics.ts

@@ -8,28 +8,28 @@ afterEach(() => {
 
 test('test_analytics_processor_track_feature_updates_analytics_data', () => {
     const aP = analyticsProcessor();
-    aP.trackFeature(1);
-    expect(aP.analyticsData[1]).toBe(1);
+    aP.trackFeature("myFeature");
+    expect(aP.analyticsData["myFeature"]).toBe(1);
 
-    aP.trackFeature(1);
-    expect(aP.analyticsData[1]).toBe(2);
+    aP.trackFeature("myFeature");
+    expect(aP.analyticsData["myFeature"]).toBe(2);
 });
 
 test('test_analytics_processor_flush_clears_analytics_data', async () => {
     const aP = analyticsProcessor();
-    aP.trackFeature(1);
+    aP.trackFeature("myFeature");
     await aP.flush();
     expect(aP.analyticsData).toStrictEqual({});
 });
 
 test('test_analytics_processor_flush_post_request_data_match_ananlytics_data', async () => {
     const aP = analyticsProcessor();
-    aP.trackFeature(1);
-    aP.trackFeature(2);
+    aP.trackFeature("myFeature1");
+    aP.trackFeature("myFeature2");
     await aP.flush();
     expect(fetch).toHaveBeenCalledTimes(1);
     expect(fetch).toHaveBeenCalledWith('http://testUrlanalytics/flags/', {
-        body: '{"1":1,"2":1}',
+        body: '{"myFeature1":1,"myFeature2":1}',
         headers: { 'Content-Type': 'application/json', 'X-Environment-Key': 'test-key' },
         method: 'POST',
         timeout: 3
@@ -39,9 +39,9 @@ test('test_analytics_processor_flush_post_request_data_match_ananlytics_data', a
 jest.useFakeTimers()
 test('test_analytics_processor_flush_post_request_data_match_ananlytics_data_test', async () => {
     const aP = analyticsProcessor();
-    aP.trackFeature(1);
+    aP.trackFeature("myFeature1");
     setTimeout(() => {
-        aP.trackFeature(2);
+        aP.trackFeature("myFeature2");
         expect(fetch).toHaveBeenCalledTimes(1);
     }, 15000);
     jest.runOnlyPendingTimers();