This started as a spelling and whitespace cleanup. The comment for

the set_trace fiddling didn't make sense to me, and I ended up reworking
that part of the code.  We really do want to save and restore
pdb.set_trace, so that each dynamically nested level of doctest gets
sys.stdout fiddled to what's appropriate for *it*.  The only "trick"
really needed is that these layers of set_trace wrappers each call the
original pdb.set_trace (instead of the current pdb.set_trace).

1 files changed, 5 insertions, 6 deletions

diff --git a/Lib/test/test_doctest.py b/Lib/test/test_doctest.py
index 130cf04..28b72cb 100644
--- a/Lib/test/test_doctest.py
+++ b/Lib/test/test_doctest.py
@@ -987,11 +987,11 @@ Run the debugger on the docstring, and then restore sys.stdin.
 def test_pdb_set_trace():
     r"""Using pdb.set_trace from a doctest
 
-    You can use pdb.set_trace from a doctest. To do so, you must
+    You can use pdb.set_trace from a doctest.  To do so, you must
     retrieve the set_trace function from the pdb module at the time
-    you use it. The doctest module changes sys,stdout so that it can
-    capture program output. It also temporarily replaces pdb.set_trace
-    with a version that restores stdout. This is necessary for you to
+    you use it.  The doctest module changes sys.stdout so that it can
+    capture program output.  It also temporarily replaces pdb.set_trace
+    with a version that restores stdout.  This is necessary for you to
     see debugger output.
 
       >>> doc = '''
@@ -1041,8 +1041,7 @@ def test_pdb_set_trace():
       ... >>> calls_set_trace()
       ... '''
       >>> test = doctest.DocTest(doc, globals(), "foo", "foo.py", 0)
-      
-      >>> import tempfile
+
       >>> fake_stdin = tempfile.TemporaryFile(mode='w+')
       >>> fake_stdin.write('\n'.join([
       ...    'up',       # up out of pdb.set_trace