summaryrefslogtreecommitdiffstats
path: root/Tests
diff options
context:
space:
mode:
authorBen McMorran <bemcmorr@microsoft.com>2023-07-12 18:25:12 (GMT)
committerBrad King <brad.king@kitware.com>2023-07-13 12:44:52 (GMT)
commit60b6383993013e720092025032a5844caac03111 (patch)
treebffca6b3e487e18b8df4ec2283ffe532ad4f651c /Tests
parentd769c59d783f0ffc46d61bb4715b5fb3a68181a8 (diff)
downloadCMake-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.cxx13
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;
}