Removes sorting from GetEnabledFeatures method. (#73)

Author: mfahadahmed

OptimizelySDK.Tests/OptimizelyTest.cs

@@ -1811,39 +1811,6 @@ public void TestGetEnabledFeaturesWithSomeFeaturesEnabledForUser()
             Array.ForEach(notEnabledFeatures, nef => CollectionAssert.DoesNotContain(actualFeaturesList, nef));
         }
 
-        [Test]
-        public void TestGetEnabledFeaturesReturnsSortedList()
-        {
-            string[] unsortedFeaturesList =
-            {                
-                 "Double_single_variable_feature_ignorecase",
-                 "double_single_variable_feature",
-                 "boolean_feature",
-                 "string_single_variable_feature",                 
-                 "multi_variate_feature",
-                 "empty_feature",
-                 "boolean_single_variable_feature"
-             };
-            string[] sortedFeaturesList =
-            {
-                 "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",
-             };
-
-            OptimizelyMock.Setup(om => om.IsFeatureEnabled(It.IsIn<string>(unsortedFeaturesList), TestUserId,
-                It.IsAny<UserAttributes>())).Returns(true);
-
-            // Verify that returned list in sorterd in ascending order.
-            var actualFeaturesList = OptimizelyMock.Object.GetEnabledFeatures(TestUserId, null);
-            CollectionAssert.AreEqual(sortedFeaturesList, actualFeaturesList);
-        }
-
         #endregion // Test GetEnabledFeatures
     }
 }

OptimizelySDK.Tests/ProjectConfigTest.cs

@@ -356,7 +356,6 @@ 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,18 +434,7 @@
 		"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,7 +591,6 @@ public List<string> GetEnabledFeatures(string userId, UserAttributes userAttribu
                     enabledFeaturesList.Add(featureKey);
             }
 
-            enabledFeaturesList.Sort(StringComparer.OrdinalIgnoreCase);
             return enabledFeaturesList;
         }