test: improve test clarity and modernize asyncio usage

- Add explanation for 200ms timeout in backpressure test
- Replace manual event loop creation with asyncio.run()
- Add assertion message with actual elapsed time for easier debugging

Author: dsarno

tests/test_resources_api.py

@@ -80,14 +80,10 @@ def test_resource_list_filters_and_rejects_traversal(resource_tools, tmp_path, m
     list_resources = resource_tools["list_resources"]
     # Only .cs under Assets should be listed
     import asyncio
-    loop = asyncio.new_event_loop()
-    try:
-        resp = loop.run_until_complete(
-            list_resources(ctx=DummyContext(), pattern="*.cs", under="Assets",
-                           limit=50, project_root=str(proj))
-        )
-    finally:
-        loop.close()
+    resp = asyncio.run(
+        list_resources(ctx=DummyContext(), pattern="*.cs", under="Assets",
+                       limit=50, project_root=str(proj))
+    )
     assert resp["success"] is True
     uris = resp["data"]["uris"]
     assert any(u.endswith("Assets/Scripts/A.cs") for u in uris)
@@ -100,14 +96,10 @@ def test_resource_list_rejects_outside_paths(resource_tools, tmp_path):
     # under points outside Assets
     list_resources = resource_tools["list_resources"]
     import asyncio
-    loop = asyncio.new_event_loop()
-    try:
-        resp = loop.run_until_complete(
-            list_resources(ctx=DummyContext(), pattern="*.cs", under="..",
-                           limit=10, project_root=str(proj))
-        )
-    finally:
-        loop.close()
+    resp = asyncio.run(
+        list_resources(ctx=DummyContext(), pattern="*.cs", under="..",
+                       limit=10, project_root=str(proj))
+    )
     assert resp["success"] is False
     assert "Assets" in resp.get(
         "error", "") or "under project root" in resp.get("error", "")

tests/test_telemetry_queue_worker.py

@@ -80,7 +80,8 @@ def slow_send(self, rec):
     elapsed_ms = (time.perf_counter() - start) * 1000.0
 
     # Should be fast despite backpressure (non-blocking enqueue or drop)
-    assert elapsed_ms < 200.0
+    # Timeout relaxed to 200ms to handle thread scheduling variance in CI/local environments
+    assert elapsed_ms < 200.0, f"Took {elapsed_ms:.1f}ms (expected <200ms)"
 
     # Allow worker to process some
     time.sleep(0.3)