Fix Windows path escaping in regression tests

The regression tests were failing on Windows because file paths containing
backslashes weren't properly escaped when inserted into Python script strings.

This commit:
- Adds a shared test utility function escape_path_for_python() that converts
  backslashes to forward slashes before quoting paths
- Updates the regression tests to use this helper for all file paths
- Refactors test_stdio.py to use the same shared utility

This ensures Windows paths like C:\Users\... are converted to C:/Users/...
which work correctly in Python strings on all platforms.

Author: felixweinberger

tests/client/test_stdio.py

@@ -17,6 +17,7 @@
 from mcp.shared.exceptions import McpError
 from mcp.shared.message import SessionMessage
 from mcp.types import CONNECTION_CLOSED, JSONRPCMessage, JSONRPCRequest, JSONRPCResponse
+from tests.shared.test_win32_utils import escape_path_for_python
 
 # Timeout for cleanup of processes that ignore SIGTERM
 # This timeout ensures the test fails quickly if the cleanup logic doesn't have
@@ -249,12 +250,6 @@ class TestChildProcessCleanup:
     This is a fundamental difference between Windows and Unix process termination.
     """
 
-    @staticmethod
-    def _escape_path_for_python(path: str) -> str:
-        """Escape a file path for use in Python code strings."""
-        # Use forward slashes which work on all platforms and don't need escaping
-        return repr(path.replace("\\", "/"))
-
     @pytest.mark.anyio
     @pytest.mark.filterwarnings("ignore::ResourceWarning" if sys.platform == "win32" else "default")
     async def test_basic_child_process_cleanup(self):
@@ -280,13 +275,13 @@ async def test_basic_child_process_cleanup(self):
                 import os
 
                 # Mark that parent started
-                with open({self._escape_path_for_python(parent_marker)}, 'w') as f:
+                with open({escape_path_for_python(parent_marker)}, 'w') as f:
                     f.write('parent started\\n')
 
                 # Child script that writes continuously
                 child_script = f'''
                 import time
-                with open({self._escape_path_for_python(marker_file)}, 'a') as f:
+                with open({escape_path_for_python(marker_file)}, 'a') as f:
                     while True:
                         f.write(f"{time.time()}")
                         f.flush()
@@ -335,9 +330,9 @@ async def test_basic_child_process_cleanup(self):
                 final_size = os.path.getsize(marker_file)
 
                 print(f"After cleanup: file size {size_after_cleanup} -> {final_size}")
-                assert final_size == size_after_cleanup, (
-                    f"Child process still running! File grew by {final_size - size_after_cleanup} bytes"
-                )
+                assert (
+                    final_size == size_after_cleanup
+                ), f"Child process still running! File grew by {final_size - size_after_cleanup} bytes"
 
             print("SUCCESS: Child process was properly terminated")
 
@@ -381,7 +376,7 @@ async def test_nested_process_tree(self):
 
                 # Grandchild just writes to file
                 grandchild_script = \"\"\"import time
-                with open({self._escape_path_for_python(grandchild_file)}, 'a') as f:
+                with open({escape_path_for_python(grandchild_file)}, 'a') as f:
                     while True:
                         f.write(f"gc {{time.time()}}")
                         f.flush()
@@ -391,7 +386,7 @@ async def test_nested_process_tree(self):
                 subprocess.Popen([sys.executable, '-c', grandchild_script])
 
                 # Child writes to its file
-                with open({self._escape_path_for_python(child_file)}, 'a') as f:
+                with open({escape_path_for_python(child_file)}, 'a') as f:
                     while True:
                         f.write(f"c {time.time()}")
                         f.flush()
@@ -401,7 +396,7 @@ async def test_nested_process_tree(self):
                 subprocess.Popen([sys.executable, '-c', child_script])
 
                 # Parent writes to its file
-                with open({self._escape_path_for_python(parent_file)}, 'a') as f:
+                with open({escape_path_for_python(parent_file)}, 'a') as f:
                     while True:
                         f.write(f"p {time.time()}")
                         f.flush()
@@ -470,7 +465,7 @@ async def test_early_parent_exit(self):
 
                 # Child that continues running
                 child_script = f'''import time
-                with open({self._escape_path_for_python(marker_file)}, 'a') as f:
+                with open({escape_path_for_python(marker_file)}, 'a') as f:
                     while True:
                         f.write(f"child {time.time()}")
                         f.flush()

tests/issues/test_1027_win_unreachable_cleanup.py

@@ -19,6 +19,7 @@
 
 from mcp import ClientSession, StdioServerParameters
 from mcp.client.stdio import _create_platform_compatible_process, stdio_client
+from tests.shared.test_win32_utils import escape_path_for_python
 
 
 @pytest.mark.anyio
@@ -53,8 +54,8 @@ async def test_lifespan_cleanup_executed():
         from contextlib import asynccontextmanager
         from mcp.server.fastmcp import FastMCP
         
-        STARTUP_MARKER = {repr(startup_marker)}
-        CLEANUP_MARKER = {repr(cleanup_marker)}
+        STARTUP_MARKER = {escape_path_for_python(startup_marker)}
+        CLEANUP_MARKER = {escape_path_for_python(cleanup_marker)}
         
         @asynccontextmanager
         async def lifespan(server):
@@ -159,8 +160,8 @@ async def test_stdin_close_triggers_cleanup():
         from contextlib import asynccontextmanager
         from mcp.server.fastmcp import FastMCP
         
-        STARTUP_MARKER = {repr(startup_marker)}
-        CLEANUP_MARKER = {repr(cleanup_marker)}
+        STARTUP_MARKER = {escape_path_for_python(startup_marker)}
+        CLEANUP_MARKER = {escape_path_for_python(cleanup_marker)}
         
         @asynccontextmanager
         async def lifespan(server):

tests/shared/test_win32_utils.py

@@ -0,0 +1,10 @@
+"""Windows-specific test utilities."""
+
+
+def escape_path_for_python(path: str) -> str:
+    """Escape a file path for use in Python code strings.
+
+    Converts backslashes to forward slashes which work on all platforms
+    and don't need escaping in Python strings.
+    """
+    return repr(path.replace("\\", "/"))