Restore test helper function

spyder_terminal/server/common.py

@@ -11,7 +11,7 @@ def create_app(shell, close_future=None):
     """Create and return a tornado Web Application instance."""
     settings = {"static_path": os.path.join(
         os.path.dirname(__file__), "static")}
-    application = tornado.web.Application(routes.ROUTES,
+    application = tornado.web.Application(routes.gen_routes(close_future),
                                           debug=True,
                                           serve_traceback=True,
                                           autoreload=True, **settings)

spyder_terminal/server/logic/term_manager.py

@@ -59,11 +59,14 @@ def client_disconnected(self, pid, socket):
         self.log.info("Websocket closed, sending SIGHUP to terminal.")
         term = self.consoles[pid]
         term.clients.remove(socket)
-        if WINDOWS:
-            term.kill()
-            self.pty_read(term.ptyproc.fd)
-            return
-        term.killpg(signal.SIGHUP)
+        try:
+            if WINDOWS:
+                term.kill()
+                self.pty_read(term.ptyproc.fd)
+                return
+            term.killpg(signal.SIGHUP)
+        except OSError:
+            pass
         del self.consoles[pid]
 
     @tornado.gen.coroutine

spyder_terminal/server/routes.py

@@ -36,3 +36,14 @@
 ]
 
 ROUTES = REST + WS + WEB
+
+
+def gen_routes(close_future):
+    """Return a list of HTML redirection routes."""
+    if close_future is not None:
+        ws = []
+        for route in WS:
+            ws.append((route[0], route[1],
+                       dict(close_future=close_future)))
+        return REST + ws + WEB
+    return ROUTES

spyder_terminal/server/websockets/term_ws.py

@@ -40,4 +40,5 @@ def on_pty_read(self, text):
         self.write_message(text)
 
     def on_pty_died(self):
+        """Close websocket if terminal was closed externally."""
         self.close()