More accurate @running for Process

portal/__init__.py

@@ -1,4 +1,4 @@
-__version__ = '3.1.12'
+__version__ = '3.2.0'
 
 import multiprocessing as mp
 try:

portal/process.py

@@ -52,7 +52,14 @@ def pid(self):
   def running(self):
     if not self.started:
       return False
-    return self.process.is_alive()
+    if not self.process.is_alive():
+      return False
+    try:
+      if psutil.Process(self.pid).status() == psutil.STATUS_ZOMBIE:
+        return False
+    except psutil.NoSuchProcess:
+      return False
+    return True
 
   @property
   def exitcode(self):

tests/test_process.py

@@ -43,7 +43,7 @@ def fn():
     worker = portal.Process(fn, start=True)
     worker.kill()
     assert not worker.running
-    assert worker.exitcode == -15
+    assert worker.exitcode < 0
 
   @pytest.mark.parametrize('repeat', range(5))
   def test_kill_with_subproc(self, repeat):
@@ -62,7 +62,7 @@ def inner(ready):
     ready.acquire()
     worker.kill()
     assert not worker.running
-    assert worker.exitcode == -15
+    assert worker.exitcode < 0
 
   @pytest.mark.parametrize('repeat', range(5))
   def test_kill_with_subthread(self, repeat):
@@ -79,7 +79,7 @@ def inner(ready):
     ready.wait()
     worker.kill()
     assert not worker.running
-    assert worker.exitcode == -15
+    assert worker.exitcode < 0
 
   def test_initfn(self):
     def init():

tests/test_thread.py

@@ -91,4 +91,4 @@ def inner(ready):
     assert not worker.running
     assert not proc[0].running
     assert worker.exitcode == 2
-    assert proc[0].exitcode == -15
+    assert proc[0].exitcode < 0