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

Unified Diff: tests/git_cl_test.py

Issue 2375393002: Revert of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (Closed)
Patch Set: Created 4 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
« no previous file with comments | « git_cl.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/git_cl_test.py
diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py
index 2985d36c6e1eb6c5a96d7600814ff5d1ee472967..1bb517e38223773a78a2e63fbeba80e6c5ecfbc9 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -884,6 +884,7 @@
calls += [
((['git', 'rev-list',
expected_upstream_ref + '..' + ref_to_push],), ''),
+ ((['git', 'config', 'rietveld.cc'],), '')
]
notify_suffix = 'notify=%s' % ('ALL' if notify else 'NONE')
@@ -891,6 +892,9 @@
ref_suffix += ',' + notify_suffix
else:
ref_suffix = '%' + notify_suffix
+
+ # Add cc from watch list.
+ ref_suffix += ',cc=joe@example.com'
if reviewers:
ref_suffix += ',' + ','.join('r=%s' % email
@@ -921,12 +925,6 @@
((['git', 'config', 'branch.master.gerritsquashhash',
'abcdef0123456789'],), ''),
]
- calls += [
- ((['git', 'config', 'rietveld.cc'],), ''),
- ((['AddReviewers', 'chromium-review.googlesource.com',
- 123456 if squash else None,
- ['joe@example.com'], False],), ''),
- ]
calls += cls._git_post_upload_calls()
return calls
@@ -960,9 +958,6 @@
self.mock(git_cl.gclient_utils, 'RunEditor',
lambda *_, **__: self._mocked_call(['RunEditor']))
self.mock(git_cl, 'DownloadGerritHook', self._mocked_call)
- self.mock(git_cl.gerrit_util, 'AddReviewers',
- lambda h, i, add, is_reviewer: self._mocked_call(
- ['AddReviewers', h, i, add, is_reviewer]))
self.calls = self._gerrit_base_calls(issue=issue)
self.calls += self._gerrit_upload_calls(
« 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