Remove sorting

Author: msohailhussain

optimizely/optimizely.py

@@ -409,7 +409,7 @@ def get_enabled_features(self, user_id, attributes=None):
       attributes: Dict representing user attributes.
 
     Returns:
-      A sorted list of the keys of the features that are enabled for the user.
+      A list of the keys of the features that are enabled for the user.
     """
 
     enabled_features = []
@@ -421,7 +421,6 @@ def get_enabled_features(self, user_id, attributes=None):
       if self.is_feature_enabled(feature.key, user_id, attributes):
         enabled_features.append(feature.key)
 
-    enabled_features.sort()
     return enabled_features
 
   def get_feature_variable_boolean(self, feature_key, variable_key, user_id, attributes=None):

tests/test_optimizely.py

@@ -1387,29 +1387,6 @@ def side_effect(*args, **kwargs):
     mock_is_feature_enabled.assert_any_call('test_feature_in_group', 'user_1', None)
     mock_is_feature_enabled.assert_any_call('test_feature_in_experiment_and_rollout', 'user_1', None)
 
-  def test_get_enabled_features_returns_a_sorted_list(self):
-    """ Test that get_enabled_features returns a sorted list of enabled feature keys. """
-
-    opt_obj = optimizely.Optimizely(json.dumps(self.config_dict_with_features))
-
-    with mock.patch('optimizely.optimizely.Optimizely.is_feature_enabled',
-                    return_value=True) as mock_is_feature_enabled:
-      received_features = opt_obj.get_enabled_features('user_1')
-
-    mock_is_feature_enabled.assert_any_call('test_feature_in_experiment', 'user_1', None)
-    mock_is_feature_enabled.assert_any_call('test_feature_in_rollout', 'user_1', None)
-    mock_is_feature_enabled.assert_any_call('test_feature_in_group', 'user_1', None)
-    mock_is_feature_enabled.assert_any_call('test_feature_in_experiment_and_rollout', 'user_1', None)
-
-    expected_sorted_features = [
-      'test_feature_in_experiment',
-      'test_feature_in_experiment_and_rollout',
-      'test_feature_in_group',
-      'test_feature_in_rollout'
-      ]
-
-    self.assertEqual(expected_sorted_features, received_features)
-
   def test_get_enabled_features__invalid_object(self):
     """ Test that get_enabled_features returns empty list if Optimizely object is not valid. """