From 1095cec0dc1f9f27bd374c0080a9fba7c1d10939 Mon Sep 17 00:00:00 2001
From: hpk <none@none>
Date: Sat, 10 Feb 2007 18:00:58 +0100
Subject: [PATCH] [svn r38398] for setup/teardown pairs the correct idiom is

    setup()
    try:
        ...
    finally:
        teardown()

and not to do the setup() within the try,
because if that raises, teardown() will
likely go wrong as well etc.

--HG--
branch : trunk
---
 py/test/rsession/executor.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/py/test/rsession/executor.py b/py/test/rsession/executor.py
index 201ae9fc0..cadad4f34 100644
--- a/py/test/rsession/executor.py
+++ b/py/test/rsession/executor.py
@@ -22,8 +22,8 @@ class RunExecutor(object):
 
     def run(self, capture=True):
         if capture:
+            self.item.startcapture()
             try:
-                self.item.startcapture()
                 self.item.run()
             finally:
                 self.item.finishcapture()