GetEnabledFeatures now returns sorted features list. (#69)

* GetEnabledFeatures now returns sorted features list ignoring case

Author: mfahadahmed

OptimizelySDK.Tests/OptimizelyTest.cs

@@ -1815,10 +1815,11 @@ public void TestGetEnabledFeaturesWithSomeFeaturesEnabledForUser()
         public void TestGetEnabledFeaturesReturnsSortedList()
         {
             string[] unsortedFeaturesList =
-            {
+            {                
+                 "Double_single_variable_feature_ignorecase",
                  "double_single_variable_feature",
                  "boolean_feature",
-                 "string_single_variable_feature",
+                 "string_single_variable_feature",                 
                  "multi_variate_feature",
                  "empty_feature",
                  "boolean_single_variable_feature"
@@ -1828,6 +1829,8 @@ public void TestGetEnabledFeaturesReturnsSortedList()
                  "boolean_feature",
                  "boolean_single_variable_feature",
                  "double_single_variable_feature",
+                 //It must come first, if case-sensitive sorted otherwise should come at 4th place
+                 "Double_single_variable_feature_ignorecase",
                  "empty_feature",
                  "multi_variate_feature",
                  "string_single_variable_feature",

OptimizelySDK.Tests/ProjectConfigTest.cs

@@ -356,6 +356,7 @@ public void TestInit()
             {
                 { "boolean_feature", Config.GetFeatureFlagFromKey("boolean_feature") },
                 { "double_single_variable_feature", Config.GetFeatureFlagFromKey("double_single_variable_feature") },
+                { "Double_single_variable_feature_ignorecase", Config.GetFeatureFlagFromKey("Double_single_variable_feature_ignorecase") },
                 { "integer_single_variable_feature", Config.GetFeatureFlagFromKey("integer_single_variable_feature") },
                 { "boolean_single_variable_feature", Config.GetFeatureFlagFromKey("boolean_single_variable_feature") },
                 { "string_single_variable_feature", Config.GetFeatureFlagFromKey("string_single_variable_feature") },

OptimizelySDK.Tests/TestData.json

@@ -434,7 +434,18 @@
 		"rolloutId": "",
 		"experimentIds": ["7723330021", "7718750065"],
 		"variables": []
-	}, {
+	},
+    {
+      "id": "1555892",
+      "key": "Double_single_variable_feature_ignorecase",
+      "rolloutId": "",
+      "experimentIds": [
+        "7723330021",
+        "7718750065"
+      ],
+      "variables": []
+    },
+  {
 		"id": "155550",
 		"key": "double_single_variable_feature",
 		"rolloutId": "",

OptimizelySDK/Optimizely.cs

@@ -591,8 +591,7 @@ public List<string> GetEnabledFeatures(string userId, UserAttributes userAttribu
                     enabledFeaturesList.Add(featureKey);
             }
 
-            enabledFeaturesList.Sort();
-
+            enabledFeaturesList.Sort(StringComparer.OrdinalIgnoreCase);
             return enabledFeaturesList;
         }