GetEnabledFeatures sorted (#220)

Author: arafay-folio3

OptimizelySDKCore/OptimizelySDKCore/Optimizely.m

@@ -364,7 +364,8 @@ - (NSString *)getFeatureVariableString:(nullable NSString *)featureKey
             [enabledFeatures addObject:featureKey];
         }
     }
-    return enabledFeatures;
+    NSArray *enabledFeaturesSorted = [enabledFeatures sortedArrayUsingSelector:@selector(caseInsensitiveCompare:)];
+    return enabledFeaturesSorted;
 }
 
 #pragma mark trackEvent methods

OptimizelySDKCore/OptimizelySDKCoreTests/OptimizelyTest.m

@@ -940,13 +940,22 @@ -(void)testGetEnabledFeaturesWithNoFeatureEnabledForUser {
     [optimizelyMock stopMocking];
 }
 
-// should return empty feature array as no feature is enabled for user
+// should return feature array as some feature is enabled for user
 -(void)testGetEnabledFeaturesWithSomeFeaturesEnabledForUser {
     NSArray<NSString *> *enabledFeatures = @[@"booleanFeature", @"booleanSingleVariableFeature", @"multiVariateFeature"];
     NSArray<NSString *> *features = [self.optimizely getEnabledFeatures:kUserId attributes:self.attributes];
     XCTAssertEqualObjects(features, enabledFeatures);
 }
 
+// should return sorted feature array as some feature is enabled for user
+-(void)testGetEnabledFeaturesSortedWithSomeFeaturesEnabledForUser {
+    NSArray<NSString *> *sortedEnabledFeatures = @[@"booleanFeature", @"booleanSingleVariableFeature", @"multiVariateFeature"];
+    NSArray<NSString *> *unSortedEnabledFeatures = @[ @"multiVariateFeature", @"booleanFeature", @"booleanSingleVariableFeature"];
+    NSArray<NSString *> *features = [self.optimizely getEnabledFeatures:kUserId attributes:self.attributes];
+    XCTAssertEqualObjects(features, sortedEnabledFeatures);
+    XCTAssertNotEqualObjects(features, unSortedEnabledFeatures);
+}
+
 #pragma mark - Helper Methods
 
 - (id)getOptimizelyMockForFeatureVariableType:(NSString *)featureVariableType variableKey:(NSString *)variableKey expectedReturn:(NSString *)expectedReturn {

OptimizelySDKCore/OptimizelySDKCoreTests/TestData/test_data_10_experiments.json

@@ -713,6 +713,29 @@
       "experimentIds": [],
       "variables": []
     },
+    {
+      "id": "155559",
+      "key": "multiVariateFeature",
+      "groupId": "",
+      "rolloutId": "",
+      "experimentIds": [
+        "6358043287"
+      ],
+      "variables": [
+        {
+          "id": "155560",
+          "key": "firstLetter",
+          "type": "string",
+          "defaultValue": "H"
+        },
+        {
+          "id": "155561",
+          "key": "restOfName",
+          "type": "string",
+          "defaultValue": "arry"
+        }
+      ]
+    },
     {
       "id": "155549",
       "key": "booleanFeature",
@@ -765,29 +788,6 @@
         }
       ]
     },
-    {
-      "id": "155559",
-      "key": "multiVariateFeature",
-      "groupId": "",
-      "rolloutId": "",
-      "experimentIds": [
-        "6358043287"
-      ],
-      "variables": [
-        {
-          "id": "155560",
-          "key": "firstLetter",
-          "type": "string",
-          "defaultValue": "H"
-        },
-        {
-          "id": "155561",
-          "key": "restOfName",
-          "type": "string",
-          "defaultValue": "arry"
-        }
-      ]
-    },
     {
       "id": "155666",
       "key": "doubleSingleVariableFeature",