[pinpoint] Change refactor.

dashboard/dashboard/pinpoint/handlers/isolate_test.py

Index: dashboard/dashboard/pinpoint/handlers/isolate_test.py
diff --git a/dashboard/dashboard/pinpoint/handlers/isolate_test.py b/dashboard/dashboard/pinpoint/handlers/isolate_test.py
index a4da78f403dc5e7e8e841e439050623de7af0a26..7d8648f74b37fe001e4d0ec0123af66c9b68e7a4 100644
--- a/dashboard/dashboard/pinpoint/handlers/isolate_test.py
+++ b/dashboard/dashboard/pinpoint/handlers/isolate_test.py
@@ -43,7 +43,7 @@ class FunctionalityTest(_IsolateTest):
     testing_common.SetIpWhitelist(['remote_ip'])
 
     builder_name = 'Mac Builder'
-    change = '{"base_commit": {"repository": "src", "git_hash": "git hash"}}'
+    change = '{"commits": [{"repository": "src", "git_hash": "git hash"}]}'
     target = 'telemetry_perf_tests'
     isolate_hash = 'a0c28d99182661887feac644317c94fa18eccbbb'
 
@@ -65,7 +65,7 @@ class FunctionalityTest(_IsolateTest):
   def testGetUnknownIsolate(self):
     params = {
         'builder_name': 'Mac Builder',
-        'change': '{"base_commit": {"repository": "src", "git_hash": "hash"}}',
+        'change': '{"commits": [{"repository": "src", "git_hash": "hash"}]}',
         'target': 'not a real target',
     }
     self.testapp.get('/isolate', params, status=404)
@@ -80,7 +80,7 @@ class ParameterValidationTest(_IsolateTest):
   def testExtraParameter(self):
     params = {
         'builder_name': 'Mac Builder',
-        'change': '{"base_commit": {"repository": "src", "git_hash": "hash"}}',
+        'change': '{"commits": [{"repository": "src", "git_hash": "hash"}]}',
         'target': 'telemetry_perf_tests',
         'extra_parameter': '',
     }
@@ -89,14 +89,14 @@ class ParameterValidationTest(_IsolateTest):
   def testMissingParameter(self):
     params = {
         'builder_name': 'Mac Builder',
-        'change': '{"base_commit": {"repository": "src", "git_hash": "hash"}}',
+        'change': '{"commits": [{"repository": "src", "git_hash": "hash"}]}',
     }
     self.testapp.get('/isolate', params, status=400)
 
   def testEmptyParameter(self):
     params = {
         'builder_name': 'Mac Builder',
-        'change': '{"base_commit": {"repository": "src", "git_hash": "hash"}}',
+        'change': '{"commits": [{"repository": "src", "git_hash": "hash"}]}',
         'target': '',
     }
     self.testapp.get('/isolate', params, status=400)
@@ -112,7 +112,7 @@ class ParameterValidationTest(_IsolateTest):
   def testBadChange(self):
     params = {
         'builder_name': 'Mac Builder',
-        'change': '{"base_commit": {}}',
+        'change': '{"commits": [{}]}',
         'target': 'telemetry_perf_tests',
     }
     self.testapp.get('/isolate', params, status=400)
@@ -120,7 +120,7 @@ class ParameterValidationTest(_IsolateTest):
   def testGetInvalidChangeBecauseOfUnknownRepository(self):
     params = {
         'builder_name': 'Mac Builder',
-        'change': '{"base_commit": {"repository": "foo", "git_hash": "hash"}}',
+        'change': '{"commits": [{"repository": "foo", "git_hash": "hash"}]}',
         'target': 'telemetry_perf_tests',
     }
     self.testapp.get('/isolate', params, status=400)
@@ -130,7 +130,7 @@ class ParameterValidationTest(_IsolateTest):
 
     params = {
         'builder_name': 'Mac Builder',
-        'change': '{"base_commit": {"repository": "foo", "git_hash": "hash"}}',
+        'change': '{"commits": [{"repository": "foo", "git_hash": "hash"}]}',
         'isolate_map': '{"telemetry_perf_tests": "a0c28d9"}',
     }
     self.testapp.post('/isolate', params, status=400)