From 31c1a32d58c6c2252bc788fb26815dbbb7ae1da1 Mon Sep 17 00:00:00 2001 From: Tyler Goodlet Date: Sun, 4 Oct 2020 19:39:00 -0400 Subject: [PATCH] Add re-entrant root breakpoint test; demonstrates a bug.. --- .../root_actor_breakpoint_forever.py | 11 +++++++++ tests/test_debugger.py | 23 ++++++++++++++++++- 2 files changed, 33 insertions(+), 1 deletion(-) create mode 100644 examples/debugging/root_actor_breakpoint_forever.py diff --git a/examples/debugging/root_actor_breakpoint_forever.py b/examples/debugging/root_actor_breakpoint_forever.py new file mode 100644 index 0000000..0332ab6 --- /dev/null +++ b/examples/debugging/root_actor_breakpoint_forever.py @@ -0,0 +1,11 @@ +import tractor + + +async def main(): + + while True: + await tractor.breakpoint() + + +if __name__ == '__main__': + tractor.run(main, debug_mode=True) diff --git a/tests/test_debugger.py b/tests/test_debugger.py index de76247..0770a14 100644 --- a/tests/test_debugger.py +++ b/tests/test_debugger.py @@ -114,6 +114,28 @@ def test_root_actor_bp(spawn, user_in_out): assert expect_err_str in str(child.before) +def test_root_actor_bp_forever(spawn): + "Re-enter a breakpoint from the root actor-task." + child = spawn('root_actor_breakpoint_forever') + + # do some "next" commands to demonstrate recurrent breakpoint + # entries + for _ in range(10): + child.sendline('next') + child.expect(r"\(Pdb\+\+\)") + + # do one continue which should trigger a new task to lock the tty + child.sendline('continue') + child.expect(r"\(Pdb\+\+\)") + + # XXX: this previously caused a bug! + child.sendline('n') + child.expect(r"\(Pdb\+\+\)") + + child.sendline('n') + child.expect(r"\(Pdb\+\+\)") + + def test_subactor_error(spawn): "Single subactor raising an error" @@ -128,7 +150,6 @@ def test_subactor_error(spawn): # send user command # (in this case it's the same for 'continue' vs. 'quit') child.sendline('continue') - child.expect('\r\n') # the debugger should enter a second time in the nursery # creating actor