[Lldb-commits] [lldb] [LLDB] Fix tests on Windows (PR #131600)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Tue Mar 18 01:04:29 PDT 2025


================
@@ -201,8 +201,15 @@ def test_watch_address_with_invalid_watch_size(self):
                 value.GetValueAsUnsigned(), 365, wp_opts, error
             )
             self.assertFalse(watchpoint)
-            self.expect(
-                error.GetCString(),
-                exe=False,
-                substrs=["Setting one of the watchpoint resources failed"],
-            )
+            if self.getPlatform() == "windows":
----------------
labath wrote:

I suspect this depends on whether you're using the lldb-server implementation (optional on windows, and the only option elsewhere) or the in process implementation (the default on windows when running locally). Maybe just make it accept both (with a regex) and add a comment.

https://github.com/llvm/llvm-project/pull/131600


More information about the lldb-commits mailing list