Index: dashboard/dashboard/pinpoint/models/quest/find_isolate_test.py |
diff --git a/dashboard/dashboard/pinpoint/models/quest/find_isolate_test.py b/dashboard/dashboard/pinpoint/models/quest/find_isolate_test.py |
index 4ba0c97fea5dc70ee80bb1ac0981afb6cae61bfa..80fb3059ae3d366de7bd6b31bd54722599cbe19e 100644 |
--- a/dashboard/dashboard/pinpoint/models/quest/find_isolate_test.py |
+++ b/dashboard/dashboard/pinpoint/models/quest/find_isolate_test.py |
@@ -113,15 +113,17 @@ class BuilderLookupTest(_FindIsolateTest): |
@mock.patch('dashboard.services.buildbucket_service.Put') |
class BuildTest(_FindIsolateTest): |
- def testBuildLifecycle(self, put, get_job_status): |
+ @mock.patch.object(change_module.GerritPatch, 'BuildParameters') |
+ def testBuildLifecycle(self, build_parameters, put, get_job_status): |
change = change_module.Change( |
(change_module.Commit('src', 'base git hash'), |
change_module.Commit('v8', 'dep git hash')), |
- patch=change_module.Patch('https://example.org', 2565263002, 20001)) |
+ patch=change_module.GerritPatch('https://example.org', 672011, '2f0d')) |
quest = find_isolate.FindIsolate('Mac Pro Perf', 'telemetry_perf_tests') |
execution = quest.Start(change) |
# Request a build. |
+ build_parameters.return_value = {'patch_storage': 'gerrit'} |
put.return_value = {'build': {'id': 'build_id'}} |
execution.Poll() |
@@ -134,10 +136,7 @@ class BuildTest(_FindIsolateTest): |
'deps_revision_overrides': { |
'https://chromium.googlesource.com/v8/v8': 'dep git hash', |
}, |
- 'patch_storage': 'rietveld', |
- 'rietveld': 'https://example.org', |
- 'issue': 2565263002, |
- 'patchset': 20001, |
+ 'patch_storage': 'gerrit', |
} |
}) |
@@ -193,9 +192,7 @@ class BuildTest(_FindIsolateTest): |
def testBuildFailure(self, put, get_job_status): |
change = change_module.Change( |
- (change_module.Commit('src', 'base git hash'), |
- change_module.Commit('v8', 'dep git hash')), |
- patch=change_module.Patch('https://example.org', 2565263002, 20001)) |
+ (change_module.Commit('src', 'base git hash'),)) |
quest = find_isolate.FindIsolate('Mac Pro Perf', 'telemetry_perf_tests') |
execution = quest.Start(change) |
@@ -217,9 +214,7 @@ class BuildTest(_FindIsolateTest): |
def testBuildCanceled(self, put, get_job_status): |
change = change_module.Change( |
- (change_module.Commit('src', 'base git hash'), |
- change_module.Commit('v8', 'dep git hash')), |
- patch=change_module.Patch('https://example.org', 2565263002, 20001)) |
+ (change_module.Commit('src', 'base git hash'),)) |
quest = find_isolate.FindIsolate('Mac Pro Perf', 'telemetry_perf_tests') |
execution = quest.Start(change) |
@@ -241,9 +236,7 @@ class BuildTest(_FindIsolateTest): |
def testBuildSucceededButIsolateIsMissing(self, put, get_job_status): |
change = change_module.Change( |
- (change_module.Commit('src', 'base git hash'), |
- change_module.Commit('v8', 'dep git hash')), |
- patch=change_module.Patch('https://example.org', 2565263002, 20001)) |
+ (change_module.Commit('src', 'base git hash'),)) |
quest = find_isolate.FindIsolate('Mac Pro Perf', 'telemetry_perf_tests') |
execution = quest.Start(change) |