Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1544)

Unified Diff: dashboard/dashboard/pinpoint/models/quest/find_isolate_test.py

Issue 3019553002: [pinpoint] Gerrit patch support. (Closed)
Patch Set: Example URL Created 3 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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)

Powered by Google App Engine
This is Rietveld 408576698