[Lldb-commits] [lldb] Minor fix to connect-url to support unix-connect sockets on localhost (PR #142875)

[Pavel Labath via lldb-commits]

================
@@ -57,3 +57,50 @@ def test_platform_process_connect(self):
         self.assertEqual(frame.GetFunction().GetName(), "main")
         self.assertEqual(frame.FindVariable("argc").GetValueAsSigned(), 2)
         process.Continue()
+
+    @skipIfRemote
+    @expectedFailureAll(hostoslist=["windows"], triple=".*-android")
+    @skipIfDarwin  # lldb-server not found correctly
+    @expectedFailureAll(oslist=["windows"])  # process modules not loaded
+    # lldb-server platform times out waiting for the gdbserver port number to be
+    # written to the pipe, yet it seems the gdbserver already has written it.
+    @expectedFailureAll(
+        archs=["aarch64"],
+        oslist=["freebsd"],
+        bugnumber="https://github.com/llvm/llvm-project/issues/84327",
+    )
+    @add_test_categories(["lldb-server"])
+    def test_platform_process_connect_with_unix_connect(self):
+        self.build()
+        import time
+
+        timestamp = int(time.time())
+        listen_url = "/tmp/listen_url_%s" % timestamp
+        port_file = "/tmp/port_file_%s" % timestamp
----------------
labath wrote:

Putting files into /tmp like this is fairly rude. For the port file, you can just use the build directory (like the other test). Named pipes are somewhat tricky as they have a fairly short limit on the length. I might use `SBHostOS.GetLLDBPath(ePathTypeLLDBTempSystemDir)` for that as it's likely to be shorter than the build dir.

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