| Index: commit-queue/tests/pending_manager_test.py
|
| ===================================================================
|
| --- commit-queue/tests/pending_manager_test.py (revision 225753)
|
| +++ commit-queue/tests/pending_manager_test.py (working copy)
|
| @@ -113,7 +113,7 @@
|
| 'apply_patch(%r)' % (self.context.rietveld.patchsets[0],)])
|
| self.context.rietveld.check_calls(
|
| [ _try_comment(),
|
| - "set_flag(%d, 1, 'commit', 'False')" % issue,
|
| + "set_flag(%d, 1, 'commit', False)" % issue,
|
| "add_comment(%d, %r)" % (issue, pc.FAILED_NO_MESSAGE)])
|
| else:
|
| if success:
|
| @@ -130,11 +130,11 @@
|
| if defered:
|
| self.context.rietveld.check_calls(
|
| [ _try_comment(),
|
| - "set_flag(%d, 1, 'commit', 'False')" % issue,
|
| + "set_flag(%d, 1, 'commit', False)" % issue,
|
| "add_comment(%d, %r)" % (issue, pc.FAILED_NO_MESSAGE)])
|
| else:
|
| self.context.rietveld.check_calls(
|
| - [ "set_flag(%d, 1, 'commit', 'False')" % issue,
|
| + [ "set_flag(%d, 1, 'commit', False)" % issue,
|
| "add_comment(%d, %r)" % (issue, pc.FAILED_NO_MESSAGE)])
|
|
|
| def _prepare_apply_commit(self, index, issue, server_hooks_missing=False):
|
| @@ -621,7 +621,7 @@
|
| pc.update_status()
|
| self.assertEqual(0, len(pc.queue.iterate()))
|
| self.context.rietveld.check_calls(
|
| - [ "set_flag(%d, 1, 'commit', 'False')" % issue,
|
| + [ "set_flag(%d, 1, 'commit', False)" % issue,
|
| "add_comment(%d, %r)" % (issue, reviewer_lgtm.LgtmStatus.NO_LGTM)])
|
| self.context.status.check_names(['abort'])
|
|
|
| @@ -661,7 +661,7 @@
|
| pc.scan_results()
|
| self.context.rietveld.check_calls(
|
| [ _try_comment(),
|
| - "set_flag(%d, 1, 'commit', 'False')" % issue,
|
| + "set_flag(%d, 1, 'commit', False)" % issue,
|
| ('add_comment(%d, "List of reviewers changed. annoying@dude.org '
|
| 'did a drive-by without LGTM\'ing!")') % issue])
|
| self.context.status.check_names(['initial', 'abort'])
|
|
|