From ac61d7a5bfcd85acbea2996fa3688bf631fcd7ab Mon Sep 17 00:00:00 2001
From: goodboy <goodboy_foss@protonmail.com>
Date: Fri, 14 Aug 2026 10:59:07 -0400
Subject: [PATCH] Use a short UDS reaper test bindspace

Darwin's pytest `tmp_path` can already exceed the 104-byte AF_UNIX
budget before appending either synthetic socket filename. That made
the new sentinel-policy regression fail identically in both macOS
matrix legs without exercising reaper behavior.

Allocate the test's real socket files under a short
`/tmp/tractor-reap-*` directory and retain scoped cleanup.

Review: PR #480 (goodboy)
https://github.com/goodboy/tractor/pull/480

(this patch was generated in some part by `opencode` using
`gpt-5.6-sol` (`openai`))
---
 tests/ipc/test_each_tpt.py | 55 +++++++++++++++++++++-----------------
 1 file changed, 30 insertions(+), 25 deletions(-)

diff --git a/tests/ipc/test_each_tpt.py b/tests/ipc/test_each_tpt.py
index c52dc3cd..b62e2341 100644
--- a/tests/ipc/test_each_tpt.py
+++ b/tests/ipc/test_each_tpt.py
@@ -8,6 +8,7 @@ from pathlib import Path
 import socket
 import stat
 import sys
+import tempfile
 from types import SimpleNamespace
 from unittest.mock import Mock
 
@@ -127,7 +128,6 @@ def test_reaper_uses_default_uds_bindspace(
 
 def test_automatic_reaper_preserves_registry_sentinel(
     monkeypatch: pytest.MonkeyPatch,
-    tmp_path: Path,
 ):
     '''
     Reserve unconditional registry cleanup for the explicit CLI.
@@ -141,31 +141,36 @@ def test_automatic_reaper_preserves_registry_sentinel(
     '''
     from tractor._testing import _reap
 
-    registry_path: Path = tmp_path / 'registry@1616.sock'
-    actor_path: Path = tmp_path / 'worker@1234.sock'
-    socks: list[socket.socket] = []
-    for path in (registry_path, actor_path):
-        sock = socket.socket(socket.AF_UNIX)
-        sock.bind(str(path))
-        socks.append(sock)
+    with tempfile.TemporaryDirectory(
+        prefix='tractor-reap-',
+        dir='/tmp',
+    ) as tmpdir:
+        bindspace: Path = Path(tmpdir)
+        registry_path: Path = bindspace / 'registry@1616.sock'
+        actor_path: Path = bindspace / 'worker@1234.sock'
+        socks: list[socket.socket] = []
+        for path in (registry_path, actor_path):
+            sock = socket.socket(socket.AF_UNIX)
+            sock.bind(str(path))
+            socks.append(sock)
 
-    monkeypatch.setattr(_reap, '_is_alive', lambda pid: False)
-    try:
-        assert _reap.find_orphaned_uds(
-            uds_dir=str(tmp_path),
-        ) == [str(actor_path)]
-        assert set(
-            _reap.find_orphaned_uds(
-                uds_dir=str(tmp_path),
-                include_registry_sentinel=True,
-            )
-        ) == {
-            str(registry_path),
-            str(actor_path),
-        }
-    finally:
-        for sock in socks:
-            sock.close()
+        monkeypatch.setattr(_reap, '_is_alive', lambda pid: False)
+        try:
+            assert _reap.find_orphaned_uds(
+                uds_dir=str(bindspace),
+            ) == [str(actor_path)]
+            assert set(
+                _reap.find_orphaned_uds(
+                    uds_dir=str(bindspace),
+                    include_registry_sentinel=True,
+                )
+            ) == {
+                str(registry_path),
+                str(actor_path),
+            }
+        finally:
+            for sock in socks:
+                sock.close()
 
 
 def test_rt_dir_rejects_non_directory(