Merge pull request #11784 from Carreau/clean-iptest

Cleanup iptest controller from old logic.

IPython/testing/iptestcontroller.py

@@ -28,7 +28,7 b' from IPython.utils.py3compat import decode'
 from IPython.utils.sysinfo import get_sys_info
 from IPython.utils.tempdir import TemporaryDirectory
 
-class TestController(object):
+class TestController:
     """Run tests in a subprocess
     """
     #: str, IPython test suite to be executed.
@@ -77,18 +77,6 b' class TestController(object):'
         self.stdout = self.stdout_capturer.get_buffer()
         return self.process.returncode
 
-    def print_extra_info(self):
-        """Print extra information about this test run.
-        
-        If we're running in parallel and showing the concise view, this is only
-        called if the test group fails. Otherwise, it's called before the test
-        group is started.
-        
-        The base implementation does nothing, but it can be overridden by
-        subclasses.
-        """
-        return
-
     def cleanup_process(self):
         """Cleanup on exit by killing any leftover processes."""
         subp = self.process
@@ -243,8 +231,6 b' def do_run(controller, buffer_output=True):'
     try:
         try:
             controller.setup()
-            if not buffer_output:
-                controller.print_extra_info()
             controller.launch(buffer_output=buffer_output)
         except Exception:
             import traceback
@@ -367,7 +353,6 b' def run_iptestall(options):'
                 res_string = 'OK' if res == 0 else 'FAILED'
                 print(justify('Test group: ' + controller.section, res_string))
                 if res:
-                    controller.print_extra_info()
                     print(decode(controller.stdout))
                     failed.append(controller)
                     if res == -signal.SIGINT: