Index: tests/git_cl_test.py |
diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py |
index 6f0eea4af071d1e9e42c401571ce40205df31874..b276b0a72d43b24baba89870d17e9ef857457545 100755 |
--- a/tests/git_cl_test.py |
+++ b/tests/git_cl_test.py |
@@ -777,87 +777,6 @@ class TestGitCl(TestCase): |
change_id = git_cl.GenerateGerritChangeId('line1\nline2\n') |
self.assertEqual(change_id, 'Ihashchange') |
- def test_config_gerrit_download_hook(self): |
- self.mock(git_cl, 'FindCodereviewSettingsFile', CodereviewSettingsFileMock) |
- def ParseCodereviewSettingsContent(content): |
- keyvals = {} |
- keyvals['CODE_REVIEW_SERVER'] = 'gerrit.chromium.org' |
- keyvals['GERRIT_HOST'] = 'True' |
- return keyvals |
- self.mock(git_cl.gclient_utils, 'ParseCodereviewSettingsContent', |
- ParseCodereviewSettingsContent) |
- self.mock(git_cl.os, 'access', self._mocked_call) |
- self.mock(git_cl.os, 'chmod', self._mocked_call) |
- src_dir = os.path.join(os.path.sep, 'usr', 'local', 'src') |
- def AbsPath(path): |
- if not path.startswith(os.path.sep): |
- return os.path.join(src_dir, path) |
- return path |
- self.mock(git_cl.os.path, 'abspath', AbsPath) |
- commit_msg_path = os.path.join(src_dir, '.git', 'hooks', 'commit-msg') |
- def Exists(path): |
- if path == commit_msg_path: |
- return False |
- # others paths, such as /usr/share/locale/.... |
- return True |
- self.mock(git_cl.os.path, 'exists', Exists) |
- self.mock(git_cl, 'urlretrieve', self._mocked_call) |
- self.mock(git_cl, 'hasSheBang', self._mocked_call) |
- self.calls = [ |
- ((['git', 'config', 'rietveld.autoupdate'],), |
- ''), |
- ((['git', 'config', 'rietveld.server', |
- 'gerrit.chromium.org'],), ''), |
- ((['git', 'config', '--unset-all', 'rietveld.cc'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.private'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.tree-status-url'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.viewvc-url'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.bug-prefix'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.cpplint-regex'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.force-https-commit-url'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.cpplint-ignore-regex'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.project'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.pending-ref-prefix'],), ''), |
- ((['git', 'config', '--unset-all', |
- 'rietveld.run-post-upload-hook'],), ''), |
- ((['git', 'config', 'gerrit.host', 'True'],), ''), |
- # GetCodereviewSettingsInteractively |
- ((['git', 'config', 'rietveld.server'],), |
- 'gerrit.chromium.org'), |
- (('Rietveld server (host[:port]) [https://gerrit.chromium.org]:',), |
- ''), |
- ((['git', 'config', 'rietveld.cc'],), ''), |
- (('CC list:',), ''), |
- ((['git', 'config', 'rietveld.private'],), ''), |
- (('Private flag (rietveld only):',), ''), |
- ((['git', 'config', 'rietveld.tree-status-url'],), ''), |
- (('Tree status URL:',), ''), |
- ((['git', 'config', 'rietveld.viewvc-url'],), ''), |
- (('ViewVC URL:',), ''), |
- ((['git', 'config', 'rietveld.bug-prefix'],), ''), |
- (('Bug Prefix:',), ''), |
- ((['git', 'config', 'rietveld.run-post-upload-hook'],), ''), |
- (('Run Post Upload Hook:',), ''), |
- # DownloadGerritHook(False) |
- ((['git', 'config', 'gerrit.host'],), 'True'), |
- ((['git', 'rev-parse', '--show-cdup'],), ''), |
- ((commit_msg_path, os.X_OK,), False), |
- (('https://gerrit-review.googlesource.com/tools/hooks/commit-msg', |
- commit_msg_path,), ''), |
- ((commit_msg_path,), True), |
- ((commit_msg_path, stat.S_IRUSR | stat.S_IWUSR | stat.S_IXUSR,), ''), |
- ] |
- git_cl.main(['config']) |
- |
def test_update_reviewers(self): |
data = [ |
('foo', [], 'foo'), |