Merge topic 'debugger-breakpoints'

60b6383993 Debugger: Always clear existing breakpoints on setBreakpoints

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !8624

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;
 }