Add ORFS Integration test

ROADMAP.md

@@ -56,7 +56,7 @@ The pre-release phase focuses on:
 ### Quality & Testing
 - [x] Core test suite (80%+ coverage)
 - [x] Type hints on all public APIs
-- [ ] Integration tests with real ORFS flows
+- [x] Integration tests with real ORFS flows
 - [ ] Performance benchmarking suite
 - [ ] Load testing (50+ concurrent sessions)
 - [ ] Memory leak detection
@@ -192,7 +192,7 @@ We welcome community involvement at every stage:
 
 ---
 
-**Last Updated:** 2025-12-08
+**Last Updated:** 2026-02-23
 **Current Phase:** Phase 1 (Pre-Release → v0.5)
 
 *This roadmap is a living document and will evolve based on community feedback and priorities.*

tests/integration/conftest.py

@@ -3,24 +3,21 @@
 import asyncio
 from collections.abc import AsyncGenerator
 
-import pytest
 import pytest_asyncio
 from mcp import ClientSession, StdioServerParameters
 from mcp.client.stdio import stdio_client
 
 
-@pytest.fixture(scope="session")
-def event_loop():
-    """Create an event loop for the test session."""
-    loop = asyncio.new_event_loop()
-    yield loop
-    loop.close()
-
-
 @pytest_asyncio.fixture(scope="function")
 async def mcp_client() -> AsyncGenerator[ClientSession]:
     """Fixture providing a MCP client session."""
-    server_params = StdioServerParameters(command="python", args=["-m", "openroad_mcp.main"], env=None)
+    server_params = StdioServerParameters(
+        command="python",
+        args=["-m", "openroad_mcp.main"],
+        env={
+            "OPENROAD_ENABLE_COMMAND_VALIDATION": "false",
+        },
+    )
 
     try:
         async with stdio_client(server_params) as (read, write):
@@ -29,8 +26,9 @@ async def mcp_client() -> AsyncGenerator[ClientSession]:
                 await asyncio.sleep(1.0)
                 yield session
     except RuntimeError as e:
-        if "cancel scope" in str(e):
-            # Skip teardown exception (openroad-mcp handles their own teardown)
+        # anyio emits a RuntimeError on cancel-scope teardown when the MCP
+        # server subprocess exits
+        if "cancel scope" in str(e).lower():
             pass
         else:
             raise