git cl set-commit: fix for Rietveld if cq dry run is active.
If dry-run was active, git cl set-commit didn't actually trigger a
full CQ run, but was just a no-op.
TBR=machenbach@chromium.org
BUG=
Review-Url: https://codereview.chromium.org/2184643004
diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py
index a4a7a58..286290f 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -1466,8 +1466,8 @@
self.assertIsNone(cl.EnsureAuthenticated(force=False))
def test_cmd_set_commit_rietveld(self):
- self.mock(git_cl._RietveldChangelistImpl, 'SetFlag',
- lambda _, f, v: self._mocked_call(['SetFlag', f, v]))
+ self.mock(git_cl._RietveldChangelistImpl, 'SetFlags',
+ lambda _, v: self._mocked_call(['SetFlags', v]))
self.calls = [
((['git', 'symbolic-ref', 'HEAD'],), 'feature'),
((['git', 'config', 'branch.feature.rietveldissue'],), '123'),
@@ -1476,7 +1476,7 @@
((['git', 'config', 'rietveld.server'],), ''),
((['git', 'config', 'branch.feature.rietveldserver'],),
'https://codereview.chromium.org'),
- ((['SetFlag', 'commit', '1'], ), ''),
+ ((['SetFlags', {'commit': '1', 'cq_dry_run': '0'}], ), ''),
]
self.assertEqual(0, git_cl.main(['set-commit']))