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

Unified Diff: tools/telemetry/telemetry/unittest/run_tests_unittest.py

Issue 659293003: Switch telemetry over to use typ to run the unit tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@rev_typ_v086
Patch Set: fix merge error in browser_test_case Created 6 years, 1 month 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: tools/telemetry/telemetry/unittest/run_tests_unittest.py
diff --git a/tools/telemetry/telemetry/unittest/run_tests_unittest.py b/tools/telemetry/telemetry/unittest/run_tests_unittest.py
index ee02d93d25b305db2ff6b06d0214330e0fc9b544..6db93316f75d98ae14df8135608b99c367ba1996 100644
--- a/tools/telemetry/telemetry/unittest/run_tests_unittest.py
+++ b/tools/telemetry/telemetry/unittest/run_tests_unittest.py
@@ -4,11 +4,17 @@
import unittest
-from telemetry import decorators
from telemetry.core import util
from telemetry.unittest import run_tests
+class MockArgs(object):
+ def __init__(self):
+ self.positional_args = []
+ self.exact_test_filter = True
+ self.run_disabled_tests = False
+
+
class MockPossibleBrowser(object):
def __init__(self, browser_type, os_name, os_version_name,
supports_tab_control):
@@ -31,25 +37,19 @@ class MockPlatform(object):
class RunTestsUnitTest(unittest.TestCase):
- def setUp(self):
- self.suite = unittest.TestSuite()
- self.suite.addTests(run_tests.Discover(
- util.GetTelemetryDir(), util.GetTelemetryDir(), 'disabled_cases.py'))
-
def _GetEnabledTests(self, browser_type, os_name, os_version_name,
supports_tab_control):
- # pylint: disable=W0212
- def MockPredicate(test):
- method = getattr(test, test._testMethodName)
- return decorators.IsEnabled(method, MockPossibleBrowser(
- browser_type, os_name, os_version_name, supports_tab_control))
-
- enabled_tests = set()
- for i in run_tests.FilterSuite(self.suite, MockPredicate)._tests:
- for j in i:
- for k in j:
- enabled_tests.add(k._testMethodName)
- return enabled_tests
+
+ runner = run_tests.typ.Runner()
+ host = runner.host
+ runner.top_level_dir = util.GetTelemetryDir()
+ runner.args.tests = [host.join(util.GetTelemetryDir(),
+ 'telemetry', 'unittest', 'disabled_cases.py')]
+ possible_browser = MockPossibleBrowser(
+ browser_type, os_name, os_version_name, supports_tab_control)
+ runner.classifier = run_tests.GetClassifier(MockArgs(), possible_browser)
+ _, test_set = runner.find_tests(runner.args)
+ return set(test.name.split('.')[-1] for test in test_set.parallel_tests)
def testSystemMacMavericks(self):
self.assertEquals(

Powered by Google App Engine
This is Rietveld 408576698