wcfs/pintimeout_kill: Fix killed testing if successful

In case 'test_wcfs_pintimeout_kill' is successful, WCFS kills the
client process. In order to prevent WCFS from killing the full
testing process, we therefore need to encapsulate the test code into
a subprocess, so that only this subprocess is killed if the test is
successful. Because of moving our test to a subprocess, we also need
to explicitly catch exceptions to mark the test as failed in case of
any error.

wcfs/wcfs_test.py

@@ -40,10 +40,13 @@ from persistent import Persistent
 from persistent.timestamp import TimeStamp
 from ZODB.utils import z64, u64, p64
 
+import ctypes
+import six
 import sys, os, os.path, subprocess
 from thread import get_ident as gettid
 from time import gmtime
 from errno import EINVAL, ENOTCONN
+from multiprocessing import Process, Value
 from resource import setrlimit, getrlimit, RLIMIT_MEMLOCK
 from golang import go, chan, select, func, defer, error, b
 from golang import context, errors, sync, time
@@ -1463,15 +1466,23 @@ def test_wcfs_pintimeout_kill():
     f = t.open(zf)
     f.assertData(['','','c2'])
 
-    # XXX move into subprocess not to kill whole testing
-    ctx, _ = context.with_timeout(context.background(), 2*tkill)
+    # move into subprocess to avoid killing testing process, in
+    # case test is successful
+    def test(err):
+        try:
+            _test(err)
+        except Exception:
+            if not err.value:
+                err.value = "unexpected error in test code"
 
+    def _test(err):
+        ctx, _ = context.with_timeout(context.background(), 2*tkill)
         wl = t.openwatch()
         wg = sync.WorkGroup(ctx)
         def _(ctx):
             # send watch. The pin handler won't be replying -> we should never get reply here.
             wl.sendReq(ctx, b"watch %s @%s" % (h(zf._p_oid), h(at1)))
-        fail("watch request completed (should not as pin handler is stuck)")
+            err.value= "watch request completed (should not as pin handler is stuck)"
         wg.go(_)
         def _(ctx):
             req = wl.recvReq(ctx)
@@ -1485,10 +1496,17 @@ def test_wcfs_pintimeout_kill():
             )
             if _ == 0:
                 raise ctx.err()
-        fail("wcfs did not killed stuck client")
+            err.value= "wcfs did not kill stuck client"
         wg.go(_)
         wg.wait()
 
+    err = Value((ctypes.c_char_p if six.PY2 else ctypes.c_wchar_p), "")
+    p = Process(target=test, args=(err,))
+    p.start()
+    p.join()
+    if err.value:
+        fail(err.value)
+
 
 # watch with @at > head - must wait for head to become >= at.
 # TODO(?) too far ahead - reject?