diff options
author | Ben McMorran <bemcmorr@microsoft.com> | 2023-07-12 18:25:12 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2023-07-13 12:44:52 (GMT) |
commit | 60b6383993013e720092025032a5844caac03111 (patch) | |
tree | bffca6b3e487e18b8df4ec2283ffe532ad4f651c /Tests | |
parent | d769c59d783f0ffc46d61bb4715b5fb3a68181a8 (diff) | |
download | CMake-60b6383993013e720092025032a5844caac03111.zip CMake-60b6383993013e720092025032a5844caac03111.tar.gz CMake-60b6383993013e720092025032a5844caac03111.tar.bz2 |
Debugger: Always clear existing breakpoints on setBreakpoints
Fixes: #25063
Diffstat (limited to 'Tests')
-rw-r--r-- | Tests/CMakeLib/testDebuggerBreakpointManager.cxx | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/Tests/CMakeLib/testDebuggerBreakpointManager.cxx b/Tests/CMakeLib/testDebuggerBreakpointManager.cxx index 83734ea..f654442 100644 --- a/Tests/CMakeLib/testDebuggerBreakpointManager.cxx +++ b/Tests/CMakeLib/testDebuggerBreakpointManager.cxx @@ -51,6 +51,13 @@ static bool testHandleBreakpointRequestBeforeFileIsLoaded() sourcePath, false); ASSERT_BREAKPOINT(response.breakpoints[2], 2, sourceBreakpoints[2].line, sourcePath, false); + ASSERT_TRUE(breakpointManager.GetBreakpointCount() == 3); + + // setBreakpoints should override any existing breakpoints + setBreakpointRequest.breakpoints.value().clear(); + helper.Client->send(setBreakpointRequest).get(); + ASSERT_TRUE(breakpointManager.GetBreakpointCount() == 0); + return true; } @@ -103,6 +110,12 @@ static bool testHandleBreakpointRequestAfterFileIsLoaded() sourcePath, true); ASSERT_TRUE(notExpectBreakpointEvents.load()); + ASSERT_TRUE(breakpointManager.GetBreakpointCount() == 5); + + // setBreakpoints should override any existing breakpoints + setBreakpointRequest.breakpoints.value().clear(); + helper.Client->send(setBreakpointRequest).get(); + ASSERT_TRUE(breakpointManager.GetBreakpointCount() == 0); return true; } |