don't start new notebook server for each test *method*

it takes much too long.

This change starts a new notebook server for each test class.

Even that is probably too much, but the module organization makes it troublesome
to start a single notebook server for all services tests.

IPython/html/tests/launchnotebook.py

@@ -1,62 +1,69 b''
 """Base class for notebook tests."""
 
 import sys
 import time
 import requests
 from subprocess import Popen, PIPE
 from unittest import TestCase
 
 from IPython.utils.tempdir import TemporaryDirectory
 
 
 class NotebookTestBase(TestCase):
     """A base class for tests that need a running notebook.
     
     This creates an empty profile in a temp ipython_dir
     and then starts the notebook server with a separate temp notebook_dir.
     """
 
     port = 12341
 
-    def wait_till_alive(self):
-        url = 'http://localhost:%i/' % self.port
+    @classmethod
+    def wait_until_alive(cls):
+        """Wait for the server to be alive"""
+        url = 'http://localhost:%i/api/notebooks' % cls.port
         while True:
-            time.sleep(.1)
             try:
-                r = requests.get(url + 'api/notebooks')
-                break
+                requests.get(url)
             except requests.exceptions.ConnectionError:
-                pass
+                time.sleep(.1)
+            else:
+                break
         
-    def wait_till_dead(self):
-        url = 'http://localhost:%i/' % self.port
+    @classmethod
+    def wait_until_dead(cls):
+        """Wait for the server to stop getting requests after shutdown"""
+        url = 'http://localhost:%i/api/notebooks' % cls.port
         while True:
-            time.sleep(.1)
             try:
-                r = requests.get(url + 'api/notebooks')
-                continue
+                requests.get(url)
             except requests.exceptions.ConnectionError:
                 break
+            else:
+                time.sleep(.1)
 
-    def setUp(self):
-        self.ipython_dir = TemporaryDirectory()
-        self.notebook_dir = TemporaryDirectory()
+    @classmethod
+    def setup_class(cls):
+        cls.ipython_dir = TemporaryDirectory()
+        cls.notebook_dir = TemporaryDirectory()
         notebook_args = [
             sys.executable, '-c',
             'from IPython.html.notebookapp import launch_new_instance; launch_new_instance()',
-            '--port=%d' % self.port,
+            '--port=%d' % cls.port,
             '--no-browser',
-            '--ipython-dir=%s' % self.ipython_dir.name,
-            '--notebook-dir=%s' % self.notebook_dir.name
+            '--ipython-dir=%s' % cls.ipython_dir.name,
+            '--notebook-dir=%s' % cls.notebook_dir.name
         ]
-        self.notebook = Popen(notebook_args, stdout=PIPE, stderr=PIPE)
-        self.wait_till_alive()
+        cls.notebook = Popen(notebook_args, stdout=PIPE, stderr=PIPE)
+        cls.wait_until_alive()
 
-    def tearDown(self):
-        self.notebook.terminate()
-        self.ipython_dir.cleanup()
-        self.notebook_dir.cleanup()
-        self.wait_till_dead()
+    @classmethod
+    def teardown_class(cls):
+        cls.notebook.terminate()
+        cls.ipython_dir.cleanup()
+        cls.notebook_dir.cleanup()
+        cls.wait_until_dead()
         
-    def base_url(self):
-        return 'http://localhost:%i/' % self.port
+    @classmethod
+    def base_url(cls):
+        return 'http://localhost:%i/' % cls.port