[pinpoint] Change refactor.

dashboard/dashboard/pinpoint/handlers/isolate_test.py

 # Copyright 2016 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import json
 
 import mock
 import webapp2
 import webtest
 
@@ skipping 25 matching lines @@
         }
     })
 
 
 class FunctionalityTest(_IsolateTest):
 
   def testPostAndGet(self):
     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'
 
     params = {
         'builder_name': builder_name,
         'change': change,
         'isolate_map': json.dumps({target: isolate_hash}),
     }
     self.testapp.post('/isolate', params, status=200)
 
     params = {
         'builder_name': builder_name,
         'change': change,
         'target': target,
     }
     response = self.testapp.get('/isolate', params, status=200)
     self.assertEqual(response.normal_body, isolate_hash)
 
   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)
 
   def testPostPermissionDenied(self):
     testing_common.SetIpWhitelist([])
     self.testapp.post('/isolate', status=403)
 
 
 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': '',
     }
     self.testapp.get('/isolate', params, status=400)
 
   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)
 
   def testBadJson(self):
     params = {
         'builder_name': 'Mac Builder',
         'change': '',
         'target': 'telemetry_perf_tests',
     }
     self.testapp.get('/isolate', params, status=400)
 
   def testBadChange(self):
     params = {
         'builder_name': 'Mac Builder',
-        'change': '{"base_commit": {}}',
+        'change': '{"commits": [{}]}',
         'target': 'telemetry_perf_tests',
     }
     self.testapp.get('/isolate', params, status=400)
 
   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)
 
   def testPostInvalidChangeBecauseOfUnknownRepository(self):
     testing_common.SetIpWhitelist(['remote_ip'])
 
     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)