| OLD | NEW | 
|---|
| 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  Loading... | 
| 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  Loading... | 
| 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 | 
| OLD | NEW | 
|---|