Separate out machinery for running JS tests

IPython/testing/iptest.py

@@ -164,7 +164,7 b" have['zmq'] = test_for('zmq.pyzmq_version_info', min_zmq, callback=lambda x: x()"
 
 test_group_names = ['parallel', 'kernel', 'kernel.inprocess', 'config', 'core',
                     'extensions', 'lib', 'terminal', 'testing', 'utils',
-                    'nbformat', 'qt', 'html', 'js', 'nbconvert'
+                    'nbformat', 'qt', 'html', 'nbconvert'
                    ]
 
 class TestSection(object):
@@ -288,9 +288,6 b" if not have['jinja2']:"
 if not have['azure']:
     sec.exclude('services.notebooks.azurenbmanager')
 
-sec = test_sections['js']
-sec.requires('zmq', 'tornado', 'jinja2', 'casperjs')
-
 # config:
 # Config files aren't really importable stand-alone
 test_sections['config'].exclude('profile')

IPython/testing/iptestcontroller.py

@@ -28,7 +28,7 b' import sys'
 import subprocess
 import time
 
-from .iptest import have, test_group_names, test_sections
+from .iptest import have, test_group_names as py_test_group_names, test_sections
 from IPython.utils.py3compat import bytes_to_str
 from IPython.utils.sysinfo import sys_info
 from IPython.utils.tempdir import TemporaryDirectory
@@ -57,14 +57,6 b' class TestController(object):'
         self.env = {}
         self.dirs = []
     
-
-    @property
-    def will_run(self):
-        try:
-            return test_sections[self.section].will_run
-        except KeyError:
-            return True
-
     def launch(self):
         # print('*** ENV:', self.env)  # dbg
         # print('*** CMD:', self.cmd)  # dbg
@@ -132,6 +124,13 b' class PyTestController(TestController):'
         # This means we won't get odd effects from our own matplotlib config
         self.env['MPLCONFIGDIR'] = workingdir.name
 
+    @property
+    def will_run(self):
+        try:
+            return test_sections[self.section].will_run
+        except KeyError:
+            return True
+
     def add_xunit(self):
         xunit_file = os.path.abspath(self.section + '.xunit.xml')
         self.cmd.extend(['--with-xunit', '--xunit-file', xunit_file])
@@ -170,6 +169,7 b' class JSController(TestController):'
         self.dirs.append(self.ipydir)
         self.env['IPYTHONDIR'] = self.ipydir.name
 
+    def launch(self):
         # start the ipython notebook, so we get the port number
         self._init_server()
 
@@ -180,6 +180,11 b' class JSController(TestController):'
         port = '--port=' + str(self.server_port)
         self.cmd = ['casperjs', 'test', port, includes, test_cases]
 
+        super(JSController, self).launch()
+
+    @property
+    def will_run(self):
+        return all(have[a] for a in ['zmq', 'tornado', 'jinja2', 'casperjs'])
 
     def _init_server(self):
         "Start the notebook server in a separate process"
@@ -193,6 +198,7 b' class JSController(TestController):'
         self.server.join()
         TestController.cleanup(self)
 
+js_test_group_names = {'js'}
 
 def run_webapp(q, nbdir, loglevel=0):
     """start the IPython Notebook, and pass port back to the queue"""
@@ -212,13 +218,17 b' def prepare_controllers(options):'
     not to run."""
     testgroups = options.testgroups
 
-    if not testgroups:
-        testgroups = test_group_names
+    if testgroups:
+        py_testgroups = [g for g in testgroups if g in py_test_group_names]
+        js_testgroups = [g for g in testgroups if g in js_test_group_names]
+    else:
+        py_testgroups = py_test_group_names
+        js_testgroups = js_test_group_names
         if not options.all:
             test_sections['parallel'].enabled = False
 
-    c_js = [JSController(name) for name in testgroups if 'js' in name]
-    c_py = [PyTestController(name) for name in testgroups if 'js' not in name]
+    c_js = [JSController(name) for name in js_testgroups]
+    c_py = [PyTestController(name) for name in py_testgroups]
 
     configure_py_controllers(c_py, xunit=options.xunit,
             coverage=options.coverage)