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

Side by Side Diff: build/android/pylib/instrumentation/setup.py

Issue 21008004: Changes argument passing to use options objects (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes imports, removes intermediate variables Created 7 years, 4 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
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 """Generates test runner factory and tests for instrumentation tests.""" 5 """Generates test runner factory and tests for instrumentation tests."""
6 6
7 import logging 7 import logging
8 import os
9
10 from pylib import android_commands
11 from pylib import constants
12 from pylib.base import base_test_result
13 from pylib.utils import report_results
14 8
15 import test_package 9 import test_package
16 import test_runner 10 import test_runner
17 11
18 12
19 def Setup(test_apk_path, test_apk_jar_path, annotations, exclude_annotations, 13 def Setup(test_options):
20 test_filter, build_type, test_data, save_perf_json,
21 screenshot_failures, tool, wait_for_debugger, disable_assertions,
22 push_deps, cleanup_test_files):
23 """Create and return the test runner factory and tests. 14 """Create and return the test runner factory and tests.
24 15
25 Args: 16 Args:
26 test_apk_path: Path to the test apk file. 17 test_options: An InstrumentationOptions object.
27 test_apk_jar_path: Path to the jar associated with the test apk.
28 annotations: Annotations for the tests.
29 exclude_annotations: Any annotations to exclude from running.
30 test_filter: Filter string for tests.
31 build_type: 'Release' or 'Debug'.
32 test_data: Location of the test data.
33 save_perf_json: Whether or not to save the JSON file from UI perf tests.
34 screenshot_failures: Take a screenshot for a test failure
35 tool: Name of the Valgrind tool.
36 wait_for_debugger: blocks until the debugger is connected.
37 disable_assertions: Whether to disable java assertions on the device.
38 push_deps: If True, push all dependencies to the device.
39 cleanup_test_files: Whether or not to cleanup test files on device.
40 18
41 Returns: 19 Returns:
42 A tuple of (TestRunnerFactory, tests). 20 A tuple of (TestRunnerFactory, tests).
43 """ 21 """
44 test_pkg = test_package.TestPackage(test_apk_path, test_apk_jar_path) 22 test_pkg = test_package.TestPackage(test_options.test_apk_path,
45 tests = test_pkg._GetAllMatchingTests(annotations, exclude_annotations, 23 test_options.test_apk_jar_path)
46 test_filter) 24 tests = test_pkg._GetAllMatchingTests(
25 test_options.annotations,
26 test_options.exclude_annotations,
27 test_options.test_filter)
47 if not tests: 28 if not tests:
48 logging.error('No instrumentation tests to run with current args.') 29 logging.error('No instrumentation tests to run with current args.')
49 30
50 def TestRunnerFactory(device, shard_index): 31 def TestRunnerFactory(device, shard_index):
51 return test_runner.TestRunner( 32 return test_runner.TestRunner(test_options, device, shard_index,
52 build_type, test_data, save_perf_json, screenshot_failures, 33 test_pkg, [])
53 tool, wait_for_debugger, disable_assertions, push_deps,
54 cleanup_test_files, device, shard_index, test_pkg, [])
55 34
56 return (TestRunnerFactory, tests) 35 return (TestRunnerFactory, tests)
OLDNEW
« no previous file with comments | « build/android/pylib/host_driven/run_python_tests.py ('k') | build/android/pylib/instrumentation/test_options.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698