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

Unified Diff: build/android/pylib/uiautomator/test_runner.py

Issue 18770008: [Android] Redesigns the sharder to allow replicated vs distributed tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Adds tagging of tests (for replication) Created 7 years, 5 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
Index: build/android/pylib/uiautomator/test_runner.py
diff --git a/build/android/pylib/uiautomator/test_runner.py b/build/android/pylib/uiautomator/test_runner.py
index b4788d018272e6461a7e494f4a6b643b6163fc2c..fd10ec5f58b6cbcaf047d8636cd5cb1a528243c2 100644
--- a/build/android/pylib/uiautomator/test_runner.py
+++ b/build/android/pylib/uiautomator/test_runner.py
@@ -10,19 +10,22 @@ from pylib.instrumentation import test_runner as instr_test_runner
class TestRunner(instr_test_runner.TestRunner):
"""Responsible for running a series of tests connected to a single device."""
- def __init__(self, options, device, shard_index, test_pkg, ports_to_forward):
+ def __init__(self, package_name, build_type, test_data, save_perf_json,
+ screenshot_failures, tool, wait_for_debugger,
+ disable_assertions, push_deps, device, shard_index,
+ test_pkg, ports_to_forward):
"""Create a new TestRunner.
Args:
- options: An options object similar to the one in parent class plus:
- - package_name: Application package name under test.
+ package_name: Application package name under test.
+ See the super class for all other args.
"""
- options.ensure_value('install_apk', True)
- options.ensure_value('wait_for_debugger', False)
super(TestRunner, self).__init__(
- options, device, shard_index, test_pkg, ports_to_forward)
+ build_type, test_data, False, save_perf_json, screenshot_failures,
+ tool, wait_for_debugger, disable_assertions, push_deps, device,
+ shard_index, test_pkg, ports_to_forward)
- self.package_name = options.package_name
+ self.package_name = package_name
#override
def InstallTestPackage(self):
@@ -41,4 +44,3 @@ class TestRunner(instr_test_runner.TestRunner):
self.flags.AddFlags(['--disable-fre'])
return self.adb.RunUIAutomatorTest(
test, self.test_pkg.GetPackageName(), timeout)
-

Powered by Google App Engine
This is Rietveld 408576698