Chromium Code Reviews| 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() |