Sorted the event features results in the test so we don't have to sor…

…t it in the event builder.

optimizely/event_builder.py

@@ -214,8 +214,7 @@ def _add_required_params_for_conversion(self, event_key, event_tags, decisions):
         })
 
       self.params[self.EventParams.EVENT_METRICS] = event_values
-
-      for event_tag_id in sorted(event_tags.keys()):
+      for event_tag_id in event_tags.keys():
         event_tag_value = event_tags.get(event_tag_id)
         if event_tag_value is None:
           continue

tests/test_optimizely.py

@@ -692,6 +692,12 @@ def test_track__with_deprecated_event_value(self):
       }],
       'accountId': '12001'
     }
+
+    # Sort event features based on ID
+    mock_dispatch_event.call_args[0][0].params['eventFeatures'] = sorted(
+      mock_dispatch_event.call_args[0][0].params['eventFeatures'], key=lambda x: x.get('name')
+    )
+
     mock_get_variation.assert_called_once_with(self.project_config.get_experiment_from_key('test_experiment'),
                                                'test_user', {'test_attribute': 'test_value'})
     self.assertEqual(1, mock_dispatch_event.call_count)
@@ -752,6 +758,10 @@ def test_track__with_invalid_event_tags(self):
       'accountId': '12001'
     }
 
+    # Sort event features based on ID
+    mock_dispatch_event.call_args[0][0].params['eventFeatures'] = sorted(
+      mock_dispatch_event.call_args[0][0].params['eventFeatures'], key=lambda x: x.get('name')
+    )
     mock_get_variation.assert_called_once_with(self.project_config.get_experiment_from_key('test_experiment'),
                                                'test_user', {'test_attribute': 'test_value'})
     self.assertEqual(1, mock_dispatch_event.call_count)