[FSSDK-11446] update: experiment_id and variation_id added to notification listener payloads (#1030)

* -added experiment id and variation id to notification listener payload
-fixed unit tests to expect experiment id and variation id in notification listener payload

* Type changed of notification listener payload

Author: FarhanAnjum-opti

lib/notification_center/type.ts

@@ -94,6 +94,8 @@ export type FlagDecisionInfo = {
   variables: VariablesMap,
   reasons: string[],
   decisionEventDispatched: boolean,
+  experimentId: string | null,
+  variationId: string | null,
 };
 
 export type DecisionInfo = {

lib/optimizely/index.tests.js

@@ -4602,6 +4602,8 @@ describe('lib/optimizely', function() {
                 variables: { i_42: 42 },
                 decisionEventDispatched: true,
                 reasons: [],
+                experimentId: '10420810910',
+                variationId: '10418551353',
               },
             },
           ];
@@ -4652,6 +4654,8 @@ describe('lib/optimizely', function() {
                 variables: { i_42: 42 },
                 decisionEventDispatched: false,
                 reasons: [],
+                experimentId: '10420810910',
+                variationId: '10418551353',
               },
             },
           ];
@@ -4704,6 +4708,8 @@ describe('lib/optimizely', function() {
                 variables: {},
                 decisionEventDispatched: false,
                 reasons: [],
+                experimentId: '10420810910',
+                variationId: '10418551353',
               },
             },
           ];
@@ -4754,6 +4760,8 @@ describe('lib/optimizely', function() {
                 variables: expectedVariables,
                 decisionEventDispatched: true,
                 reasons: [],
+                experimentId: '18322080788',
+                variationId: '18257766532',
               },
             },
           ];
@@ -4807,6 +4815,8 @@ describe('lib/optimizely', function() {
                 variables: expectedVariables,
                 decisionEventDispatched: false,
                 reasons: [],
+                experimentId: '18322080788',
+                variationId: '18257766532',
               },
             },
           ];
@@ -4857,6 +4867,8 @@ describe('lib/optimizely', function() {
                 variables: expectedVariables,
                 decisionEventDispatched: true,
                 reasons: [],
+                experimentId: null,
+                variationId: null,
               },
             },
           ];
@@ -4907,6 +4919,8 @@ describe('lib/optimizely', function() {
                 variables: {},
                 decisionEventDispatched: true,
                 reasons: [],
+                experimentId: "10420810910",
+                variationId: "10418551353",
               },
             },
           ];
@@ -4955,6 +4969,8 @@ describe('lib/optimizely', function() {
                 variables: {},
                 decisionEventDispatched: false,
                 reasons: [],
+                experimentId: '10420810910',
+                variationId: '10418551353',
               },
             },
           ];
@@ -5024,6 +5040,8 @@ describe('lib/optimizely', function() {
                 variables: expectedVariables,
                 decisionEventDispatched: false,
                 reasons: [],
+                experimentId: '10420810910',
+                variationId: '10418551353',
               },
             },
           ];

lib/optimizely/index.ts

@@ -1469,7 +1469,9 @@ export default class Optimizely extends BaseService implements Client {
       const feature = configObj.featureKeyMap[key]
       const decisionSource = decisionObj.decisionSource;
       const experimentKey = decisionObj.experiment?.key ?? null;
+      const experimentId = decisionObj.experiment?.id ?? null;
       const variationKey = decisionObj.variation?.key ?? null;
+      const variationId = decisionObj.variation?.id ?? null;
       const flagEnabled: boolean = decision.getFeatureEnabledFromVariation(decisionObj);
       const variablesMap: { [key: string]: unknown } = {};
       let decisionEventDispatched = false;
@@ -1516,6 +1518,8 @@ export default class Optimizely extends BaseService implements Client {
         variables: variablesMap,
         reasons: reportedReasons,
         decisionEventDispatched: decisionEventDispatched,
+        experimentId: experimentId,
+        variationId: variationId,
       };
 
       this.notificationCenter.sendNotifications(NOTIFICATION_TYPES.DECISION, {

lib/optimizely_user_context/index.tests.js

@@ -565,6 +565,8 @@ describe('lib/optimizely_user_context', function() {
                     userId
                   ),
                 ],
+                experimentId: null,
+                variationId: '3324490562'
               },
             },
           ];
@@ -654,6 +656,8 @@ describe('lib/optimizely_user_context', function() {
                     userId
                   ),
                 ],
+                experimentId: '10390977673',
+                variationId: '10416523121',
               },
             },
           ];
@@ -734,6 +738,8 @@ describe('lib/optimizely_user_context', function() {
                 },
                 decisionEventDispatched: true,
                 reasons: [],
+                experimentId: '3332020515',
+                variationId: '3324490633',
               },
             },
           ];