Revert "add experiment and variation id to decision notification payload"

This reverts commit 6c3e277.

Author: muzahidul-opti

Sources/Implementation/DecisionInfo.swift

@@ -140,17 +140,9 @@ struct DecisionInfo {
             decisionInfo[Constants.DecisionInfoKeys.variationKey] = variation?.key ?? NSNull()      // keep key in the map even with nil value
             decisionInfo[Constants.DecisionInfoKeys.ruleKey] = ruleKey ?? NSNull()                  //
             decisionInfo[Constants.DecisionInfoKeys.reasons] = reasons
+            decisionInfo[Constants.DecisionInfoKeys.decisionEventDispatched] = decisionEventDispatched
         }
 
-        decisionInfo[Constants.DecisionInfoKeys.decisionEventDispatched] = decisionEventDispatched
-        
-        if let expId = experiment?.id {
-            decisionInfo[Constants.ExperimentDecisionInfoKeys.experimentId] = expId
-        }
-        
-        if let varId = variation?.id {
-            decisionInfo[Constants.ExperimentDecisionInfoKeys.variationId] = varId
-        }
         return decisionInfo
     }
 

Sources/Utils/Constants.swift

@@ -88,9 +88,7 @@ struct Constants {
     }
 
     struct ExperimentDecisionInfoKeys {
-        static let experimentId = "experimentId"
         static let experiment = "experimentKey"
-        static let variationId = "variationId"
         static let variation = "variationKey"
     }
 

Tests/OptimizelyTests-Common/DecisionListenerTests_Datafile.swift

@@ -37,16 +37,12 @@ class DecisionListenerTests_Datafile: XCTestCase {
         var notificationVariation: String?
         var notificationExperiment: String?
         var notificationType: String?
-        var expId: String?
-        var varId: String?
 
         let exp = expectation(description: "x")
 
         _ = notificationCenter.addDecisionNotificationListener(decisionListener: { (type, _, _, decisionInfo) in
             notificationExperiment = decisionInfo[Constants.ExperimentDecisionInfoKeys.experiment] as? String
             notificationVariation = decisionInfo[Constants.ExperimentDecisionInfoKeys.variation] as? String
-            expId = decisionInfo[Constants.ExperimentDecisionInfoKeys.experimentId] as? String
-            varId = decisionInfo[Constants.ExperimentDecisionInfoKeys.variationId] as? String
             notificationType = type
             exp.fulfill()
         })
@@ -60,27 +56,19 @@ class DecisionListenerTests_Datafile: XCTestCase {
         XCTAssertEqual(variation, "all_traffic_variation")
         XCTAssertEqual(notificationExperiment, "ab_running_exp_audience_combo_empty_conditions")
         XCTAssertEqual(notificationVariation, "all_traffic_variation")
-        XCTAssertEqual(expId, "10390977723")
-        XCTAssertEqual(varId, "10416523170")
         XCTAssertEqual(notificationType, Constants.DecisionType.abTest.rawValue)
     }
 
     func testDecisionListenerWithActivateWhenUserNotInExperiment() {
         var notificationVariation: String?
         var notificationExperiment: String?
         var notificationType: String?
-        var expId: String?
-        var varId: String?
 
         let exp = expectation(description: "x")
 
         _ = notificationCenter.addDecisionNotificationListener(decisionListener: { (type, _, _, decisionInfo) in
             notificationExperiment = decisionInfo[Constants.ExperimentDecisionInfoKeys.experiment] as? String
             notificationVariation = decisionInfo[Constants.ExperimentDecisionInfoKeys.variation] as? String
-            expId = decisionInfo[Constants.ExperimentDecisionInfoKeys.experimentId] as? String
-            varId = decisionInfo[Constants.ExperimentDecisionInfoKeys.variationId] as? String
-            expId = decisionInfo[Constants.ExperimentDecisionInfoKeys.experimentId] as? String
-            varId = decisionInfo[Constants.ExperimentDecisionInfoKeys.variationId] as? String
             notificationType = type
             exp.fulfill()
         })
@@ -93,8 +81,6 @@ class DecisionListenerTests_Datafile: XCTestCase {
 
         XCTAssertEqual(notificationExperiment, "ab_running_exp_audience_combo_exact_foo_or_true__and__42_or_4_2")
         XCTAssertEqual(notificationVariation, nil)
-        XCTAssertEqual(expId, "10390977714")
-        XCTAssertEqual(varId, nil)
         XCTAssertEqual(notificationType, Constants.DecisionType.abTest.rawValue)
         notificationCenter.clearAllNotificationListeners()
     }
@@ -107,16 +93,12 @@ class DecisionListenerTests_Datafile: XCTestCase {
         var notificationVariation: String?
         var notificationExperiment: String?
         var notificationType: String?
-        var expId: String?
-        var varId: String?
 
         let exp = expectation(description: "x")
 
         _ = notificationCenter.addDecisionNotificationListener(decisionListener: { (type, _, _, decisionInfo) in
             notificationExperiment = decisionInfo[Constants.ExperimentDecisionInfoKeys.experiment] as? String
             notificationVariation = decisionInfo[Constants.ExperimentDecisionInfoKeys.variation] as? String
-            expId = decisionInfo[Constants.ExperimentDecisionInfoKeys.experimentId] as? String
-            varId = decisionInfo[Constants.ExperimentDecisionInfoKeys.variationId] as? String
             notificationType = type
             exp.fulfill()
        })
@@ -129,24 +111,19 @@ class DecisionListenerTests_Datafile: XCTestCase {
         XCTAssertEqual(notificationExperiment, "ab_running_exp_audience_combo_empty_conditions")
         XCTAssertEqual(notificationVariation, "all_traffic_variation")
         XCTAssertEqual(notificationType, Constants.DecisionType.abTest.rawValue)
-        XCTAssertEqual(expId, "10390977723")
-        XCTAssertEqual(varId, "10416523170")
         notificationCenter.clearAllNotificationListeners()
     }
 
     func testDecisionListenerWithGetVariationWhenUserNotInExperiment() {
         var notificationVariation: String?
         var notificationExperiment: String?
         var notificationType: String?
-        var expId: String?
-        var varId: String?
+        
         let exp = expectation(description: "x")
 
         _ = notificationCenter.addDecisionNotificationListener(decisionListener: { (type, _, _, decisionInfo) in
             notificationExperiment = decisionInfo[Constants.ExperimentDecisionInfoKeys.experiment] as? String
             notificationVariation = decisionInfo[Constants.ExperimentDecisionInfoKeys.variation] as? String
-            expId = decisionInfo[Constants.ExperimentDecisionInfoKeys.experimentId] as? String
-            varId = decisionInfo[Constants.ExperimentDecisionInfoKeys.variationId] as? String
             notificationType = type
             exp.fulfill()
         })
@@ -156,8 +133,6 @@ class DecisionListenerTests_Datafile: XCTestCase {
 
         XCTAssertEqual(notificationExperiment, "ab_running_exp_audience_combo_exact_foo_or_true__and__42_or_4_2")
         XCTAssertEqual(notificationVariation, nil)
-        XCTAssertEqual(expId, "10390977714")
-        XCTAssertEqual(varId, nil)
         XCTAssertEqual(notificationType, Constants.DecisionType.abTest.rawValue)
         notificationCenter.clearAllNotificationListeners()
     }