[Python-checkins] [3.9] bpo-28528: Fix pdb.checkline() attribute error when 'curframe' is None (GH-25438) (GH-26053)
miss-islington
webhook-mailer at python.org
Wed May 12 05:04:24 EDT 2021
https://github.com/python/cpython/commit/6c190b5ae5887d592bc8915148440a63da8b7cdd
commit: 6c190b5ae5887d592bc8915148440a63da8b7cdd
branch: 3.9
author: Erlend Egeberg Aasland <erlend.aasland at innova.no>
committer: miss-islington <31488909+miss-islington at users.noreply.github.com>
date: 2021-05-12T02:04:10-07:00
summary:
[3.9] bpo-28528: Fix pdb.checkline() attribute error when 'curframe' is None (GH-25438) (GH-26053)
Co-authored-by: Thomas Kluyver <takowl at gmail.com>
(cherry picked from commit 8563a7052ccd98e6a381d361664ce567afd5eb6e)
Co-authored-by: Erlend Egeberg Aasland <erlend.aasland at innova.no>
Automerge-Triggered-By: GH:iritkatriel
files:
A Misc/NEWS.d/next/Library/2021-04-29-00-48-00.bpo-28528.JLAVWj.rst
M Lib/pdb.py
M Lib/test/test_pdb.py
diff --git a/Lib/pdb.py b/Lib/pdb.py
index 98dc975eafafdc..a888a0a287f9c2 100755
--- a/Lib/pdb.py
+++ b/Lib/pdb.py
@@ -752,7 +752,8 @@ def checkline(self, filename, lineno):
"""
# this method should be callable before starting debugging, so default
# to "no globals" if there is no current frame
- globs = self.curframe.f_globals if hasattr(self, 'curframe') else None
+ frame = getattr(self, 'curframe', None)
+ globs = frame.f_globals if frame else None
line = linecache.getline(filename, lineno, globs)
if not line:
self.message('End of file')
diff --git a/Lib/test/test_pdb.py b/Lib/test/test_pdb.py
index 2523a5d691ac7e..b1cad22c830ffe 100644
--- a/Lib/test/test_pdb.py
+++ b/Lib/test/test_pdb.py
@@ -9,6 +9,7 @@
import unittest
import subprocess
import textwrap
+import linecache
from contextlib import ExitStack
from io import StringIO
@@ -1743,10 +1744,47 @@ def test_issue42383(self):
self.assertEqual(stdout.split('\n')[6].rstrip('\r'), expected)
+class ChecklineTests(unittest.TestCase):
+ def setUp(self):
+ linecache.clearcache() # Pdb.checkline() uses linecache.getline()
+
+ def tearDown(self):
+ support.unlink(support.TESTFN)
+
+ def test_checkline_before_debugging(self):
+ with open(support.TESTFN, "w") as f:
+ f.write("print(123)")
+ db = pdb.Pdb()
+ self.assertEqual(db.checkline(support.TESTFN, 1), 1)
+
+ def test_checkline_after_reset(self):
+ with open(support.TESTFN, "w") as f:
+ f.write("print(123)")
+ db = pdb.Pdb()
+ db.reset()
+ self.assertEqual(db.checkline(support.TESTFN, 1), 1)
+
+ def test_checkline_is_not_executable(self):
+ with open(support.TESTFN, "w") as f:
+ # Test for comments, docstrings and empty lines
+ s = textwrap.dedent("""
+ # Comment
+ \"\"\" docstring \"\"\"
+ ''' docstring '''
+
+ """)
+ f.write(s)
+ db = pdb.Pdb()
+ num_lines = len(s.splitlines()) + 2 # Test for EOF
+ for lineno in range(num_lines):
+ self.assertFalse(db.checkline(support.TESTFN, lineno))
+
+
def load_tests(*args):
from test import test_pdb
suites = [
unittest.makeSuite(PdbTestCase),
+ unittest.makeSuite(ChecklineTests),
doctest.DocTestSuite(test_pdb)
]
return unittest.TestSuite(suites)
diff --git a/Misc/NEWS.d/next/Library/2021-04-29-00-48-00.bpo-28528.JLAVWj.rst b/Misc/NEWS.d/next/Library/2021-04-29-00-48-00.bpo-28528.JLAVWj.rst
new file mode 100644
index 00000000000000..97731ad882e032
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2021-04-29-00-48-00.bpo-28528.JLAVWj.rst
@@ -0,0 +1,2 @@
+Fix a bug in :mod:`pdb` where :meth:`~pdb.Pdb.checkline` raises
+:exc:`AttributeError` if it is called after :meth:`~pdb.Pdb.reset`.
More information about the Python-checkins
mailing list