* reworked capturing to only capture once per runtest cycle

* added readouterr() method to py.io capturing helpers

--HG--
branch : 1.0.x

CHANGELOG

@@ -10,6 +10,13 @@ Changes between 1.0.0b8 and 1.0.0b9
 
 * simplified py.test.mark API - see keyword plugin documentation
 
+* integrate better with logging: capturing now by default captures
+  test functions and their immediate setup/teardown in a single stream 
+
+* capsys and capfd funcargs now have a readouterr() and a close() method
+  (underlyingly py.io.StdCapture/FD objects are used which grew a 
+  readouterr() method as well to return snapshots of captured out/err)
+
 * make assert-reinterpretation work better with comparisons not 
   returning bools (reported with numpy from thanks maciej fijalkowski)
 

MANIFEST

@@ -351,6 +351,7 @@ py/test/plugin/pytest_terminal.py
 py/test/plugin/pytest_tmpdir.py
 py/test/plugin/pytest_unittest.py
 py/test/plugin/pytest_xfail.py
+py/test/plugin/test_pytest_iocapture.py
 py/test/plugin/test_pytest_runner.py
 py/test/plugin/test_pytest_runner_xunit.py
 py/test/plugin/test_pytest_terminal.py

doc/test/plugin/doctest.txt

@@ -37,7 +37,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_doctest.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_doctest.py
+.. _`pytest_doctest.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_doctest.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/figleaf.txt

@@ -32,7 +32,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_figleaf.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_figleaf.py
+.. _`pytest_figleaf.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_figleaf.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/hooklog.txt

@@ -28,7 +28,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_hooklog.py`: http://bitbucket.org/hpk42/py-trunk/raw/2b8d56b82ce6966960cf41d38dc2b794797912ba/py/test/plugin/pytest_hooklog.py
+.. _`pytest_hooklog.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_hooklog.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/index.txt

@@ -8,7 +8,7 @@ figleaf_ write and report coverage data with 'figleaf'.
 
 monkeypatch_ safely patch object attributes, dicts and environment variables.
 
-iocapture_ convenient capturing of writes to stdout/stderror streams and file descriptors.
+iocapture_ configurable per-test stdout/stderr capturing mechanisms.
 
 recwarn_ helpers for asserting deprecation and other warnings.
 
@@ -35,6 +35,16 @@ resultlog_ resultlog plugin for machine-readable logging of test results.
 terminal_ Implements terminal reporting of the full testing process.
 
 
+internal plugins / core functionality
+=====================================
+
+pdb_ interactive debugging with the Python Debugger.
+
+keyword_ mark test functions with keywords that may hold values.
+
+hooklog_ log invocations of extension hooks to a file.
+
+
 .. _`xfail`: xfail.html
 .. _`figleaf`: figleaf.html
 .. _`monkeypatch`: monkeypatch.html
@@ -47,3 +57,6 @@ terminal_ Implements terminal reporting of the full testing process.
 .. _`pocoo`: pocoo.html
 .. _`resultlog`: resultlog.html
 .. _`terminal`: terminal.html
+.. _`pdb`: pdb.html
+.. _`keyword`: keyword.html
+.. _`hooklog`: hooklog.html

doc/test/plugin/iocapture.txt

@@ -2,34 +2,90 @@
 pytest_iocapture plugin
 =======================
 
-convenient capturing of writes to stdout/stderror streams and file descriptors.
+configurable per-test stdout/stderr capturing mechanisms.
 
 .. contents::
   :local:
 
-Example Usage
+This plugin captures stdout/stderr output for each test separately. 
-----------------------
+In case of test failures this captured output is shown grouped 
+togtther with the test. 
 
-You can use the `capsys funcarg`_ to capture writes 
+The plugin also provides test function arguments that help to
-to stdout and stderr streams by using it in a test 
+assert stdout/stderr output from within your tests, see the 
-likes this:
+`funcarg example`_. 
+
+
+Capturing of input/output streams during tests 
+---------------------------------------------------
+
+By default ``sys.stdout`` and ``sys.stderr`` are substituted with
+temporary streams during the execution of tests and setup/teardown code.  
+During the whole testing process it will re-use the same temporary 
+streams allowing to play well with the logging module which easily
+takes ownership on these streams. 
+
+Also, 'sys.stdin' is substituted with a file-like "null" object that 
+does not return any values.  This is to immediately error out
+on tests that wait on reading something from stdin. 
+
+You can influence output capturing mechanisms from the command line::
+
+    py.test -s            # disable all capturing
+    py.test --capture=sys # set StringIO() to each of sys.stdout/stderr 
+    py.test --capture=fd  # capture stdout/stderr on Filedescriptors 1/2 
+
+If you set capturing values in a conftest file like this::
+
+    # conftest.py
+    conf_capture = 'fd'
+
+then all tests in that directory will execute with "fd" style capturing. 
+
+sys-level capturing 
+------------------------------------------
+
+Capturing on 'sys' level means that ``sys.stdout`` and ``sys.stderr`` 
+will be replaced with StringIO() objects.   
+
+FD-level capturing and subprocesses
+------------------------------------------
+
+The ``fd`` based method means that writes going to system level files
+based on the standard file descriptors will be captured, for example 
+writes such as ``os.write(1, 'hello')`` will be captured properly. 
+Capturing on fd-level will include output generated from 
+any subprocesses created during a test. 
+
+.. _`funcarg example`:
+
+Example Usage of the capturing Function arguments
+---------------------------------------------------
+
+You can use the `capsys funcarg`_ and `capfd funcarg`_ to 
+capture writes to stdout and stderr streams.  Using the
+funcargs frees your test from having to care about setting/resetting 
+the old streams and also interacts well with py.test's own 
+per-test capturing.  Here is an example test function:
 
 .. sourcecode:: python
 
     def test_myoutput(capsys):
         print "hello" 
         print >>sys.stderr, "world"
-        out, err = capsys.reset()
+        out, err = capsys.readouterr()
         assert out == "hello\n"
         assert err == "world\n"
         print "next"
-        out, err = capsys.reset()
+        out, err = capsys.readouterr()
         assert out == "next\n" 
 
-The ``reset()`` call returns a tuple and will restart 
+The ``readouterr()`` call snapshots the output so far - 
-capturing so that you can successively check for output. 
+and capturing will be continued.  After the test 
-After the test function finishes the original streams
+function finishes the original streams will 
-will be restored.
+be restored.  If you want to capture on 
+the filedescriptor level you can use the ``capfd`` function
+argument which offers the same interface.
 
 .. _`capsys funcarg`:
 
@@ -38,8 +94,8 @@ the 'capsys' test function argument
 -----------------------------------
 
 captures writes to sys.stdout/sys.stderr and makes 
-them available successively via a ``capsys.reset()`` method 
+them available successively via a ``capsys.readouterr()`` method 
-which returns a ``(out, err)`` tuple of captured strings. 
+which returns a ``(out, err)`` tuple of captured snapshot strings. 
 
 .. _`capfd funcarg`:
 
@@ -48,8 +104,17 @@ the 'capfd' test function argument
 ----------------------------------
 
 captures writes to file descriptors 1 and 2 and makes 
-them available successively via a ``capsys.reset()`` method 
+snapshotted ``(out, err)`` string tuples available 
-which returns a ``(out, err)`` tuple of captured strings. 
+via the ``capsys.readouterr()`` method. 
+
+command line options
+--------------------
+
+
+``-s``
+    shortcut for --capture=no.
+``--capture=capture``
+    set IO capturing method during tests: sys|fd|no.
 
 Start improving this plugin in 30 seconds
 =========================================
@@ -63,7 +128,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_iocapture.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_iocapture.py
+.. _`pytest_iocapture.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_iocapture.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/keyword.txt

@@ -43,7 +43,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_keyword.py`: http://bitbucket.org/hpk42/py-trunk/raw/2b8d56b82ce6966960cf41d38dc2b794797912ba/py/test/plugin/pytest_keyword.py
+.. _`pytest_keyword.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_keyword.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/monkeypatch.txt

@@ -58,7 +58,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_monkeypatch.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_monkeypatch.py
+.. _`pytest_monkeypatch.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_monkeypatch.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/pdb.txt

@@ -28,7 +28,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_pdb.py`: http://bitbucket.org/hpk42/py-trunk/raw/2b8d56b82ce6966960cf41d38dc2b794797912ba/py/test/plugin/pytest_pdb.py
+.. _`pytest_pdb.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_pdb.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/pocoo.txt

@@ -28,7 +28,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_pocoo.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_pocoo.py
+.. _`pytest_pocoo.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_pocoo.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/recwarn.txt

@@ -58,7 +58,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_recwarn.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_recwarn.py
+.. _`pytest_recwarn.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_recwarn.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/restdoc.txt

@@ -32,7 +32,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_restdoc.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_restdoc.py
+.. _`pytest_restdoc.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_restdoc.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/resultlog.txt

@@ -28,7 +28,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_resultlog.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_resultlog.py
+.. _`pytest_resultlog.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_resultlog.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/terminal.txt

@@ -21,7 +21,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_terminal.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_terminal.py
+.. _`pytest_terminal.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_terminal.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/unittest.txt

@@ -31,7 +31,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_unittest.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_unittest.py
+.. _`pytest_unittest.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_unittest.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

doc/test/plugin/xfail.txt

@@ -33,7 +33,7 @@ Do you find the above documentation or the plugin itself lacking?
 
 Further information: extend_ documentation, other plugins_ or contact_.  
 
-.. _`pytest_xfail.py`: http://bitbucket.org/hpk42/py-trunk/raw/85fe614ab05f301f206935d11a477df184cbbce6/py/test/plugin/pytest_xfail.py
+.. _`pytest_xfail.py`: http://bitbucket.org/hpk42/py-trunk/raw/6e9879aca934933c6065776820f22095634a7edf/py/test/plugin/pytest_xfail.py
 .. _`extend`: ../extend.html
 .. _`plugins`: index.html
 .. _`contact`: ../../contact.html

py/conftest.py

@@ -43,3 +43,13 @@ def getsocketspec(config=None):
         if spec.socket:
             return spec
     py.test.skip("need '--gx socket=...'")
+
+
+def pytest_generate_tests(metafunc):
+    multi = getattr(metafunc.function, 'multi', None)
+    if multi is None:
+        return
+    assert len(multi.__dict__) == 1
+    for name, l in multi.__dict__.items():
+        for val in l:
+            metafunc.addcall(funcargs={name: val})

py/io/stdcapture.py

@@ -21,22 +21,53 @@ class Capture(object):
         return res, out, err 
     call = classmethod(call) 
 
-    def reset(self): 
+    def reset(self):
-        """ reset sys.stdout and sys.stderr and return captured output 
+        """ reset sys.stdout/stderr and return captured output as strings. """
-            as strings and restore sys.stdout/err.
+        if hasattr(self, '_suspended'):
-        """
+            outfile = self._kwargs['out']
-        x, y = self.done() 
+            errfile = self._kwargs['err']
-        outerr = x.read(), y.read() 
+            del self._kwargs
-        x.close()
+        else:
-        y.close()
+            outfile, errfile = self.done() 
+        out, err = "", ""
+        if outfile:
+            out = outfile.read()
+            outfile.close()
+        if errfile and errfile != outfile:
+            err = errfile.read()
+            errfile.close()
+        return out, err
+
+    def suspend(self):
+        """ return current snapshot captures, memorize tempfiles. """
+        assert not hasattr(self, '_suspended')
+        self._suspended = True
+        outerr = self.readouterr()
+        outfile, errfile = self.done()
+        self._kwargs['out'] = outfile
+        self._kwargs['err'] = errfile
         return outerr
 
+    def resume(self):
+        """ resume capturing with original temp files. """
+        assert self._suspended
+        self._initialize(**self._kwargs)
+        del self._suspended
+
+
 class StdCaptureFD(Capture): 
     """ This class allows to capture writes to FD1 and FD2 
         and may connect a NULL file to FD0 (and prevent
         reads from sys.stdin)
     """
-    def __init__(self, out=True, err=True, mixed=False, in_=True, patchsys=True): 
+    def __init__(self, out=True, err=True, 
+                 mixed=False, in_=True, patchsys=True): 
+        self._kwargs = locals().copy()
+        del self._kwargs['self']
+        self._initialize(**self._kwargs)
+
+    def _initialize(self, out=True, err=True, 
+                    mixed=False, in_=True, patchsys=True): 
         if in_:
             self._oldin = (sys.stdin, os.dup(0))
             sys.stdin  = DontReadFromInput()
@@ -44,14 +75,19 @@ class StdCaptureFD(Capture):
             os.dup2(fd, 0)
             os.close(fd)
         if out: 
-            self.out = py.io.FDCapture(1) 
+            tmpfile = None
+            if isinstance(out, file):
+                tmpfile = out
+            self.out = py.io.FDCapture(1, tmpfile=tmpfile)
             if patchsys: 
                 self.out.setasfile('stdout')
         if err: 
             if mixed and out:
                 tmpfile = self.out.tmpfile 
+            elif isinstance(err, file):
+                tmpfile = err
             else:
-                tmpfile = None    
+                tmpfile = None
             self.err = py.io.FDCapture(2, tmpfile=tmpfile) 
             if patchsys: 
                 self.err.setasfile('stderr')
@@ -61,11 +97,11 @@ class StdCaptureFD(Capture):
         if hasattr(self, 'out'): 
             outfile = self.out.done() 
         else:
-            outfile = StringIO()
+            outfile = None
         if hasattr(self, 'err'): 
             errfile = self.err.done() 
         else:
-            errfile = StringIO()
+            errfile = None 
         if hasattr(self, '_oldin'):
             oldsys, oldfd = self._oldin 
             os.dup2(oldfd, 0)
@@ -73,6 +109,20 @@ class StdCaptureFD(Capture):
             sys.stdin = oldsys 
         return outfile, errfile 
 
+    def readouterr(self):
+        """ return snapshot value of stdout/stderr capturings. """
+        l = []
+        for name in ('out', 'err'):
+            res = ""
+            if hasattr(self, name):
+                f = getattr(self, name).tmpfile
+                f.seek(0)
+                res = f.read()
+                f.truncate(0)
+                f.seek(0)
+            l.append(res)
+        return l 
+
 class StdCapture(Capture):
     """ This class allows to capture writes to sys.stdout|stderr "in-memory"
         and will raise errors on tries to read from sys.stdin. It only
@@ -80,21 +130,28 @@ class StdCapture(Capture):
         touch underlying File Descriptors (use StdCaptureFD for that). 
     """
     def __init__(self, out=True, err=True, in_=True, mixed=False):
+        self._kwargs = locals().copy()
+        del self._kwargs['self']
+        self._initialize(**self._kwargs)
+
+    def _initialize(self, out, err, in_, mixed):
         self._out = out
         self._err = err 
         self._in = in_
         if out: 
-            self.oldout = sys.stdout
+            self._oldout = sys.stdout
-            sys.stdout = self.newout = StringIO()
+            if not hasattr(out, 'write'):
+                out = StringIO()
+            sys.stdout = self.out = out
         if err: 
-            self.olderr = sys.stderr
+            self._olderr = sys.stderr
             if out and mixed: 
-                newerr = self.newout 
+                err = self.out 
-            else:
+            elif not hasattr(err, 'write'):
-                newerr = StringIO()
+                err = StringIO()
-            sys.stderr = self.newerr = newerr
+            sys.stderr = self.err = err
         if in_:
-            self.oldin  = sys.stdin
+            self._oldin  = sys.stdin
             sys.stdin  = self.newin  = DontReadFromInput()
 
     def done(self): 
@@ -102,28 +159,39 @@ class StdCapture(Capture):
         o,e = sys.stdout, sys.stderr
         if self._out: 
             try:
-                sys.stdout = self.oldout 
+                sys.stdout = self._oldout 
             except AttributeError:
                 raise IOError("stdout capturing already reset")
-            del self.oldout
+            del self._oldout
-            outfile = self.newout
+            outfile = self.out
             outfile.seek(0)
         else:
-            outfile = StringIO()
+            outfile = None
         if self._err: 
             try:
-                sys.stderr = self.olderr 
+                sys.stderr = self._olderr 
             except AttributeError:
                 raise IOError("stderr capturing already reset")
-            del self.olderr 
+            del self._olderr 
-            errfile = self.newerr 
+            errfile = self.err 
             errfile.seek(0)
         else:
-            errfile = StringIO()
+            errfile = None
         if self._in:
-            sys.stdin = self.oldin 
+            sys.stdin = self._oldin 
         return outfile, errfile
 
+    def readouterr(self):
+        """ return snapshot value of stdout/stderr capturings. """
+        out = err = ""
+        if self._out:
+            out = sys.stdout.getvalue()
+            sys.stdout.truncate(0)
+        if self._err:
+            err = sys.stderr.getvalue()
+            sys.stderr.truncate(0)
+        return out, err 
+
 class DontReadFromInput:
     """Temporary stub class.  Ideally when stdin is accessed, the
     capturing should be turned off, with possibly all data captured

py/io/testing/test_stdcapture.py

@@ -30,6 +30,19 @@ class TestStdCapture:
         assert out == "hello world\n"
         assert err == "hello error\n"
 
+    def test_capturing_readouterr(self):
+        cap = self.getcapture()
+        try:
+            print "hello world"
+            print >>sys.stderr, "hello error"
+            out, err = cap.readouterr()
+            assert out == "hello world\n"
+            assert err == "hello error\n"
+            print >>sys.stderr, "error2"
+        finally:
+            out, err = cap.reset()
+        assert err == "error2\n"
+
     def test_capturing_mixed(self):
         cap = self.getcapture(mixed=True)
         print "hello",
@@ -43,7 +56,7 @@ class TestStdCapture:
         cap = self.getcapture() 
         print "hello"
         cap.reset()
-        py.test.raises(EnvironmentError, "cap.reset()")
+        py.test.raises(Exception, "cap.reset()")
 
     def test_capturing_modify_sysouterr_in_between(self):
         oldout = sys.stdout 
@@ -67,7 +80,7 @@ class TestStdCapture:
         cap2 = self.getcapture() 
         print "cap2"
         out2, err2 = cap2.reset()
-        py.test.raises(EnvironmentError, "cap2.reset()")
+        py.test.raises(Exception, "cap2.reset()")
         out1, err1 = cap1.reset() 
         assert out1 == "cap1\n"
         assert out2 == "cap2\n"
@@ -104,6 +117,24 @@ class TestStdCapture:
         py.test.raises(IOError, "sys.stdin.read()")
         out, err = cap.reset()
 
+    def test_suspend_resume(self):
+        cap = self.getcapture(out=True, err=False, in_=False)
+        try:
+            print "hello"
+            sys.stderr.write("error\n")
+            out, err = cap.suspend()
+            assert out == "hello\n"
+            assert not err 
+            print "in between"
+            sys.stderr.write("in between\n")
+            cap.resume()
+            print "after"
+            sys.stderr.write("error_after\n")
+        finally:
+            out, err = cap.reset()
+        assert out == "after\n"
+        assert not err 
+
 class TestStdCaptureFD(TestStdCapture): 
     def getcapture(self, **kw): 
         return py.io.StdCaptureFD(**kw)
@@ -150,10 +181,14 @@ def test_callcapture_nofd():
         os.write(1, "hello")
         os.write(2, "hello")
         print x
-        print >>py.std.sys.stderr, y
+        print >>sys.stderr, y
         return 42
    
-    res, out, err = py.io.StdCapture.call(func, 3, y=4) 
+    capfd = py.io.StdCaptureFD(patchsys=False)
+    try:
+        res, out, err = py.io.StdCapture.call(func, 3, y=4) 
+    finally:
+        capfd.reset()
     assert res == 42 
     assert out.startswith("3") 
     assert err.startswith("4") 

py/test/defaultconftest.py

@@ -12,3 +12,4 @@ Instance = py.test.collect.Instance
 
 pytest_plugins = "default runner iocapture terminal keyword xfail tmpdir execnetcleanup monkeypatch recwarn pdb unittest".split()
 
+conf_capture = "fd"

py/test/plugin/pytest_default.py

@@ -81,9 +81,6 @@ def pytest_addoption(parser):
                help="don't cut any tracebacks (default is to cut).")
     group.addoption('--basetemp', dest="basetemp", default=None, metavar="dir",
                help="base temporary directory for this test run.")
-    group._addoption('--iocapture', action="store", default="fd", metavar="method",
-               type="choice", choices=['fd', 'sys', 'no'],
-               help="set iocapturing method: fd|sys|no.")
     group.addoption('--debug',
                action="store_true", dest="debug", default=False,
                help="generate and show debugging information.")

py/test/plugin/pytest_iocapture.py

@@ -1,60 +1,97 @@
 """
-convenient capturing of writes to stdout/stderror streams and file descriptors. 
+configurable per-test stdout/stderr capturing mechanisms. 
 
-Example Usage
+This plugin captures stdout/stderr output for each test separately. 
-----------------------
+In case of test failures this captured output is shown grouped 
+togtther with the test. 
 
-You can use the `capsys funcarg`_ to capture writes 
+The plugin also provides test function arguments that help to
-to stdout and stderr streams by using it in a test 
+assert stdout/stderr output from within your tests, see the 
-likes this:
+`funcarg example`_. 
+
+
+Capturing of input/output streams during tests 
+---------------------------------------------------
+
+By default ``sys.stdout`` and ``sys.stderr`` are substituted with
+temporary streams during the execution of tests and setup/teardown code.  
+During the whole testing process it will re-use the same temporary 
+streams allowing to play well with the logging module which easily
+takes ownership on these streams. 
+
+Also, 'sys.stdin' is substituted with a file-like "null" object that 
+does not return any values.  This is to immediately error out
+on tests that wait on reading something from stdin. 
+
+You can influence output capturing mechanisms from the command line::
+
+    py.test -s            # disable all capturing
+    py.test --capture=sys # set StringIO() to each of sys.stdout/stderr 
+    py.test --capture=fd  # capture stdout/stderr on Filedescriptors 1/2 
+
+If you set capturing values in a conftest file like this::
+
+    # conftest.py
+    conf_capture = 'fd'
+
+then all tests in that directory will execute with "fd" style capturing. 
+
+sys-level capturing 
+------------------------------------------
+
+Capturing on 'sys' level means that ``sys.stdout`` and ``sys.stderr`` 
+will be replaced with StringIO() objects.   
+
+FD-level capturing and subprocesses
+------------------------------------------
+
+The ``fd`` based method means that writes going to system level files
+based on the standard file descriptors will be captured, for example 
+writes such as ``os.write(1, 'hello')`` will be captured properly. 
+Capturing on fd-level will include output generated from 
+any subprocesses created during a test. 
+
+.. _`funcarg example`:
+
+Example Usage of the capturing Function arguments
+---------------------------------------------------
+
+You can use the `capsys funcarg`_ and `capfd funcarg`_ to 
+capture writes to stdout and stderr streams.  Using the
+funcargs frees your test from having to care about setting/resetting 
+the old streams and also interacts well with py.test's own 
+per-test capturing.  Here is an example test function:
 
 .. sourcecode:: python
 
     def test_myoutput(capsys):
         print "hello" 
         print >>sys.stderr, "world"
-        out, err = capsys.reset()
+        out, err = capsys.readouterr()
         assert out == "hello\\n"
         assert err == "world\\n"
         print "next"
-        out, err = capsys.reset()
+        out, err = capsys.readouterr()
         assert out == "next\\n" 
 
-The ``reset()`` call returns a tuple and will restart 
+The ``readouterr()`` call snapshots the output so far - 
-capturing so that you can successively check for output. 
+and capturing will be continued.  After the test 
-After the test function finishes the original streams
+function finishes the original streams will 
-will be restored. 
+be restored.  If you want to capture on 
+the filedescriptor level you can use the ``capfd`` function
+argument which offers the same interface. 
 """
 
 import py
 
 def pytest_addoption(parser):
     group = parser.getgroup("general")
-    group._addoption('-s', 
+    group._addoption('-s', action="store_const", const="no", dest="capture", 
-       action="store_true", dest="nocapture", default=False,
+        help="shortcut for --capture=no.")
-       help="disable catching of stdout/stderr during test run.")
+    group._addoption('--capture', action="store", default=None,
+        metavar="capture", type="choice", choices=['fd', 'sys', 'no'],
+        help="set IO capturing method during tests: sys|fd|no.")
 
-def determine_capturing(config, path=None):
-    iocapture = config.getvalue("iocapture", path=path)
-    if iocapture == "fd": 
-        return py.io.StdCaptureFD()
-    elif iocapture == "sys":
-        return py.io.StdCapture()
-    elif iocapture == "no": 
-        return py.io.StdCapture(out=False, err=False, in_=False)
-    else:
-        # how to raise errors here? 
-        raise config.Error("unknown io capturing: " + iocapture)
-
-def pytest_make_collect_report(__call__, collector):
-    cap = determine_capturing(collector.config, collector.fspath)
-    try:
-        rep = __call__.execute(firstresult=True)
-    finally:
-        outerr = cap.reset()
-    addouterr(rep, outerr)
-    return rep
-   
 def addouterr(rep, outerr):
     repr = getattr(rep, 'longrepr', None)
     if not hasattr(repr, 'addsection'):
@@ -64,79 +101,140 @@ def addouterr(rep, outerr):
             repr.addsection("Captured std%s" % secname, content.rstrip())
 
 def pytest_configure(config):
-    if not config.option.nocapture:
+    config.pluginmanager.register(CaptureManager(), 'capturemanager')
-        config.pluginmanager.register(CapturePerTest())
 
-
+class CaptureManager:
-class CapturePerTest:
     def __init__(self):
-        self.item2capture = {}
+        self._method2capture = {}
-        
+
-    def _setcapture(self, item):
+    def _startcapture(self, method):
-        assert item not in self.item2capture
+        if method == "fd": 
-        cap = determine_capturing(item.config, path=item.fspath)
+            return py.io.StdCaptureFD()
-        self.item2capture[item] = cap
+        elif method == "sys":
+            return py.io.StdCapture()
+        else:
+            raise ValueError("unknown capturing method: %r" % method)
+
+    def _getmethod(self, config, fspath):
+        if config.option.capture:
+            return config.option.capture
+        return config._conftest.rget("conf_capture", path=fspath)
+
+    def resumecapture_item(self, item):
+        method = self._getmethod(item.config, item.fspath)
+        if not hasattr(item, 'outerr'):
+            item.outerr = ('', '') # we accumulate outerr on the item
+        return self.resumecapture(method)
+
+    def resumecapture(self, method):
+        if hasattr(self, '_capturing'):
+            raise ValueError("cannot resume, already capturing with %r" % 
+                (self._capturing,))
+        if method != "no":
+            cap = self._method2capture.get(method)
+            if cap is None:
+                cap = self._startcapture(method)
+                self._method2capture[method] = cap 
+            else:
+                cap.resume()
+        self._capturing = method 
+
+    def suspendcapture(self):
+        self.deactivate_funcargs()
+        method = self._capturing
+        if method != "no":
+            cap = self._method2capture[method]
+            outerr = cap.suspend()
+        else:
+            outerr = "", ""
+        del self._capturing
+        return outerr 
+
+    def activate_funcargs(self, pyfuncitem):
+        if not hasattr(pyfuncitem, 'funcargs'):
+            return
+        assert not hasattr(self, '_capturing_funcargs')
+        l = []
+        for name, obj in pyfuncitem.funcargs.items():
+            if name in ('capsys', 'capfd'):
+                obj._start()
+                l.append(obj)
+        if l:
+            self._capturing_funcargs = l
+
+    def deactivate_funcargs(self):
+        if hasattr(self, '_capturing_funcargs'):
+            for capfuncarg in self._capturing_funcargs:
+                capfuncarg._finalize()
+            del self._capturing_funcargs
+
+    def pytest_make_collect_report(self, __call__, collector):
+        method = self._getmethod(collector.config, collector.fspath)
+        self.resumecapture(method)
+        try:
+            rep = __call__.execute(firstresult=True)
+        finally:
+            outerr = self.suspendcapture()
+        addouterr(rep, outerr)
+        return rep
 
     def pytest_runtest_setup(self, item):
-        self._setcapture(item)
+        self.resumecapture_item(item)
 
     def pytest_runtest_call(self, item):
-        self._setcapture(item)
+        self.resumecapture_item(item)
+        self.activate_funcargs(item)
 
     def pytest_runtest_teardown(self, item):
-        self._setcapture(item)
+        self.resumecapture_item(item)
 
     def pytest_keyboard_interrupt(self, excinfo):
-        for cap in self.item2capture.values():
+        if hasattr(self, '_capturing'):
-            cap.reset()
+            self.suspendcapture()
-        self.item2capture.clear()
 
     def pytest_runtest_makereport(self, __call__, item, call):
-        capture = self.item2capture.pop(item)
+        self.deactivate_funcargs()
-        outerr = capture.reset()
-        # XXX shift reporting elsewhere 
         rep = __call__.execute(firstresult=True)
-        addouterr(rep, outerr)
+        outerr = self.suspendcapture()
-
+        outerr = (item.outerr[0] + outerr[0], item.outerr[1] + outerr[1])
+        if not rep.passed:
+            addouterr(rep, outerr)
+        if not rep.passed or rep.when == "teardown":
+            outerr = ('', '')
+        item.outerr = outerr 
         return rep
 
 def pytest_funcarg__capsys(request):
     """captures writes to sys.stdout/sys.stderr and makes 
-    them available successively via a ``capsys.reset()`` method 
+    them available successively via a ``capsys.readouterr()`` method 
-    which returns a ``(out, err)`` tuple of captured strings. 
+    which returns a ``(out, err)`` tuple of captured snapshot strings. 
     """ 
-    capture = CaptureFuncarg(py.io.StdCapture)
+    return CaptureFuncarg(request, py.io.StdCapture)
-    request.addfinalizer(capture.finalize)
-    return capture 
 
 def pytest_funcarg__capfd(request):
     """captures writes to file descriptors 1 and 2 and makes 
-    them available successively via a ``capsys.reset()`` method 
+    snapshotted ``(out, err)`` string tuples available 
-    which returns a ``(out, err)`` tuple of captured strings. 
+    via the ``capsys.readouterr()`` method. 
     """ 
-    capture = CaptureFuncarg(py.io.StdCaptureFD)
+    return CaptureFuncarg(request, py.io.StdCaptureFD)
-    request.addfinalizer(capture.finalize)
-    return capture 
 
-def pytest_pyfunc_call(pyfuncitem):
-    if hasattr(pyfuncitem, 'funcargs'):
-        for funcarg, value in pyfuncitem.funcargs.items():
-            if funcarg == "capsys" or funcarg == "capfd":
-                value.reset()
 
 class CaptureFuncarg:
-    _capture = None
+    def __init__(self, request, captureclass):
-    def __init__(self, captureclass):
+        self._cclass = captureclass
-        self._captureclass = captureclass
+        #request.addfinalizer(self._finalize)
 
-    def finalize(self):
+    def _start(self):
-        if self._capture:
+        self.capture = self._cclass()
-            self._capture.reset()
 
-    def reset(self):
+    def _finalize(self):
-        res = None
+        if hasattr(self, 'capture'):
-        if self._capture:
+            self.capture.reset()
-            res = self._capture.reset()
+            del self.capture 
-        self._capture = self._captureclass()
-        return res 
 
+    def readouterr(self):
+        return self.capture.readouterr()
+
+    def close(self):
+        self.capture.reset()
+        del self.capture

py/test/plugin/pytest_pdb.py

@@ -23,11 +23,18 @@ def pytest_configure(config):
 class PdbInvoke:
     def pytest_runtest_makereport(self, item, call):
         if call.excinfo and not call.excinfo.errisinstance(Skipped): 
+            # XXX hack hack hack to play well with capturing
+            capman = item.config.pluginmanager.impname2plugin['capturemanager']
+            capman.suspendcapture() 
+
             tw = py.io.TerminalWriter()
             repr = call.excinfo.getrepr()
             repr.toterminal(tw) 
             post_mortem(call.excinfo._excinfo[2])
 
+            # XXX hack end 
+            capman.resumecapture_item(item)
+
 class Pdb(py.std.pdb.Pdb):
     def do_list(self, arg):
         self.lastcmd = 'list'

py/test/plugin/pytest_pytester.py

@@ -301,6 +301,9 @@ class TmpTestdir:
         assert script.check()
         return py.std.sys.executable, script
 
+    def runpython(self, script):
+        return self.run(py.std.sys.executable, script)
+
     def runpytest(self, *args):
         p = py.path.local.make_numbered_dir(prefix="runpytest-", 
             keep=None, rootdir=self.tmpdir)
@@ -520,7 +523,6 @@ def test_testdir_runs_with_plugin(testdir):
 def pytest_funcarg__venv(request):
     p = request.config.mktemp(request.function.__name__, numbered=True)
     venv = VirtualEnv(str(p)) 
-    venv.create()
     return venv 
    
 def pytest_funcarg__py_setup(request):
@@ -533,6 +535,7 @@ def pytest_funcarg__py_setup(request):
 class SetupBuilder:
     def __init__(self, setup_path):
         self.setup_path = setup_path
+        assert setup_path.check()
 
     def make_sdist(self, destdir=None):
         temp = py.path.local.mkdtemp()
@@ -567,9 +570,9 @@ class VirtualEnv(object):
     def _cmd(self, name):
         return os.path.join(self.path, 'bin', name)
 
-    @property
+    def ensure(self):
-    def valid(self):
+        if not os.path.exists(self._cmd('python')):
-        return os.path.exists(self._cmd('python'))
+            self.create()
 
     def create(self, sitepackages=False):
         args = ['virtualenv', self.path]
@@ -582,7 +585,7 @@ class VirtualEnv(object):
         return py.execnet.makegateway("popen//python=%s" %(python,))
 
     def pcall(self, cmd, *args, **kw):
-        assert self.valid
+        self.ensure()
         return subprocess.call([
                 self._cmd(cmd)
             ] + list(args),

py/test/plugin/pytest_terminal.py

@@ -314,7 +314,8 @@ class TerminalReporter:
                 self.write_sep("_", msg)
                 if hasattr(rep, 'node'):
                     self.write_line(self.gateway2info.get(
-                        rep.node.gateway, "node %r (platinfo not found? strange)")
+                        rep.node.gateway, 
+                        "node %r (platinfo not found? strange)")
                             [:self._tw.fullwidth-1])
                 rep.toterminal(self._tw)
 

py/test/plugin/test_pytest_iocapture.py

@@ -0,0 +1,363 @@
+import py, os, sys
+from py.__.test.plugin.pytest_iocapture import CaptureManager
+
+class TestCaptureManager:
+
+    def test_configure_per_fspath(self, testdir):
+        config = testdir.parseconfig(testdir.tmpdir)
+        assert config.getvalue("capture") is None
+        capman = CaptureManager()
+        assert capman._getmethod(config, None) == "fd" # default
+
+        for name in ('no', 'fd', 'sys'):
+            sub = testdir.tmpdir.mkdir("dir" + name)
+            sub.ensure("__init__.py")
+            sub.join("conftest.py").write('conf_capture = %r' % name)
+            assert capman._getmethod(config, sub.join("test_hello.py")) == name
+
+    @py.test.mark.multi(method=['no', 'fd', 'sys'])
+    def test_capturing_basic_api(self, method):
+        capouter = py.io.StdCaptureFD()
+        old = sys.stdout, sys.stderr, sys.stdin
+        try:
+            capman = CaptureManager()
+            capman.resumecapture(method)
+            print "hello"
+            out, err = capman.suspendcapture()
+            if method == "no":
+                assert old == (sys.stdout, sys.stderr, sys.stdin)
+            else:
+                assert out == "hello\n"
+            capman.resumecapture(method)
+            out, err = capman.suspendcapture()
+            assert not out and not err 
+        finally:
+            capouter.reset()
+      
+    def test_juggle_capturings(self, testdir):
+        capouter = py.io.StdCaptureFD()
+        try:
+            config = testdir.parseconfig(testdir.tmpdir)
+            capman = CaptureManager()
+            capman.resumecapture("fd")
+            py.test.raises(ValueError, 'capman.resumecapture("fd")')
+            py.test.raises(ValueError, 'capman.resumecapture("sys")')
+            os.write(1, "hello\n")
+            out, err = capman.suspendcapture()
+            assert out == "hello\n"
+            capman.resumecapture("sys")
+            os.write(1, "hello\n")
+            print >>sys.stderr, "world"
+            out, err = capman.suspendcapture()
+            assert not out
+            assert err == "world\n"
+        finally:
+            capouter.reset()
+
+def test_collect_capturing(testdir):
+    p = testdir.makepyfile("""
+        print "collect %s failure" % 13
+        import xyz42123
+    """)
+    result = testdir.runpytest(p)
+    result.stdout.fnmatch_lines([
+        "*Captured stdout*",
+        "*collect 13 failure*", 
+    ])
+
+class TestPerTestCapturing:
+    def test_capture_and_fixtures(self, testdir):
+        p = testdir.makepyfile("""
+            def setup_module(mod):
+                print "setup module"
+            def setup_function(function):
+                print "setup", function.__name__
+            def test_func1():
+                print "in func1"
+                assert 0
+            def test_func2():
+                print "in func2"
+                assert 0
+        """)
+        result = testdir.runpytest(p)
+        result.stdout.fnmatch_lines([
+            "setup module*",
+            "setup test_func1*",
+            "in func1*",
+            "setup test_func2*",
+            "in func2*", 
+        ]) 
+
+    def test_no_carry_over(self, testdir):
+        p = testdir.makepyfile("""
+            def test_func1():
+                print "in func1"
+            def test_func2():
+                print "in func2"
+                assert 0
+        """)
+        result = testdir.runpytest(p)
+        s = result.stdout.str()
+        assert "in func1" not in s 
+        assert "in func2" in s 
+
+
+    def test_teardown_capturing(self, testdir):
+        p = testdir.makepyfile("""
+            def setup_function(function):
+                print "setup func1"
+            def teardown_function(function):
+                print "teardown func1"
+                assert 0
+            def test_func1(): 
+                print "in func1"
+                pass
+        """)
+        result = testdir.runpytest(p)
+        assert result.stdout.fnmatch_lines([
+            '*teardown_function*',
+            '*Captured stdout*',
+            "setup func1*",
+            "in func1*",
+            "teardown func1*",
+            #"*1 fixture failure*"
+        ])
+
+    @py.test.mark.xfail
+    def test_teardown_final_capturing(self, testdir):
+        p = testdir.makepyfile("""
+            def teardown_module(mod):
+                print "teardown module"
+                assert 0
+            def test_func():
+                pass
+        """)
+        result = testdir.runpytest(p)
+        assert result.stdout.fnmatch_lines([
+            "teardown module*", 
+            #"*1 fixture failure*"
+        ])
+
+    def test_capturing_outerr(self, testdir): 
+        p1 = testdir.makepyfile("""
+            import sys 
+            def test_capturing():
+                print 42
+                print >>sys.stderr, 23 
+            def test_capturing_error():
+                print 1
+                print >>sys.stderr, 2
+                raise ValueError
+        """)
+        result = testdir.runpytest(p1)
+        result.stdout.fnmatch_lines([
+            "*test_capturing_outerr.py .F", 
+            "====* FAILURES *====",
+            "____*____", 
+            "*test_capturing_outerr.py:8: ValueError",
+            "*--- Captured stdout ---*",
+            "1",
+            "*--- Captured stderr ---*",
+            "2",
+        ])
+
+class TestLoggingInteraction:
+    def test_logging_stream_ownership(self, testdir):
+        p = testdir.makepyfile("""
+            def test_logging():
+                import logging
+                import StringIO
+                stream = StringIO.StringIO()
+                logging.basicConfig(stream=stream)
+                stream.close() # to free memory/release resources
+        """)
+        result = testdir.runpytest(p)
+        result.stderr.str().find("atexit") == -1
+
+    def test_capturing_and_logging_fundamentals(self, testdir):
+        # here we check a fundamental feature 
+        rootdir = str(py.path.local(py.__file__).dirpath().dirpath())
+        p = testdir.makepyfile("""
+            import sys
+            sys.path.insert(0, %r)
+            import py, logging
+            cap = py.io.StdCaptureFD(out=False, in_=False)
+            logging.warn("hello1")
+            outerr = cap.suspend()
+
+            print "suspeneded and captured", outerr
+
+            logging.warn("hello2")
+
+            cap.resume()
+            logging.warn("hello3")
+
+            outerr = cap.suspend()
+            print "suspend2 and captured", outerr
+        """ % rootdir)
+        result = testdir.runpython(p)
+        assert result.stdout.fnmatch_lines([
+            "suspeneded and captured*hello1*",
+            "suspend2 and captured*hello2*WARNING:root:hello3*",
+        ])
+        assert "atexit" not in result.stderr.str()
+        
+           
+    def test_logging_and_immediate_setupteardown(self, testdir):
+        p = testdir.makepyfile("""
+            import logging
+            def setup_function(function):
+                logging.warn("hello1")
+
+            def test_logging():
+                logging.warn("hello2")
+                assert 0
+
+            def teardown_function(function):
+                logging.warn("hello3")
+                assert 0
+        """)
+        for optargs in (('--capture=sys',), ('--capture=fd',)):
+            print optargs
+            result = testdir.runpytest(p, *optargs)
+            s = result.stdout.str()
+            result.stdout.fnmatch_lines([
+                "*WARN*hello1", 
+                "*WARN*hello2", 
+                "*WARN*hello3", 
+            ])
+            # verify proper termination
+            assert "closed" not in s
+
+    @py.test.mark.xfail
+    def test_logging_and_crossscope_fixtures(self, testdir):
+        # XXX also needs final teardown reporting to work!
+        p = testdir.makepyfile("""
+            import logging
+            def setup_module(function):
+                logging.warn("hello1")
+
+            def test_logging():
+                logging.warn("hello2")
+                assert 0
+
+            def teardown_module(function):
+                logging.warn("hello3")
+                assert 0
+        """)
+        for optargs in (('--iocapture=sys',), ('--iocapture=fd',)):
+            print optargs
+            result = testdir.runpytest(p, *optargs)
+            s = result.stdout.str()
+            result.stdout.fnmatch_lines([
+                "*WARN*hello1", 
+                "*WARN*hello2", 
+                "*WARN*hello3", 
+            ])
+            # verify proper termination
+            assert "closed" not in s
+
+class TestCaptureFuncarg:
+    def test_std_functional(self, testdir):        
+        reprec = testdir.inline_runsource("""
+            def test_hello(capsys):
+                print 42
+                out, err = capsys.readouterr()
+                assert out.startswith("42")
+        """)
+        reprec.assertoutcome(passed=1)
+        
+    def test_stdfd_functional(self, testdir):        
+        reprec = testdir.inline_runsource("""
+            def test_hello(capfd):
+                import os
+                os.write(1, "42")
+                out, err = capfd.readouterr()
+                assert out.startswith("42")
+                capfd.close()
+        """)
+        reprec.assertoutcome(passed=1)
+
+    def test_partial_setup_failure(self, testdir):        
+        p = testdir.makepyfile("""
+            def test_hello(capfd, missingarg):
+                pass
+        """)
+        result = testdir.runpytest(p)
+        assert result.stdout.fnmatch_lines([
+            "*test_partial_setup_failure*",
+            "*1 failed*",
+        ])
+
+    def test_keyboardinterrupt_disables_capturing(self, testdir):        
+        p = testdir.makepyfile("""
+            def test_hello(capfd):
+                import os
+                os.write(1, "42")
+                raise KeyboardInterrupt()
+        """)
+        result = testdir.runpytest(p)
+        result.stdout.fnmatch_lines([
+            "*KEYBOARD INTERRUPT*"
+        ])
+        assert result.ret == 2
+
+
+
+class TestFixtureReporting:
+    @py.test.mark.xfail
+    def test_setup_fixture_error(self, testdir):
+        p = testdir.makepyfile("""
+            def setup_function(function):
+                print "setup func"
+                assert 0
+            def test_nada():
+                pass
+        """)
+        result = testdir.runpytest()
+        result.stdout.fnmatch_lines([
+            "*FIXTURE ERROR at setup of test_nada*",
+            "*setup_function(function):*",
+            "*setup func*",
+            "*assert 0*",
+            "*0 passed*1 error*",
+        ])
+        assert result.ret != 0
+    
+    @py.test.mark.xfail
+    def test_teardown_fixture_error(self, testdir):
+        p = testdir.makepyfile("""
+            def test_nada():
+                pass
+            def teardown_function(function):
+                print "teardown func"
+                assert 0
+        """)
+        result = testdir.runpytest()
+        result.stdout.fnmatch_lines([
+            "*FIXTURE ERROR at teardown*", 
+            "*teardown_function(function):*",
+            "*teardown func*",
+            "*assert 0*",
+            "*1 passed*1 error*",
+        ])
+
+    @py.test.mark.xfail
+    def test_teardown_fixture_error_and_test_failure(self, testdir):
+        p = testdir.makepyfile("""
+            def test_fail():
+                assert 0, "failingfunc"
+
+            def teardown_function(function):
+                print "teardown func"
+                assert 0
+        """)
+        result = testdir.runpytest()
+        result.stdout.fnmatch_lines([
+            "*failingfunc*", 
+            "*FIXTURE ERROR at teardown*", 
+            "*teardown_function(function):*",
+            "*teardown func*",
+            "*assert 0*",
+            "*1 failed*1 error",
+         ])

py/test/testing/test_install.py

@@ -1,9 +1,16 @@
 import py
 
-def test_make_sdist_and_run_it(py_setup, venv):
+def test_make_sdist_and_run_it(capfd, py_setup, venv):
-    sdist = py_setup.make_sdist(venv.path)
+    try:
-    venv.easy_install(str(sdist)) 
+        sdist = py_setup.make_sdist(venv.path)
-    gw = venv.makegateway()
+        venv.easy_install(str(sdist)) 
-    ch = gw.remote_exec("import py ; channel.send(py.__version__)")
+        gw = venv.makegateway()
-    version = ch.receive()
+        ch = gw.remote_exec("import py ; channel.send(py.__version__)")
-    assert version == py.__version__
+        version = ch.receive()
+        assert version == py.__version__
+    except KeyboardInterrupt:
+        raise
+    except:
+        print capfd.readouterr()
+        raise
+    capfd.close()

py/test/testing/test_parseopt.py

@@ -1,13 +1,11 @@
 import py
 from py.__.test import parseopt 
 
-pytest_plugins = 'pytest_iocapture'
-
 class TestParser:
     def test_init(self, capsys):
         parser = parseopt.Parser(usage="xyz")
         py.test.raises(SystemExit, 'parser.parse(["-h"])')
-        out, err = capsys.reset()
+        out, err = capsys.readouterr()
         assert out.find("xyz") != -1
 
     def test_group_add_and_get(self):