test(blockbuster): xfail async-close ServiceBrowser.cancel join blockers

bdraco · bdraco · commit f71bc55044a4 · 2026-05-20T11:24:36.000-05:00
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -24,15 +24,27 @@
 _BENCHMARKS_DIR = "tests/benchmarks"
 
 # Tests that perform sync IO inside the asyncio event loop and trip
-# blockbuster. Marked xfail so CI stays green; pop entries as they get
-# fixed so the underlying blocking call is gone for good. The single
-# entry below pins the deliberate sync-bootstrap path Zeroconf takes
-# when constructed with `use_asyncio=False` from inside a running loop
-# — that block on the loop-thread-ready event is the behavior the test
-# is documenting, so it stays xfail by design.
+# blockbuster. Marked xfail (strict=False) so CI stays green; pop
+# entries as the underlying blocking calls get fixed. Most of the
+# `test_async_service_registration*` and `test_async_tasks` entries
+# share a single root cause: `Zeroconf.async_close()` -> ... ->
+# `ServiceBrowser.cancel()` calls `Thread.join()` to drain the
+# dedicated browser thread, and on Python 3.10-3.12 the thread is
+# still alive when the join happens. `test_use_asyncio_false_*` is
+# by design (sync bootstrap when `use_asyncio=False` is requested from
+# inside a running loop); `test_run_coro_with_timeout` exercises the
+# sync-from-thread bridge intentionally. The strict=False marker keeps
+# the suite green on the Python versions where the race resolves the
+# other way.
 _KNOWN_BLOCKING: frozenset[str] = frozenset(
     {
+        "tests/test_asyncio.py::test_async_service_registration",
+        "tests/test_asyncio.py::test_async_service_registration_with_server_missing",
+        "tests/test_asyncio.py::test_async_service_registration_same_server_different_ports",
+        "tests/test_asyncio.py::test_async_service_registration_same_server_same_ports",
+        "tests/test_asyncio.py::test_async_tasks",
         "tests/test_core.py::Framework::test_use_asyncio_false_forces_thread_when_loop_running",
+        "tests/utils/test_asyncio.py::test_run_coro_with_timeout",
     }
 )
 
