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

Side by Side Diff: recipe_modules/properties/test_api.py

Issue 2465703003: Fix issue overriding bug properties.test_api.tryserver for Gerrit. (Closed)
Patch Set: space Created 4 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « recipe_modules/properties/example.expected/buildbot_tryserver_gerrit_override_both.json ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The LUCI Authors. All rights reserved. 1 # Copyright 2015 The LUCI Authors. All rights reserved.
2 # Use of this source code is governed under the Apache License, Version 2.0 2 # Use of this source code is governed under the Apache License, Version 2.0
3 # that can be found in the LICENSE file. 3 # that can be found in the LICENSE file.
4 4
5 import re 5 import re
6 import urlparse 6 import urlparse
7 7
8 from recipe_engine import recipe_test_api 8 from recipe_engine import recipe_test_api
9 9
10 class PropertiesTestApi(recipe_test_api.RecipeTestApi): 10 class PropertiesTestApi(recipe_test_api.RecipeTestApi):
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 parsed = list(urlparse.urlparse(git_url)) 78 parsed = list(urlparse.urlparse(git_url))
79 m = re.match(r'^((\w+)(-\w+)*).googlesource.com$', parsed[1]) 79 m = re.match(r'^((\w+)(-\w+)*).googlesource.com$', parsed[1])
80 if not m: # pragma: no cover 80 if not m: # pragma: no cover
81 raise AssertionError('Can\'t guess gerrit_url from git_url "%s", ' 81 raise AssertionError('Can\'t guess gerrit_url from git_url "%s", '
82 'specify it as extra kwarg' % parsed[1]) 82 'specify it as extra kwarg' % parsed[1])
83 parsed[1] = m.group(1) + '-review.googlesource.com' 83 parsed[1] = m.group(1) + '-review.googlesource.com'
84 gerrit_url = urlparse.urlunparse(parsed[:2] + [''] * len(parsed[2:])) 84 gerrit_url = urlparse.urlunparse(parsed[:2] + [''] * len(parsed[2:]))
85 assert project 85 assert project
86 assert git_url 86 assert git_url
87 assert gerrit_url 87 assert gerrit_url
88 # Pop old style values from kwargs. 88 # Support old and new style patch{set,issue} specification.
89 patch_issue = int(kwargs.pop('issue', 456789)) 89 patch_issue = int(kwargs.pop('issue', kwargs.pop('patch_issue', 456789)))
90 patch_set = int(kwargs.pop('patchset', 12)) 90 patch_set = int(kwargs.pop('patchset', kwargs.pop('patch_set', 12)))
91 # Note that new Gerrit patch properties all start with 'patch_' prefix. 91 # Note that new Gerrit patch properties all start with 'patch_' prefix.
92 ret = self.generic( 92 ret = self.generic(
93 patch_storage='gerrit', 93 patch_storage='gerrit',
94 patch_gerrit_url=gerrit_url, 94 patch_gerrit_url=gerrit_url,
95 patch_project=project, 95 patch_project=project,
96 patch_branch='master', 96 patch_branch='master',
97 patch_issue=patch_issue, 97 patch_issue=patch_issue,
98 patch_set=patch_set, 98 patch_set=patch_set,
99 patch_repository_url=git_url, 99 patch_repository_url=git_url,
100 patch_ref='refs/changes/%2d/%d/%d' % ( 100 patch_ref='refs/changes/%2d/%d/%d' % (
(...skipping 19 matching lines...) Expand all
120 patchset=1, 120 patchset=1,
121 project='chrome', 121 project='chrome',
122 repository='', 122 repository='',
123 requester='commit-bot@chromium.org', 123 requester='commit-bot@chromium.org',
124 revision='HEAD', 124 revision='HEAD',
125 rietveld='https://codereview.chromium.org', 125 rietveld='https://codereview.chromium.org',
126 patch_project='chromium', 126 patch_project='chromium',
127 ) 127 )
128 ret.properties.update(kwargs) 128 ret.properties.update(kwargs)
129 return ret 129 return ret
OLDNEW
« no previous file with comments | « recipe_modules/properties/example.expected/buildbot_tryserver_gerrit_override_both.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698