fix: Remove deprecated FastMCP mount pattern from tests

sitbon · sitbon · commit b60e2dd827be · 2025-07-15T23:35:14.000-07:00
Comment out test code that used deprecated mount(server=client, as_proxy=True)
pattern which triggered DeprecationWarning. Production code already uses the
correct FastMCP.as_proxy() approach. Keep as commented documentation of what
not to do.
diff --git a/magg/test/test_mounting_real.py b/magg/test/test_mounting_real.py
@@ -54,17 +54,19 @@ def test_tool(message: str) -> str:
             except Exception as e:
                 print(f"✗ Direct client mount failed: {e}")
 
-            # Approach 2: Try with proxy flag
-            try:
-                transport = NoValidatePythonStdioTransport(
-                    script_path=str(server_file),
-                    cwd=tmpdir
-                )
-                client = Client(transport)
-                main_server.mount(server=client, prefix="test2", as_proxy=True)
-                print("✓ Client mount with as_proxy=True succeeded")
-            except Exception as e:
-                print(f"✗ Client mount with as_proxy=True failed: {e}")
+            # Approach 2: Try with proxy flag (deprecated approach - remove this test)
+            # This approach is deprecated and causes warnings
+            # Keeping as commented documentation of what NOT to do
+            # try:
+            #     transport = NoValidatePythonStdioTransport(
+            #         script_path=str(server_file),
+            #         cwd=tmpdir
+            #     )
+            #     client = Client(transport)
+            #     main_server.mount(server=client, prefix="test2", as_proxy=True)
+            #     print("✓ Client mount with as_proxy=True succeeded")
+            # except Exception as e:
+            #     print(f"✗ Client mount with as_proxy=True failed: {e}")
 
             # Approach 3: Try as_proxy (new way)
             try:
