Commit 0b8a0468 authored by jbudorick's avatar jbudorick Committed by Commit bot

[Android] Stop emitting a nonzero value from the test runner if a device goes offline.

Recipes interprets all nonzero exit codes as errors.

BUG=424587

Review URL: https://codereview.chromium.org/651833004

Cr-Commit-Position: refs/heads/master@{#300614}
parent 4441cc1d
...@@ -272,7 +272,6 @@ def _RunAllTests(runners, test_collection_factory, num_retries, timeout=None, ...@@ -272,7 +272,6 @@ def _RunAllTests(runners, test_collection_factory, num_retries, timeout=None,
# for the above are switched or wrapped. # for the above are switched or wrapped.
android_commands.errors.DeviceUnresponsiveError) as e: android_commands.errors.DeviceUnresponsiveError) as e:
logging.error(e) logging.error(e)
exit_code = constants.WARNING_EXIT_CODE
if not all((len(tc) == 0 for tc in test_collections)): if not all((len(tc) == 0 for tc in test_collections)):
logging.error('Only ran %d tests (all devices are likely offline).' % logging.error('Only ran %d tests (all devices are likely offline).' %
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment