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

Unified Diff: scripts/slave/recipe_modules/android/api.py

Issue 23889036: Refactor the way that TestApi works so that it is actually useful. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Move gclient test_api to got_revisions cl 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
Index: scripts/slave/recipe_modules/android/api.py
diff --git a/scripts/slave/recipe_modules/android/api.py b/scripts/slave/recipe_modules/android/api.py
index e47ec0e529d9ecbf2a81d244c7aec26bc86c9e5d..347f6e10febe25650a29c17d487e520fa6a63801 100644
--- a/scripts/slave/recipe_modules/android/api.py
+++ b/scripts/slave/recipe_modules/android/api.py
@@ -46,10 +46,20 @@ class AOSPApi(recipe_api.RecipeApi):
yield self.m.step(
'calculate trimmed deps',
- [self.m.path.checkout('android_webview', 'buildbot', 'deps_whitelist.py'),
- '--method', 'android_build',
- '--path-to-deps', self.m.path.checkout('DEPS'),
- self.m.json.output()])
+ [
+ self.m.path.checkout('android_webview', 'buildbot',
+ 'deps_whitelist.py'),
+ '--method', 'android_build',
+ '--path-to-deps', self.m.path.checkout('DEPS'),
+ self.m.json.output()
+ ],
+ mock_fn=lambda: self.test_api.m.json.output({
+ 'blacklist': {
+ 'src/blacklist/project/1': None,
+ 'src/blacklist/project/2': None,
+ }
+ })
+ )
spec = self.m.gclient.make_config('chromium_bare')
deps_blacklist = self.m.step_history.last_step().json.output['blacklist']

Powered by Google App Engine
This is Rietveld 408576698