refactor: reorganize tests into src/tests/integration directory

- Moved all test files from root tests/ to MCPForUnity/UnityMcpServer~/src/tests/integration/ for better organization
- Added conftest.py with telemetry and dependency stubs to simplify test setup
- Removed redundant path manipulation and module loading code from individual test files

Author: msanatan

MCPForUnity/UnityMcpServer~/src/pyproject.toml

@@ -37,4 +37,4 @@ py-modules = [
     "telemetry_decorator",
     "unity_connection"
 ]
-packages = ["tools", "resources", "registry"]
+packages = ["tools", "resources", "registry", "tests", "tests.integration"]

MCPForUnity/UnityMcpServer~/src/tests/__init__.py

@@ -0,0 +1,48 @@
+import os
+import sys
+import types
+
+# Ensure telemetry is disabled during test collection and execution to avoid
+# any background network or thread startup that could slow or block pytest.
+os.environ.setdefault("DISABLE_TELEMETRY", "true")
+os.environ.setdefault("UNITY_MCP_DISABLE_TELEMETRY", "true")
+os.environ.setdefault("MCP_DISABLE_TELEMETRY", "true")
+
+# NOTE: These tests are integration tests for the MCP server Python code.
+# They test tools, resources, and utilities without requiring Unity to be running.
+# Tests can now import directly from the parent package since they're inside src/
+# To run: cd MCPForUnity/UnityMcpServer~/src && uv run pytest tests/integration/ -v
+
+# Stub telemetry modules to avoid file I/O during import of tools package
+telemetry = types.ModuleType("telemetry")
+def _noop(*args, **kwargs):
+    pass
+class MilestoneType:
+    pass
+telemetry.record_resource_usage = _noop
+telemetry.record_tool_usage = _noop
+telemetry.record_milestone = _noop
+telemetry.MilestoneType = MilestoneType
+telemetry.get_package_version = lambda: "0.0.0"
+sys.modules.setdefault("telemetry", telemetry)
+
+telemetry_decorator = types.ModuleType("telemetry_decorator")
+def telemetry_tool(*dargs, **dkwargs):
+    def _wrap(fn):
+        return fn
+    return _wrap
+telemetry_decorator.telemetry_tool = telemetry_tool
+sys.modules.setdefault("telemetry_decorator", telemetry_decorator)
+
+# Stub fastmcp module (not mcp.server.fastmcp)
+fastmcp = types.ModuleType("fastmcp")
+
+class _DummyFastMCP:
+    pass
+
+class _DummyContext:
+    pass
+
+fastmcp.FastMCP = _DummyFastMCP
+fastmcp.Context = _DummyContext
+sys.modules.setdefault("fastmcp", fastmcp)

tests/__init__.py renamed to MCPForUnity/UnityMcpServer~/src/tests/integration/__init__.py

@@ -1,23 +1,4 @@
-import sys
-import pathlib
-import importlib.util
-
-
-ROOT = pathlib.Path(__file__).resolve().parents[1]
-SRC = ROOT / "MCPForUnity" / "UnityMcpServer~" / "src"
-sys.path.insert(0, str(SRC))
-
-
-def _load(path: pathlib.Path, name: str):
-    spec = importlib.util.spec_from_file_location(name, path)
-    mod = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(mod)
-    return mod
-
-
-manage_script = _load(SRC / "tools" / "manage_script.py", "manage_script_mod2")
-manage_script_edits = _load(
-    SRC / "tools" / "script_apply_edits.py", "script_apply_edits_mod2")
+from tests.integration.test_helpers import DummyContext
 
 
 class DummyMCP:
@@ -28,9 +9,6 @@ def deco(fn): self.tools[fn.__name__] = fn; return fn
         return deco
 
 
-from tests.test_helpers import DummyContext
-
-
 def setup_tools():
     mcp = DummyMCP()
     # Import the tools module to trigger decorator registration

MCPForUnity/UnityMcpServer~/src/tests/integration/conftest.py

@@ -1,53 +1,4 @@
-import sys
-import pathlib
-import importlib.util
-import types
-
-from tests.test_helpers import DummyContext
-
-
-ROOT = pathlib.Path(__file__).resolve().parents[1]
-SRC = ROOT / "MCPForUnity" / "UnityMcpServer~" / "src"
-sys.path.insert(0, str(SRC))
-
-# Stub telemetry modules to avoid file I/O during import of tools package
-telemetry = types.ModuleType("telemetry")
-def _noop(*args, **kwargs):
-    pass
-class MilestoneType:
-    pass
-telemetry.record_resource_usage = _noop
-telemetry.record_tool_usage = _noop
-telemetry.record_milestone = _noop
-telemetry.MilestoneType = MilestoneType
-telemetry.get_package_version = lambda: "0.0.0"
-sys.modules.setdefault("telemetry", telemetry)
-
-telemetry_decorator = types.ModuleType("telemetry_decorator")
-def telemetry_tool(*dargs, **dkwargs):
-    def _wrap(fn):
-        return fn
-    return _wrap
-telemetry_decorator.telemetry_tool = telemetry_tool
-sys.modules.setdefault("telemetry_decorator", telemetry_decorator)
-
-# stub mcp.server.fastmcp
-mcp_pkg = types.ModuleType("mcp")
-server_pkg = types.ModuleType("mcp.server")
-fastmcp_pkg = types.ModuleType("mcp.server.fastmcp")
-
-
-class _Dummy:
-    pass
-
-
-fastmcp_pkg.FastMCP = _Dummy
-fastmcp_pkg.Context = _Dummy
-server_pkg.fastmcp = fastmcp_pkg
-mcp_pkg.server = server_pkg
-sys.modules.setdefault("mcp", mcp_pkg)
-sys.modules.setdefault("mcp.server", server_pkg)
-sys.modules.setdefault("mcp.server.fastmcp", fastmcp_pkg)
+from tests.integration.test_helpers import DummyContext
 
 
 class DummyMCP:

tests/test_edit_normalization_and_noop.py renamed to MCPForUnity/UnityMcpServer~/src/tests/integration/test_edit_normalization_and_noop.py

@@ -1,36 +1,7 @@
-import sys
-import pathlib
-import importlib.util
-import types
 import asyncio
 import pytest
 
-from tests.test_helpers import DummyContext
-
-ROOT = pathlib.Path(__file__).resolve().parents[1]
-SRC = ROOT / "MCPForUnity" / "UnityMcpServer~" / "src"
-sys.path.insert(0, str(SRC))
-
-# Stub telemetry modules to avoid file I/O during import of tools package
-telemetry = types.ModuleType("telemetry")
-def _noop(*args, **kwargs):
-    pass
-class MilestoneType:
-    pass
-telemetry.record_resource_usage = _noop
-telemetry.record_tool_usage = _noop
-telemetry.record_milestone = _noop
-telemetry.MilestoneType = MilestoneType
-telemetry.get_package_version = lambda: "0.0.0"
-sys.modules.setdefault("telemetry", telemetry)
-
-telemetry_decorator = types.ModuleType("telemetry_decorator")
-def telemetry_tool(*dargs, **dkwargs):
-    def _wrap(fn):
-        return fn
-    return _wrap
-telemetry_decorator.telemetry_tool = telemetry_tool
-sys.modules.setdefault("telemetry_decorator", telemetry_decorator)
+from tests.integration.test_helpers import DummyContext
 
 
 class DummyMCP:

tests/test_edit_strict_and_warnings.py renamed to MCPForUnity/UnityMcpServer~/src/tests/integration/test_edit_strict_and_warnings.py

@@ -1,24 +1,4 @@
-import sys
-import pathlib
-import importlib.util
-
-
-ROOT = pathlib.Path(__file__).resolve().parents[1]
-SRC = ROOT / "MCPForUnity" / "UnityMcpServer~" / "src"
-sys.path.insert(0, str(SRC))
-
-
-def _load_module(path: pathlib.Path, name: str):
-    spec = importlib.util.spec_from_file_location(name, path)
-    if spec is None or spec.loader is None:
-        raise ImportError(f"Cannot load module {name} from {path}")
-    mod = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(mod)
-    return mod
-
-
-manage_script = _load_module(
-    SRC / "tools" / "manage_script.py", "manage_script_mod")
+from tests.integration.test_helpers import DummyContext
 
 
 class DummyMCP:
@@ -32,9 +12,6 @@ def deco(fn):
         return deco
 
 
-from tests.test_helpers import DummyContext
-
-
 def setup_tools():
     mcp = DummyMCP()
     # Import the tools module to trigger decorator registration

tests/test_find_in_file_minimal.py renamed to MCPForUnity/UnityMcpServer~/src/tests/integration/test_find_in_file_minimal.py

@@ -2,25 +2,9 @@
 Test the improved anchor matching logic.
 """
 
-import sys
-import pathlib
-import importlib.util
+import re
 
-# add server src to path and load modules
-ROOT = pathlib.Path(__file__).resolve().parents[1]
-SRC = ROOT / "MCPForUnity" / "UnityMcpServer~" / "src"
-sys.path.insert(0, str(SRC))
-
-
-def load_module(path, name):
-    spec = importlib.util.spec_from_file_location(name, path)
-    module = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(module)
-    return module
-
-
-script_apply_edits_module = load_module(
-    SRC / "tools" / "script_apply_edits.py", "script_apply_edits_module")
+import tools.script_apply_edits as script_apply_edits_module
 
 
 def test_improved_anchor_matching():
@@ -41,8 +25,6 @@ def test_improved_anchor_matching():
     }
 }'''
 
-    import re
-
     # Test the problematic anchor pattern
     anchor_pattern = r"\s*}\s*$"
     flags = re.MULTILINE