[Android] JUnit test runner changes.

build/android/pylib/junit/test_runner.py

Index: build/android/pylib/junit/test_runner.py
diff --git a/build/android/pylib/junit/test_runner.py b/build/android/pylib/junit/test_runner.py
new file mode 100644
index 0000000000000000000000000000000000000000..5098ad7d3d90ec7158266d9e5a6c4c5c11ddf8ea
--- /dev/null
+++ b/build/android/pylib/junit/test_runner.py
@@ -0,0 +1,39 @@
+# Copyright 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import os
+
+from pylib import cmd_helper
+from pylib import constants
+
+class JavaTestRunner(object):
+  """ Runs java tests on the host.
+  """

[klundberg, 2014/09/11 20:25:13]

nit: you can have """ on line 11

[jbudorick, 2014/09/11 20:33:54]

This is a very Frank-esque comment. :)

Cleaned up comments throughout the review.

+  def __init__(self, options):
+    self._package_filter = options.package_filter
+    self._runner_filter = options.runner_filter
+    self._sdk_version = options.sdk_version
+    self._test_filter = options.test_filter
+    self._test_suite = options.test_suite
+
+  def SetUp(self):
+    pass
+
+  def RunTest(self, _test):
+    command = ['java',
+               '-jar', os.path.join(constants.GetOutDirectory(), 'lib.java',
+                                    '%s.jar' % self._test_suite)]
+    if self._test_filter:
+      command.extend(['-gtest-filter', self._test_filter])
+    if self._package_filter:
+      command.extend(['-package-filter', self._package_filter])
+    if self._runner_filter:
+      command.extend(['-runner-filter', self._runner_filter])
+    if self._sdk_version:
+      command.extend(['-sdk-version', self._sdk_version])
+    return cmd_helper.RunCmd(command)
+
+  def TearDown(self):
+    pass
+