Lint bisect-related modules; obey pylint.

scripts/slave/recipe_modules/auto_bisect/bisector_test.py

 # Copyright 2015 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
 import sys
 import unittest
 
 # TODO(robertocn): Use abspath for these, to prevent relative path errors.
 _RECIPE_MODULES_DIR = os.path.join(os.path.dirname(__file__), os.path.pardir)
 # For the importing of mock.
 _ROOT_DIR = os.path.join(os.path.dirname(__file__), os.path.pardir,
                          os.path.pardir, os.path.pardir, os.path.pardir)
 
 sys.path.append(_RECIPE_MODULES_DIR)
 sys.path.append(os.path.join(_ROOT_DIR, 'third_party', 'mock-1.0.1'))
 
 import mock
 
 from auto_bisect.bisector import Bisector
 
 
 class MockRevisionClass(object):  # pragma: no cover
+
   def __init__(self, rev_string, bisector):
     self.commit_pos = int(rev_string)
     self.revision_string = rev_string
     self.bisector = bisector
     self.previous_revision = None
     self.next_revision = None
     self.values = []
 
   def get_next_url(self):
     if self.in_progress:
       return 'mockurl'
     return None
 
 
 class BisectorTest(unittest.TestCase):  # pragma: no cover
+
   def setUp(self):
     self.bisect_config = {
         'test_type': 'perf',
-        'command': 'tools/perf/run_benchmark -v '
-                    '--browser=release page_cycler.intl_ar_fa_he',
+        'command': ('tools/perf/run_benchmark -v '
+                    '--browser=release page_cycler.intl_ar_fa_he'),
         'good_revision': '306475',
         'bad_revision': '306478',
         'metric': 'warm_times/page_load_time',
         'repeat_count': '2',
         'max_time_minutes': '5',
         'truncate_percent': '25',
         'bug_id': '425582',
         'gs_bucket': 'chrome-perf',
         'builder_host': 'master4.golo.chromium.org',
         'builder_port': '8341',
         'dummy_builds': True,
     }
     self.dummy_api = mock.Mock()
 
-
-
   def test_create_bisector(self):
     bisector = Bisector(self.dummy_api, self.bisect_config, MockRevisionClass)
     # Check the proper revision range is initialized
     self.assertEqual(4, len(bisector.revisions))
     a, b, c, d = bisector.revisions
     # Check that revisions are properly chained
     self.assertEqual(a, b.previous_revision)
     self.assertEqual(b, c.previous_revision)
     self.assertEqual(c, d.previous_revision)
     self.assertEqual(d, c.next_revision)
@@ skipping 181 matching lines @@
         self.assertTrue(self.called_abort)
 
         # Verifying the side effects of updating the candidate range
         self.assertEqual(r[2], bisector.lkgr)
         self.assertEqual(r[3], bisector.fkbr)
 
 
 # TODO(robertocn): Add test for bisector.check_bisect_finished.
 if __name__ == '__main__':
   unittest.main()  # pragma: no cover