Merge pull request #41 from zopefoundation/FakeInputContinueGenerator…

…-close

Fixed: Using the -j option to run tests in multiple processes

CHANGES.rst

@@ -4,7 +4,10 @@ zope.testrunner Changelog
 4.5.1 (unreleased)
 ==================
 
-- Nothing changed yet.
+- Fixed: Using the ``-j`` option to run tests in multiple processes
+  caused tests that used the ``multiprocessing`` package to hang
+  (because the testrunner replaced ``sys.stdin`` with an unclosable
+  object).
 
 
 4.5.0 (2016-05-02)

src/zope/testrunner/runner.py

@@ -959,3 +959,6 @@ def readline(self):
         print('*'*70)
         print()
         return 'c\n'
+
+    def close(self):
+        pass

src/zope/testrunner/tests/test_runner.py

@@ -171,3 +171,12 @@ class ZZ(K1, K2, K3): pass
         # Sorting by reverse MRO, as computed by Python's MRO algorithm,
         # would put the layers in a different order: K3, K1, K2, ZZ.
         # Does that matter?  The class diagram is symmetric, so I think not.
+
+    def test_FakeInputContinueGenerator_close(self):
+        # multiprocessing (and likely other forkful frameworks want to
+        # close sys.stdin.  The test runner replaces sys.stdin with a
+        # FakeInputContinueGenerator for some reason. It should be
+        # closeable.
+
+        f = runner.FakeInputContinueGenerator()
+        f.close()