 Chromium Code Reviews
 Chromium Code Reviews Issue 10905138:
  Add test jar generation logic for ant builds.  (Closed) 
  Base URL: http://git.chromium.org/chromium/src.git@master
    
  
    Issue 10905138:
  Add test jar generation logic for ant builds.  (Closed) 
  Base URL: http://git.chromium.org/chromium/src.git@master| 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() |