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

Side by Side Diff: tests/git_cl_test.py

Issue 11043031: Allow special java statements (for example imports) to be as long as neccessary. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Fixed existing tests Created 8 years, 2 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « git_cl.py ('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 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Unit tests for git_cl.py.""" 6 """Unit tests for git_cl.py."""
7 7
8 import os 8 import os
9 import StringIO 9 import StringIO
10 import stat 10 import stat
(...skipping 403 matching lines...) Expand 10 before | Expand all | Expand 10 after
414 description, 414 description,
415 reviewers): 415 reviewers):
416 """Generic gerrit reviewer test framework.""" 416 """Generic gerrit reviewer test framework."""
417 self.calls = self._gerrit_base_calls() 417 self.calls = self._gerrit_base_calls()
418 self.calls += self._gerrit_upload_calls(description, reviewers) 418 self.calls += self._gerrit_upload_calls(description, reviewers)
419 git_cl.main(['upload'] + upload_args) 419 git_cl.main(['upload'] + upload_args)
420 420
421 def test_gerrit_no_reviewer(self): 421 def test_gerrit_no_reviewer(self):
422 self._run_gerrit_reviewer_test( 422 self._run_gerrit_reviewer_test(
423 [], 423 [],
424 'desc\n\nBUG=\n', 424 'desc\n\nBUG=\nChange-Id:123456789\n',
425 []) 425 [])
426 426
427 def test_gerrit_reviewers_cmd_line(self): 427 def test_gerrit_reviewers_cmd_line(self):
428 self._run_gerrit_reviewer_test( 428 self._run_gerrit_reviewer_test(
429 ['-r', 'foo@example.com'], 429 ['-r', 'foo@example.com'],
430 'desc\n\nBUG=\n', 430 'desc\n\nBUG=\nChange-Id:123456789',
431 ['foo@example.com']) 431 ['foo@example.com'])
432 432
433 def test_gerrit_reviewer_multiple(self): 433 def test_gerrit_reviewer_multiple(self):
434 self._run_gerrit_reviewer_test( 434 self._run_gerrit_reviewer_test(
435 [], 435 [],
436 'desc\nTBR=reviewer@example.com\nBUG=\nR=another@example.com\n', 436 'desc\nTBR=reviewer@example.com\nBUG=\nR=another@example.com\n'
437 'Change-Id:123456789\n',
437 ['reviewer@example.com', 'another@example.com']) 438 ['reviewer@example.com', 'another@example.com'])
438 439
439 440
440 def test_config_gerrit_download_hook(self): 441 def test_config_gerrit_download_hook(self):
441 self.mock(git_cl, 'FindCodereviewSettingsFile', CodereviewSettingsFileMock) 442 self.mock(git_cl, 'FindCodereviewSettingsFile', CodereviewSettingsFileMock)
442 def ParseCodereviewSettingsContent(content): 443 def ParseCodereviewSettingsContent(content):
443 keyvals = {} 444 keyvals = {}
444 keyvals['CODE_REVIEW_SERVER'] = 'gerrit.chromium.org' 445 keyvals['CODE_REVIEW_SERVER'] = 'gerrit.chromium.org'
445 keyvals['GERRIT_HOST'] = 'gerrit.chromium.org' 446 keyvals['GERRIT_HOST'] = 'gerrit.chromium.org'
446 keyvals['GERRIT_PORT'] = '29418' 447 keyvals['GERRIT_PORT'] = '29418'
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
489 ((['git', 'config', 'rietveld.viewvc-url'],), ''), 490 ((['git', 'config', 'rietveld.viewvc-url'],), ''),
490 (('ViewVC URL:',), ''), 491 (('ViewVC URL:',), ''),
491 # DownloadHooks(True) 492 # DownloadHooks(True)
492 ((commit_msg_path, os.X_OK,), True), 493 ((commit_msg_path, os.X_OK,), True),
493 ] 494 ]
494 git_cl.main(['config']) 495 git_cl.main(['config'])
495 496
496 497
497 if __name__ == '__main__': 498 if __name__ == '__main__':
498 unittest.main() 499 unittest.main()
OLDNEW
« no previous file with comments | « git_cl.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698