From 1a85d7d24d5cfe4055e04431c8d547f6361d7430 Mon Sep 17 00:00:00 2001 From: Sylvain Corlay Date: Sat, 7 Feb 2015 18:58:39 -0500 Subject: [PATCH] support case where no server is started --- IPython/testing/iptestcontroller.py | 47 +++++++++++++++-------------- 1 file changed, 24 insertions(+), 23 deletions(-) diff --git a/IPython/testing/iptestcontroller.py b/IPython/testing/iptestcontroller.py index cd313e259..c502d7f5a 100644 --- a/IPython/testing/iptestcontroller.py +++ b/IPython/testing/iptestcontroller.py @@ -396,33 +396,34 @@ class JSController(TestController): self.server_port = info['port'] def cleanup(self): - try: - self.server.terminate() - except OSError: - # already dead - pass - # wait 10s for the server to shutdown - try: - popen_wait(self.server, NOTEBOOK_SHUTDOWN_TIMEOUT) - except TimeoutExpired: - # server didn't terminate, kill it + if hasattr(self, 'server'): try: - print("Failed to terminate notebook server, killing it.", - file=sys.stderr - ) - self.server.kill() + self.server.terminate() except OSError: # already dead pass - # wait another 10s - try: - popen_wait(self.server, NOTEBOOK_SHUTDOWN_TIMEOUT) - except TimeoutExpired: - print("Notebook server still running (%s)" % self.server_info_file, - file=sys.stderr - ) - - self.stream_capturer.halt() + # wait 10s for the server to shutdown + try: + popen_wait(self.server, NOTEBOOK_SHUTDOWN_TIMEOUT) + except TimeoutExpired: + # server didn't terminate, kill it + try: + print("Failed to terminate notebook server, killing it.", + file=sys.stderr + ) + self.server.kill() + except OSError: + # already dead + pass + # wait another 10s + try: + popen_wait(self.server, NOTEBOOK_SHUTDOWN_TIMEOUT) + except TimeoutExpired: + print("Notebook server still running (%s)" % self.server_info_file, + file=sys.stderr + ) + + self.stream_capturer.halt() TestController.cleanup(self)