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

Unified Diff: build/android/pylib/test_options_parser.py

Issue 10905138: Add test jar generation logic for ant builds. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: rebase Created 8 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: build/android/pylib/test_options_parser.py
diff --git a/build/android/pylib/test_options_parser.py b/build/android/pylib/test_options_parser.py
index 8492ec3b3c9a0772bf4754840450a9a90cc80471..69bf6bd1c15047d40697130e578377331de7b407 100644
--- a/build/android/pylib/test_options_parser.py
+++ b/build/android/pylib/test_options_parser.py
@@ -85,10 +85,9 @@ def AddInstrumentationOptions(option_parser):
'of the result. (Default is 1)'))
option_parser.add_option('--test-apk', dest='test_apk',
help=('The name of the apk containing the tests '
- '(without the .apk extension) or for SDK '
- 'builds, the path to the APK from '
- 'out/(Debug|Release) (for example, '
- 'content_shell_test/ContentShellTest-debug).'))
+ '(without the .apk extension). For SDK '
+ 'builds, the apk name without the debug'
+ ' suffix(for example, ContentShellTest).'))
cjhopman 2012/09/18 21:40:20 The space should go at the end of the previous lin
option_parser.add_option('--screenshot', dest='screenshot_failures',
action='store_true',
help='Capture screenshots of test failures')
@@ -119,12 +118,15 @@ def ValidateInstrumentationOptions(option_parser, options, args):
elif options.python_only:
options.run_java_tests = False
+ # In case of SDK Build, the jars and apks have a -debug suffix.
options.test_apk_path = os.path.join(_SDK_OUT_DIR,
options.build_type,
- '%s.apk' % options.test_apk)
+ constants.SDK_BUILD_APKS_DIR,
+ '%s-debug.apk' % options.test_apk)
options.test_apk_jar_path = os.path.join(_SDK_OUT_DIR,
options.build_type,
- '%s.jar'
+ constants.SDK_BUILD_TEST_JAVALIB_DIR,
+ '%s-debug.jar'
% options.test_apk)
cjhopman 2012/09/18 21:40:20 Nit: this should fit on the line above and that wo
if options.annotation_str:
options.annotation = options.annotation_str.split()

Powered by Google App Engine
This is Rietveld 408576698