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

Unified Diff: build/android/test_runner.py

Issue 1292053006: Revert of [Android] Add --blacklist-file as a command-line option. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/android/screenshot.py ('k') | build/android/tombstones.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/test_runner.py
diff --git a/build/android/test_runner.py b/build/android/test_runner.py
index 8bae397f261112ebac8721402035fd58a475d08f..7b49c9e69a1f2c181d268393487c3e0aeebdecb4 100755
--- a/build/android/test_runner.py
+++ b/build/android/test_runner.py
@@ -24,7 +24,6 @@
from pylib.base import test_dispatcher
from pylib.base import test_instance_factory
from pylib.base import test_run_factory
-from pylib.device import device_blacklist
from pylib.device import device_errors
from pylib.device import device_utils
from pylib.gtest import gtest_config
@@ -189,7 +188,6 @@
group.add_argument('-d', '--device', dest='test_device',
help=('Target device for the test suite '
'to run on.'))
- group.add_argument('--blacklist-file', help='Device blacklist file.')
def AddGTestOptions(parser):
@@ -769,7 +767,7 @@
"""Subcommand of RunTestsCommands which runs uiautomator tests."""
uiautomator_options = ProcessUIAutomatorOptions(args)
- runner_factory, tests = uiautomator_setup.Setup(uiautomator_options, devices)
+ runner_factory, tests = uiautomator_setup.Setup(uiautomator_options)
results, exit_code = test_dispatcher.RunTests(
tests, runner_factory, devices, shard=True, test_timeout=None,
@@ -825,7 +823,7 @@
return exit_code
-def _RunPerfTests(args, active_devices):
+def _RunPerfTests(args):
"""Subcommand of RunTestsCommands which runs perf tests."""
perf_options = ProcessPerfTestOptions(args)
@@ -839,8 +837,7 @@
return perf_test_runner.PrintTestOutput(
perf_options.print_step, perf_options.output_chartjson_data)
- runner_factory, tests, devices = perf_setup.Setup(
- perf_options, active_devices)
+ runner_factory, tests, devices = perf_setup.Setup(perf_options)
# shard=False means that each device will get the full list of tests
# and then each one will decide their own affinity.
@@ -885,7 +882,7 @@
sys.path = sys.path[1:]
-def _GetAttachedDevices(blacklist_file, test_device):
+def _GetAttachedDevices(test_device=None):
"""Get all attached devices.
Args:
@@ -894,14 +891,7 @@
Returns:
A list of attached devices.
"""
- if not blacklist_file:
- # TODO(jbudorick): Remove this once bots pass the blacklist file.
- blacklist_file = device_blacklist.BLACKLIST_JSON
- logging.warning('Using default device blacklist %s',
- device_blacklist.BLACKLIST_JSON)
-
- blacklist = device_blacklist.Blacklist(blacklist_file)
- attached_devices = device_utils.DeviceUtils.HealthyDevices(blacklist)
+ attached_devices = device_utils.DeviceUtils.HealthyDevices()
if test_device:
test_device = [d for d in attached_devices if d == test_device]
if not test_device:
@@ -940,7 +930,7 @@
if command in constants.LOCAL_MACHINE_TESTS:
devices = []
else:
- devices = _GetAttachedDevices(args.blacklist_file, args.test_device)
+ devices = _GetAttachedDevices(args.test_device)
forwarder.Forwarder.RemoveHostLog()
if not ports.ResetTestServerPortAllocation():
@@ -961,7 +951,7 @@
elif command == 'monkey':
return _RunMonkeyTests(args, devices)
elif command == 'perf':
- return _RunPerfTests(args, devices)
+ return _RunPerfTests(args)
elif command == 'python':
return _RunPythonTests(args)
else:
« no previous file with comments | « build/android/screenshot.py ('k') | build/android/tombstones.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698