Skip to content

Commit

Permalink
Make program_with_two_breakpoints has exactly 2 breakpoints (#517)
Browse files Browse the repository at this point in the history
This test case was introduced as a regression test of
#274
in #406.

The description of issues/274 says an exception will be raised
if the byebug CLI is terminated when main program is sleeping.

Before this commit there are 2 breakpoints (`byebug` method call)
before `sleep 3` and `"cont"` is called once because the arugment of
`remote_debug_connect_and_interrupt` is `"cont"`.
I think this is not intended.

And the method name of test case is `program_with_two_breakpoints`,
on the other hand the program had 3 breakpoints (`byebug`).
  • Loading branch information
yui-knk authored and deivid-rodriguez committed Dec 8, 2018
1 parent b02cf78 commit 49f524e
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions test/remote_debugging_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -55,13 +55,12 @@ def program_with_two_breakpoints
16:
17: byebug
18: thingy = #{example_class}.new
19: byebug
20: thingy.a
21: sleep 3
22: thingy.a
23: byebug
24: thingy.a
25: end
19: thingy.a
20: sleep 3
21: thingy.a
22: byebug
23: thingy.a
24: end
RUBY
end

Expand Down

0 comments on commit 49f524e

Please sign in to comment.