#200: fixed flickering test

Author: bugy

src/communications/alerts_service.py

@@ -1,4 +1,5 @@
 import logging
+from collections import OrderedDict
 
 from communications import destination_base
 from communications.communicaton_service import CommunicationsService
@@ -49,7 +50,7 @@ def __init__(self, params_dict):
         self._destination = http.HttpDestination(params_dict)
 
     def send(self, title, body, files=None):
-        data = {'title': title, 'message': body}
+        data = OrderedDict([('title', title), ('message', body)])
 
         self._destination.send(title, data, files=files)
 

src/tests/alerts_service_test.py

@@ -2,6 +2,7 @@
 import json
 import sys
 import unittest
+from collections import OrderedDict
 from unittest import mock
 
 from communications import alerts_service
@@ -138,12 +139,12 @@ def test_send_http_alert(self):
 
         self.send_alert(alerts_service, body, files, title)
 
-        expected_body = json.dumps({
-            'title': title,
-            'message': body,
-            'files': {
+        expected_body = json.dumps(OrderedDict([
+            ('title', title),
+            ('message', body),
+            ('files', {
                 'log.txt': 'doing X'
-            }})
+            })]))
         self.assertEqual([(None, expected_body, None)], self.get_communicators()[0].messages)
 
     def test_import_alerts_service_with_missing_dependencies(self):