Merge pull request #2 from davidbrochart/zombieProcFix

Zombie proc fix

Author: MSeal

nbclient/exceptions.py

@@ -49,9 +49,11 @@ class CellExecutionError(CellControlSignal):
     failures gracefully.
     """
 
-    def __init__(self, traceback):
+    def __init__(self, traceback, ename, evalue):
         super(CellExecutionError, self).__init__(traceback)
         self.traceback = traceback
+        self.ename = ename
+        self.evalue = evalue
 
     def __str__(self):
         s = self.__unicode__()
@@ -74,7 +76,9 @@ def from_cell_and_msg(cls, cell, msg):
                 traceback=tb,
                 ename=msg.get('ename', '<Error>'),
                 evalue=msg.get('evalue', ''),
-            )
+            ),
+            ename=msg.get('ename', '<Error>'),
+            evalue=msg.get('evalue', '')
         )
 
 

nbclient/tests/test_client.py

@@ -505,12 +505,13 @@ def test_kernel_death(self):
             output_nb = executor.execute()
         km = executor.start_kernel_manager()
 
-        with patch.object(km, "is_alive") as alive_mock:
-            alive_mock.return_value = False
-            # Will be a RuntimeError or subclass DeadKernelError depending
-            # on if jupyter_client or nbconvert catches the dead client first
-            with pytest.raises(RuntimeError):
-                input_nb, output_nb = executor.execute()
+        async def is_alive():
+            return False
+        km.is_alive = is_alive
+        # Will be a RuntimeError or subclass DeadKernelError depending
+        # on if jupyter_client or nbconvert catches the dead client first
+        with pytest.raises(RuntimeError):
+            input_nb, output_nb = executor.execute()
 
     def test_allow_errors(self):
         """