run-tests: move bisect logic to a separate method

This removes 8 space indentation and makes upcoming changes easier.

Differential Revision: https://phab.mercurial-scm.org/D947

tests/run-tests.py

@@ -2085,6 +2085,20 class TextTestRunner(unittest.TextTestRu
             savetimes(self._runner._outputdir, result)
 
             if failed and self._runner.options.known_good_rev:
+                self._bisecttests(t for t, m in result.failures)
+            self.stream.writeln(
+                '# Ran %d tests, %d skipped, %d failed.'
+                % (result.testsRun, skipped + ignored, failed))
+            if failed:
+                self.stream.writeln('python hash seed: %s' %
+                    os.environ['PYTHONHASHSEED'])
+            if self._runner.options.time:
+                self.printtimes(result.times)
+            self.stream.flush()
+
+        return result
+
+    def _bisecttests(self, tests):
                 bisectcmd = ['hg', 'bisect']
                 bisectrepo = self._runner.options.bisect_repo
                 if bisectrepo:
@@ -2094,7 +2108,7 class TextTestRunner(unittest.TextTestRu
                                          stdout=subprocess.PIPE)
                     p.stdout.read()
                     p.wait()
-                for test, msg in result.failures:
+        for test in tests:
                     nooutput(bisectcmd + ['--reset']),
                     nooutput(bisectcmd + ['--bad', '.'])
                     nooutput(bisectcmd + ['--good',
@@ -2126,17 +2140,6 class TextTestRunner(unittest.TextTestRu
                     self.stream.writeln(
                         '%s %s by %s (%s)' % (
                             test, verb, dat['node'], dat['summary']))
-            self.stream.writeln(
-                '# Ran %d tests, %d skipped, %d failed.'
-                % (result.testsRun, skipped + ignored, failed))
-            if failed:
-                self.stream.writeln('python hash seed: %s' %
-                    os.environ['PYTHONHASHSEED'])
-            if self._runner.options.time:
-                self.printtimes(result.times)
-            self.stream.flush()
-
-        return result
 
     def printtimes(self, times):
         # iolock held by run