[Lldb-commits] [lldb] [lldb-dap] Migrate DAP attach tests. (PR #210814)

[Charles Zablit via lldb-commits]

================
@@ -194,6 +202,24 @@ def create_server_debug_adapter(
         self.assertTrue(adapter.is_server, "adapter should run as a server.")
         return adapter
 
+    def get_debug_server_path(self) -> Optional[Path]:
+        # Tries to find simulation/lldb-server/gdbserver tool path.
+        platform = self.getPlatform()
+        if platform == "windows":
+            # TODO: Update this for windows as it now supports lldb-server.
+            return None
+
+        if self.platformIsDarwin():
+            if platform == "macosx":
+                server_exe = lldbgdbserverutils.get_debugserver_exe()
+            return None
----------------
charles-zablit wrote:

I see, but we are always returning `None` for `self.platformIsDarwin():` right now. Is that expected? The tests should run on macOS right?

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