Move browser_test_runner.LoadAllTestsInModule to serially_executed_browser_test_case

telemetry/examples/browser_tests/failed_tests.py

 # Copyright 2016 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 sys
 
 from telemetry.testing import serially_executed_browser_test_case
 
 
 class SetUpClassFailedTest(
     serially_executed_browser_test_case.SeriallyExecutedBrowserTestCase):
 
   @classmethod
   def setUpClass(cls):
     raise Exception
@@ skipping 16 matching lines @@
     raise Exception
 
   @classmethod
   def GenerateTestCases_DummyTest(cls, options):
     del options  # Unused.
     for i in xrange(0, 100):
       yield 'dummy_test_%i' % i, ()
 
   def DummyTest(self):
     pass
+
+
+def load_tests(loader, tests, pattern):
+  del loader, tests, pattern  # Unused.

[Dirk Pranke, 2016/12/11 03:23:01]

Nit: maybe use a pylint annotation here instead?

[nednguyen, 2016/12/11 13:01:44]

This is our preferred pattern: go/del-for-unused (Googler only, sorry)

+  return serially_executed_browser_test_case.LoadAllTestsInModule(
+      sys.modules[__name__])