8000 gh-124703: Change back to raising bdb.BdbQuit when exiting pdb in 'inline' mode in a REPL session by ADThomas-astro · Pull Request #130395 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-124703: Change back to raising bdb.BdbQuit when exiting pdb in 'inline' mode in a REPL session #130395

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Feb 25, 2025
Merged

gh-124703: Change back to raising bdb.BdbQuit when exiting pdb in 'inline' mode in a REPL session #130395

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
f77047c
Add fix and test
ADThomas-astro Feb 21, 2025
02884ab
📜🤖 Added by blurb_it.
blurb-it[bot] Feb 21, 2025
28f937a
Add "GH" prefix to issue number in Lib/test/test_pdb.py
ADThomas-astro Feb 22, 2025
670a6da
Update Misc/NEWS.d/next/Library/2025-02-21-09-05-44.gh-issue-124703.A…
ADThomas-astro Feb 22, 2025
c76a571
Add comment in 'do_quit' to address code review
ADThomas-astro Feb 22, 2025
2995125
Change word in comment
ADThomas-astro Feb 22, 2025
b45d1ca
Remove env var that is ignored due to -I
ADThomas-astro Feb 22, 2025
cdccdea
Make test a bit more robust
ADThomas-astro Feb 22, 2025
5864a35
Import 'spwan_repl' from test_reply.py
ADThomas-astro Feb 24, 2025
0cc42e0
Add comment about deferred import
ADThomas-astro Feb 24, 2025
acb9826
Sort alphabetically
ADThomas-astro Feb 24, 2025
c800b0f
Relocate import statement
ADThomas-astro Feb 24, 2025
e437fa0
Make test case strings more intuitive
ADThomas-astro Feb 24, 2025
f516a30
Remove variable for 'Quit anyway'
ADThomas-astro Feb 25, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Import 'spwan_repl' from test_reply.py
  • Loading branch information
@ADThomas-astro
ADThomas-astro committed Feb 24, 2025
commit 5864a354a5ddf1e5b00048065b87f4158f081954
17 changes: 1 addition & 16 deletions Lib/test/test_pdb.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4345,22 +4345,6 @@ def test_quit(self):
self.assertEqual(stdout.count(QUIT_PROMPT_QUESTION), 2)


def spawn_repl():
"""Run the basic Python REPL. Returns a Popen object."""
# This function is based on the function "spawn_repl" in test_repl.py.
# See comments there for the rationale for the command line args.
stdin_fname = os.path.join(os.path.dirname(sys.executable), '<stdin>')
proc = subprocess.Popen(
[stdin_fname, '-I', '-i'],
executable=sys.executable,
text=True,
stdin=subprocess.PIPE,
stdout=subprocess.PIPE,
stderr=subprocess.STDOUT,
)
return proc


@support.force_not_colorized_test_class
@support.requires_subprocess()
class TestREPLSession(unittest.TestCase):
Expand All @@ -4373,6 +4357,7 @@ def test_return_from_inline_mode_to_REPL(self):
pdb.set_trace(commands=['x * 3', 'q'])
x[::-1] * 3
"""
from test.test_repl import spawn_repl
p = spawn_repl()
p.stdin.write(textwrap.dedent(user_input))
output = kill_python(p)
Expand Down
Loading
0