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

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

Issue 355203002: Revert of [telemetry] Refactor run_tests to remove old cruft. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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 | « tools/telemetry/telemetry/unittest/run_tests.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..20bc88ddb4a55ff95c795f525c6952b75c88dd58 100644
--- a/tools/telemetry/telemetry/unittest/run_tests_unittest.py
+++ b/tools/telemetry/telemetry/unittest/run_tests_unittest.py
@@ -32,9 +32,8 @@
class RunTestsUnitTest(unittest.TestCase):
def setUp(self):
- self.suite = unittest.TestSuite()
- self.suite.addTests(run_tests.Discover(
- util.GetTelemetryDir(), util.GetTelemetryDir(), 'disabled_cases.py'))
+ self.suite = run_tests.Discover(
+ util.GetTelemetryDir(), util.GetTelemetryDir(), 'disabled_cases.py')
def _GetEnabledTests(self, browser_type, os_name, os_version_name,
supports_tab_control):
@@ -46,8 +45,8 @@
enabled_tests = set()
for i in run_tests.FilterSuite(self.suite, MockPredicate)._tests:
- for j in i:
- for k in j:
+ for j in i._tests:
+ for k in j._tests:
enabled_tests.add(k._testMethodName)
return enabled_tests
« no previous file with comments | « tools/telemetry/telemetry/unittest/run_tests.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698