Fix contruction of page and URL result metric IDs

Author: robingustafsson

loadimpact/resources.py

@@ -314,8 +314,8 @@ def result_id_for_page(cls, page_name, load_zone_id, user_scenario_id):
         else:
             if isinstance(page_name, unicode):
                 page_name = page_name.encode('utf-8')
-        return '__li_page%s:%s:%s' % (hashlib.md5(page_name).hexdigest(),
-                                      str(load_zone_id), str(user_scenario_id))
+        return '__li_page_%s:%s:%s' % (hashlib.md5(page_name).hexdigest(),
+                                       str(load_zone_id), str(user_scenario_id))
 
     @classmethod
     def result_id_for_url(cls, url, load_zone_id, user_scenario_id,
@@ -325,10 +325,10 @@ def result_id_for_url(cls, url, load_zone_id, user_scenario_id,
         else:
             if isinstance(url, unicode):
                 url = url.encode('utf-8')
-        return '__li_url%s:%s:%s:%s:%s' % (hashlib.md5(url).hexdigest(),
-                                           str(load_zone_id),
-                                           str(user_scenario_id),
-                                           str(status_code), method)
+        return '__li_url_%s:%s:%s:%s:%s' % (hashlib.md5(url).hexdigest(),
+                                            str(load_zone_id),
+                                            str(user_scenario_id),
+                                            str(status_code), method)
 
 
 class _TestResultStream(Resource):

test/test_resources.py

@@ -290,7 +290,7 @@ def test_result_id_for_page(self):
         result_id = TestResult.result_id_for_page(name, 1, 1)
         if sys.version_info >= (3, 0) and isinstance(name, str):
             name = name.encode('utf-8')
-        self.assertEqual(result_id, '__li_page%s:1:1'
+        self.assertEqual(result_id, '__li_page_%s:1:1'
                                     % hashlib.md5(name).hexdigest())
 
     def test_result_id_for_url(self):
@@ -299,7 +299,7 @@ def test_result_id_for_url(self):
                                                  status_code=200)
         if sys.version_info >= (3, 0) and isinstance(url, str):
             url = url.encode('utf-8')
-        self.assertEqual(result_id, '__li_url%s:1:1:200:GET'
+        self.assertEqual(result_id, '__li_url_%s:1:1:200:GET'
                                     % hashlib.md5(url).hexdigest())