PenaDevops/delve
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

pkg/proc: add tests for next interrupted by bp (#2632)

Browse Source 
Adds tests that make sure that when a next request
is interrupted by a breakpoint, the stepping breakpoints
are cleared.
This commit is contained in:
Suzy Mueller 2021-08-03 10:47:24 -06:00 committed by GitHub
parent 10406f96d5
commit df5812bf2d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 34 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
pkg/proc/proc_test.go
View File

@ -2545,6 +2545,40 @@ func TestStepConcurrentPtr(t *testing.T) {
}) })
} }
func TestStepOutBreakpoint(t *testing.T) {
protest.AllowRecording(t)
withTestProcess("testnextprog", t, func(p *proc.Target, fixture protest.Fixture) {
bp := setFileBreakpoint(p, t, fixture.Source, 13)
assertNoError(p.Continue(), t, "Continue()")
p.ClearBreakpoint(bp.Addr)
// StepOut should be interrupted by a breakpoint on the same goroutine.
setFileBreakpoint(p, t, fixture.Source, 14)
assertNoError(p.StepOut(), t, "StepOut()")
assertLineNumber(p, t, 14, "wrong line number")
if p.Breakpoints().HasSteppingBreakpoints() {
t.Fatal("has internal breakpoints after hitting breakpoint on same goroutine")
}
})
}
func TestNextBreakpoint(t *testing.T) {
protest.AllowRecording(t)
withTestProcess("testnextprog", t, func(p *proc.Target, fixture protest.Fixture) {
bp := setFileBreakpoint(p, t, fixture.Source, 34)
assertNoError(p.Continue(), t, "Continue()")
p.ClearBreakpoint(bp.Addr)
// Next should be interrupted by a breakpoint on the same goroutine.
setFileBreakpoint(p, t, fixture.Source, 14)
assertNoError(p.Next(), t, "Next()")
assertLineNumber(p, t, 14, "wrong line number")
if p.Breakpoints().HasSteppingBreakpoints() {
t.Fatal("has internal breakpoints after hitting breakpoint on same goroutine")
}
})
}
func TestStepOutDefer(t *testing.T) { func TestStepOutDefer(t *testing.T) {
protest.AllowRecording(t) protest.AllowRecording(t)
withTestProcess("testnextdefer", t, func(p *proc.Target, fixture protest.Fixture) { withTestProcess("testnextdefer", t, func(p *proc.Target, fixture protest.Fixture) {