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

Unified Diff: tests/try_server_test.py

Issue 23483019: Changes to support One-Click Revert in Commit Queue (Closed) Base URL: https://src.chromium.org/chrome/trunk/tools/commit-queue/
Patch Set: Created 7 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 | « tests/project_test.py ('k') | verification/committer_revert.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/try_server_test.py
===================================================================
--- tests/try_server_test.py (revision 220012)
+++ tests/try_server_test.py (working copy)
@@ -462,21 +462,25 @@
'builders/linux/builds/_all', 'builders/mac/builds/_all'])
self.context.checkout.check_calls(
[ 'prepare(123)',
- 'apply_patch(%r)' % self.context.rietveld.patchsets[-2],
+ 'apply_patch(%r, revert=False)' % (
+ self.context.rietveld.patchsets[-2]),
'prepare(123)',
- 'apply_patch(%r)' % self.context.rietveld.patchsets[-1]])
+ 'apply_patch(%r, revert=False)' % (
+ self.context.rietveld.patchsets[-1])])
elif is_failure(status_linux):
self.try_server.check_calls(
['builders/?select=linux', 'builders/linux/builds/_all'])
self.context.checkout.check_calls(
[ 'prepare(123)',
- 'apply_patch(%r)' % self.context.rietveld.patchsets[-1]])
+ 'apply_patch(%r, revert=False)' % (
+ self.context.rietveld.patchsets[-1])])
else:
self.try_server.check_calls(
['builders/?select=mac', 'builders/mac/builds/_all'])
self.context.checkout.check_calls(
[ 'prepare(123)',
- 'apply_patch(%r)' % self.context.rietveld.patchsets[-1]])
+ 'apply_patch(%r, revert=False)' % (
+ self.context.rietveld.patchsets[-1])])
else:
self.assertPending(base.SUCCEEDED, 2, None)
self.try_server.check_calls(
@@ -699,7 +703,8 @@
self.try_runner.update_status([self.pending])
self.context.checkout.check_calls(
[ 'prepare(123)',
- 'apply_patch(%r)' % self.context.rietveld.patchsets[-1]])
+ 'apply_patch(%r, revert=False)' % (
+ self.context.rietveld.patchsets[-1])])
self.context.status.check_names(['try server'] * 7)
def testFailedStepRetryLkgr(self):
@@ -719,7 +724,8 @@
self.assertEqual(['test1'], self.get_verif().try_jobs[0].failed_steps)
self.context.checkout.check_calls(
[ 'prepare(122)',
- 'apply_patch(%r)' % self.context.rietveld.patchsets[-1]])
+ 'apply_patch(%r, revert=False)' % (
+ self.context.rietveld.patchsets[-1])])
self.context.status.check_names(['try server'] * 5)
def testFailedUpdate(self):
@@ -766,7 +772,8 @@
self.try_runner.update_status([self.pending])
self.context.checkout.check_calls(
[ 'prepare(122)',
- 'apply_patch(%r)' % self.context.rietveld.patchsets[-1]])
+ 'apply_patch(%r, revert=False)' % (
+ self.context.rietveld.patchsets[-1])])
self.try_server.check_calls(
[ 'builders/?select=linux&select=mac',
'builders/linux/builds/_all', 'builders/mac/builds/_all',
« no previous file with comments | « tests/project_test.py ('k') | verification/committer_revert.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698