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

Unified Diff: tests/checkout_test.py

Issue 7044090: Make post_processors part of the Checkout object state. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 9 years, 6 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 | « checkout.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/checkout_test.py
diff --git a/tests/checkout_test.py b/tests/checkout_test.py
index 59a3f529a6404f3839f48dff0bf17a714fa6b2eb..4ea73a06eb6c1ec67a7aa827c6de6e632555287d 100755
--- a/tests/checkout_test.py
+++ b/tests/checkout_test.py
@@ -236,10 +236,11 @@ class BaseTest(fake_repos.FakeReposTestBase):
def _test_process(self, co):
"""Makes sure the process lambda is called correctly."""
+ co.post_processors = [lambda *args: results.append(args)]
co.prepare()
ps = self.get_patches()
results = []
- co.apply_patch(ps, [lambda *args: results.append(args)])
+ co.apply_patch(ps)
expected = [(co, p) for p in ps.patches]
self.assertEquals(expected, results)
@@ -479,7 +480,7 @@ class RawCheckout(SvnBaseTest):
self.base_co.prepare()
def _get_co(self, read_only):
- co = checkout.RawCheckout(self.root_dir, self.name)
+ co = checkout.RawCheckout(self.root_dir, self.name, None)
if read_only:
return checkout.ReadOnlyCheckout(co)
return co
« no previous file with comments | « checkout.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698