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

Side by Side Diff: scripts/slave/recipe_modules/chromium_android/api.py

Issue 1857273003: Allow run_instrumentation_suite to use --test-apk with a wrapper script (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 8 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import contextlib 5 import contextlib
6 import datetime 6 import datetime
7 import json 7 import json
8 import os 8 import os
9 import re 9 import re
10 import urllib 10 import urllib
(...skipping 695 matching lines...) Expand 10 before | Expand all | Expand 10 after
706 if json_results_file: 706 if json_results_file:
707 args.extend(['--json-results-file', json_results_file]) 707 args.extend(['--json-results-file', json_results_file])
708 if timeout_scale: 708 if timeout_scale:
709 args.extend(['--timeout-scale', timeout_scale]) 709 args.extend(['--timeout-scale', timeout_scale])
710 if strict_mode: 710 if strict_mode:
711 args.extend(['--strict-mode', strict_mode]) 711 args.extend(['--strict-mode', strict_mode])
712 if num_retries is not None: 712 if num_retries is not None:
713 args.extend(['--num-retries', str(num_retries)]) 713 args.extend(['--num-retries', str(num_retries)])
714 if device_flags: 714 if device_flags:
715 args.extend(['--device-flags', device_flags]) 715 args.extend(['--device-flags', device_flags])
716 if test_apk:
717 args.extend(['--test-apk', test_apk])
718 if apk_under_test:
719 args.extend(['--apk-under-test', apk_under_test])
716 720
717 if not wrapper_script_suite_name: 721 if not wrapper_script_suite_name:
718 args.insert(0, 'instrumentation') 722 args.insert(0, 'instrumentation')
719 args.extend(['--test-apk', test_apk])
720 if apk_under_test:
721 args.extend(['--apk-under-test', apk_under_test])
722 for a in additional_apks or []: 723 for a in additional_apks or []:
723 args.extend(['--additional-apk', a]) 724 args.extend(['--additional-apk', a])
724 if isolate_file_path: 725 if isolate_file_path:
725 args.extend(['--isolate-file-path', isolate_file_path]) 726 args.extend(['--isolate-file-path', isolate_file_path])
726 if self.m.chromium.c.BUILD_CONFIG == 'Release': 727 if self.m.chromium.c.BUILD_CONFIG == 'Release':
727 args.append('--release') 728 args.append('--release')
728 if official_build: 729 if official_build:
729 args.extend(['--official-build']) 730 args.extend(['--official-build'])
730 731
731 step_result = self.test_runner( 732 step_result = self.test_runner(
(...skipping 527 matching lines...) Expand 10 before | Expand all | Expand 10 after
1259 Args: 1260 Args:
1260 step_name: Name of the step. 1261 step_name: Name of the step.
1261 args: Testrunner arguments. 1262 args: Testrunner arguments.
1262 """ 1263 """
1263 with self.handle_exit_codes(): 1264 with self.handle_exit_codes():
1264 script = self.c.test_runner 1265 script = self.c.test_runner
1265 if wrapper_script_suite_name: 1266 if wrapper_script_suite_name:
1266 script = self.m.chromium.output_dir.join('bin', 'run_%s' % 1267 script = self.m.chromium.output_dir.join('bin', 'run_%s' %
1267 wrapper_script_suite_name) 1268 wrapper_script_suite_name)
1268 return self.m.python(step_name, script, args, **kwargs) 1269 return self.m.python(step_name, script, args, **kwargs)
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698