From 91bd4bbaeac37d12e61c9c7b033f55ec9f1ab562 Mon Sep 17 00:00:00 2001
From: Maru Newby <marun@redhat.com>
Date: Tue, 9 Apr 2013 02:33:03 +0000
Subject: [PATCH] Allow pdb debugging in manually-invoked tests.

 * The old behavior required setting environment variables to
   disable capture of stdout and stderr to allow debugging via pdb.
   The new behavior requires enabling capture of stdout/stderr
   instead. This is already the case in nova.
 * testr behavior is unmodified.
 * Tip 'o the hat to Robert Collins (lifeless)

Change-Id: Ica021eeab07d5a540ce42436c9cc8000c3fd239a
---
 .testr.conf           | 2 +-
 quantum/tests/base.py | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/.testr.conf b/.testr.conf
index 9339183aa..54f82b362 100644
--- a/.testr.conf
+++ b/.testr.conf
@@ -1,4 +1,4 @@
 [DEFAULT]
-test_command=${PYTHON:-python} -m subunit.run discover -t ./ quantum/tests/unit $LISTOPT $IDOPTION
+test_command=OS_STDOUT_CAPTURE=1 OS_STDERR_CAPTURE=1 ${PYTHON:-python} -m subunit.run discover -t ./ quantum/tests/unit $LISTOPT $IDOPTION
 test_id_option=--load-list $IDFILE
 test_list_option=--list
diff --git a/quantum/tests/base.py b/quantum/tests/base.py
index 95664490a..42f48dd28 100644
--- a/quantum/tests/base.py
+++ b/quantum/tests/base.py
@@ -50,10 +50,10 @@ class BaseTestCase(testtools.TestCase):
 
         self.addCleanup(CONF.reset)
 
-        if os.environ.get('OS_STDOUT_NOCAPTURE') not in TRUE_STRING:
+        if os.environ.get('OS_STDOUT_CAPTURE') in TRUE_STRING:
             stdout = self.useFixture(fixtures.StringStream('stdout')).stream
             self.useFixture(fixtures.MonkeyPatch('sys.stdout', stdout))
-        if os.environ.get('OS_STDERR_NOCAPTURE') not in TRUE_STRING:
+        if os.environ.get('OS_STDERR_CAPTURE') in TRUE_STRING:
             stderr = self.useFixture(fixtures.StringStream('stderr')).stream
             self.useFixture(fixtures.MonkeyPatch('sys.stderr', stderr))
         self.stubs = stubout.StubOutForTesting()
-- 
2.45.2