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

Unified Diff: trunk/src/build/android/run_uiautomator_tests.py

Issue 18732002: Revert 210035 "Creates a new test running script test_runner.py" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: 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
« no previous file with comments | « trunk/src/build/android/run_tests.py ('k') | trunk/src/build/android/test_runner.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/build/android/run_uiautomator_tests.py
===================================================================
--- trunk/src/build/android/run_uiautomator_tests.py (revision 210250)
+++ trunk/src/build/android/run_uiautomator_tests.py (working copy)
@@ -6,19 +6,74 @@
"""Runs both the Python and Java UIAutomator tests."""
-import logging
+import optparse
import os
import sys
+import time
-from pylib import cmd_helper
+from pylib import buildbot_report
+from pylib import constants
+from pylib import ports
+from pylib.base import base_test_result
+from pylib.host_driven import run_python_tests
+from pylib.uiautomator import dispatch
+from pylib.utils import report_results
+from pylib.utils import run_tests_helper
+from pylib.utils import test_options_parser
+def DispatchUIAutomatorTests(options):
+ """Dispatches the UIAutomator tests, sharding if possible.
+
+ Uses the logging module to print the combined final results and
+ summary of the Java and Python tests. If the java_only option is set, only
+ the Java tests run. If the python_only option is set, only the python tests
+ run. If neither are set, run both Java and Python tests.
+
+ Args:
+ options: command-line options for running the Java and Python tests.
+
+ Returns:
+ An integer representing the number of broken tests.
+ """
+ if not options.keep_test_server_ports:
+ # Reset the test port allocation. It's important to do it before starting
+ # to dispatch any tests.
+ if not ports.ResetTestServerPortAllocation():
+ raise Exception('Failed to reset test server port.')
+
+ all_results = base_test_result.TestRunResults()
+
+ if options.run_java_tests:
+ all_results.AddTestRunResults(dispatch.Dispatch(options))
+ if options.run_python_tests:
+ all_results.AddTestRunResults(run_python_tests.DispatchPythonTests(options))
+
+ report_results.LogFull(
+ results=all_results,
+ test_type='UIAutomator',
+ test_package=os.path.basename(options.test_jar),
+ annotation=options.annotations,
+ build_type=options.build_type,
+ flakiness_server=options.flakiness_dashboard_server)
+
+ return len(all_results.GetNotPass())
+
+
+def main(argv):
+ option_parser = optparse.OptionParser()
+ test_options_parser.AddUIAutomatorOptions(option_parser)
+ options, args = option_parser.parse_args(argv)
+ test_options_parser.ValidateUIAutomatorOptions(option_parser, options, args)
+
+ run_tests_helper.SetLogLevel(options.verbose_count)
+ ret = 1
+ try:
+ ret = DispatchUIAutomatorTests(options)
+ finally:
+ buildbot_report.PrintStepResultIfNeeded(options, ret)
+ return ret
+
+
if __name__ == '__main__':
- args = ['python',
- os.path.join(os.path.dirname(__file__), 'test_runner.py'),
- 'uiautomator'] + sys.argv[1:]
- logging.warning('*' * 80)
- logging.warning('This script is deprecated and will be removed soon.')
- logging.warning('Use the following instead: %s', ' '.join(args))
- logging.warning('*' * 80)
- sys.exit(cmd_helper.RunCmd(args))
+ sys.exit(main(sys.argv))
« no previous file with comments | « trunk/src/build/android/run_tests.py ('k') | trunk/src/build/android/test_runner.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698