[FSSDK-12337] Add experiment type constants for type-safe experiment type checks

Author: FarhanAnjum-opti

core-api/src/main/java/com/optimizely/ab/config/DatafileProjectConfig.java

@@ -403,7 +403,7 @@ private List<Experiment> injectFeatureRolloutVariations(
                     continue;
                 }
                 Experiment experiment = result.get(index);
-                if (!"feature_rollout".equals(experiment.getType())) {
+                if (!Experiment.TYPE_FR.equals(experiment.getType())) {
                     continue;
                 }
 

core-api/src/main/java/com/optimizely/ab/config/Experiment.java

@@ -53,6 +53,12 @@ public class Experiment implements ExperimentCore {
     private final Map<String, Variation> variationIdToVariationMap;
     private final Map<String, String> userIdToVariationKeyMap;
 
+    public static final String TYPE_AB = "a/b";
+    public static final String TYPE_MAB = "multi_armed_bandit";
+    public static final String TYPE_CMAB = "contextual_multi_armed_bandit";
+    public static final String TYPE_TD = "targeted_delivery";
+    public static final String TYPE_FR = "feature_rollout";
+
     public enum ExperimentStatus {
         RUNNING("Running"),
         LAUNCHED("Launched"),

core-api/src/test/java/com/optimizely/ab/config/FeatureRolloutConfigTest.java

@@ -63,7 +63,7 @@ public void experimentWithoutTypeFieldHasNullType() {
     public void featureRolloutExperimentGetsEveryoneElseVariationInjected() {
         Experiment experiment = projectConfig.getExperimentKeyMapping().get("feature_rollout_experiment");
         assertNotNull("Experiment should exist", experiment);
-        assertEquals("feature_rollout", experiment.getType());
+        assertEquals(Experiment.TYPE_FR, experiment.getType());
 
         // Should have 2 variations: original + everyone else
         assertEquals("Should have 2 variations after injection", 2, experiment.getVariations().size());
@@ -107,7 +107,7 @@ public void variationMapsContainInjectedVariation() {
     public void abTestExperimentNotModified() {
         Experiment experiment = projectConfig.getExperimentKeyMapping().get("ab_test_experiment");
         assertNotNull("Experiment should exist", experiment);
-        assertEquals("a/b", experiment.getType());
+        assertEquals(Experiment.TYPE_AB, experiment.getType());
 
         // Should still have exactly 2 original variations
         assertEquals("A/B test should keep original 2 variations", 2, experiment.getVariations().size());
@@ -127,7 +127,7 @@ public void abTestExperimentNotModified() {
     public void featureRolloutWithEmptyRolloutIdDoesNotCrash() {
         Experiment experiment = projectConfig.getExperimentKeyMapping().get("rollout_no_rollout_id_experiment");
         assertNotNull("Experiment should exist", experiment);
-        assertEquals("feature_rollout", experiment.getType());
+        assertEquals(Experiment.TYPE_FR, experiment.getType());
 
         // Should keep only original variation since rollout cannot be resolved
         assertEquals("Should keep only original variation", 1, experiment.getVariations().size());
@@ -142,11 +142,11 @@ public void featureRolloutWithEmptyRolloutIdDoesNotCrash() {
     public void typeFieldCorrectlyParsed() {
         Experiment rolloutExp = projectConfig.getExperimentKeyMapping().get("feature_rollout_experiment");
         assertNotNull(rolloutExp);
-        assertEquals("feature_rollout", rolloutExp.getType());
+        assertEquals(Experiment.TYPE_FR, rolloutExp.getType());
 
         Experiment abExp = projectConfig.getExperimentKeyMapping().get("ab_test_experiment");
         assertNotNull(abExp);
-        assertEquals("a/b", abExp.getType());
+        assertEquals(Experiment.TYPE_AB, abExp.getType());
 
         Experiment noTypeExp = projectConfig.getExperimentKeyMapping().get("no_type_experiment");
         assertNotNull(noTypeExp);