diff --git a/OptimizelySDK.Tests/OptimizelyTest.cs b/OptimizelySDK.Tests/OptimizelyTest.cs index 45820ef9..b054363b 100644 --- a/OptimizelySDK.Tests/OptimizelyTest.cs +++ b/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(unsortedFeaturesList), TestUserId, - It.IsAny())).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 } } diff --git a/OptimizelySDK.Tests/ProjectConfigTest.cs b/OptimizelySDK.Tests/ProjectConfigTest.cs index f7a055db..18fa24d8 100644 --- a/OptimizelySDK.Tests/ProjectConfigTest.cs +++ b/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") }, diff --git a/OptimizelySDK.Tests/TestData.json b/OptimizelySDK.Tests/TestData.json index 15411695..500ab885 100644 --- a/OptimizelySDK.Tests/TestData.json +++ b/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": "", diff --git a/OptimizelySDK/Optimizely.cs b/OptimizelySDK/Optimizely.cs index b687ec39..9f3ab32f 100644 --- a/OptimizelySDK/Optimizely.cs +++ b/OptimizelySDK/Optimizely.cs @@ -591,7 +591,6 @@ public List GetEnabledFeatures(string userId, UserAttributes userAttribu enabledFeaturesList.Add(featureKey); } - enabledFeaturesList.Sort(StringComparer.OrdinalIgnoreCase); return enabledFeaturesList; }