[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