Merge branch 'main' of https://github.com/CoderGamester/mcp-unity

Author: CoderGamester

Editor/Tools/RunTestsTool.cs

@@ -114,6 +114,12 @@ public void TestStarted(ITestAdaptor test)
         // Called when a test finishes
         public void TestFinished(ITestResultAdaptor result)
         {
+            // Skip test suites (tests with children)
+            if (result.Test.HasChildren)
+            {
+                return;
+            }
+            
             _testResults.Add(new TestResult
             {
                 Name = result.Test.Name,
@@ -136,8 +142,11 @@ public void RunFinished(ITestResultAdaptor result)
             // Create test results summary
             var summary = new JObject
             {
-                ["testCount"] = _testResults.Count,
-                ["passCount"] = _testResults.FindAll(r => r.Passed).Count,
+                ["testCount"] = result.PassCount + result.FailCount + result.SkipCount + result.InconclusiveCount,
+                ["passCount"] = result.PassCount,
+                ["failCount"] = result.FailCount,
+                ["skipCount"] = result.SkipCount,
+                ["inconclusiveCount"] = result.InconclusiveCount,
                 ["duration"] = result.Duration,
                 ["success"] = result.ResultState == "Passed",
                 ["status"] = "completed",
@@ -166,7 +175,13 @@ public void RunFinished(ITestResultAdaptor result)
                 {
                     ["success"] = true,
                     ["type"] = "text",
-                    ["message"] = summary["message"].Value<string>()
+                    ["message"] = summary["message"].Value<string>(),
+                    ["testCount"] = summary["testCount"],
+                    ["passCount"] = summary["passCount"],
+                    ["failCount"] = summary["failCount"],
+                    ["skipCount"] = summary["skipCount"],
+                    ["inconclusiveCount"] = summary["inconclusiveCount"],
+                    ["results"] = summary["results"]
                 });
             }
             catch (Exception ex)

README-ja.md

@@ -244,6 +244,17 @@ MCP Unityサーバーをデバッグするには、以下の方法を使用で
 - メニュー項目が確認を必要とするかどうかを確認
 - メニュー項目が現在のコンテキストで利用可能であることを確認
 
+#### Play Modeテスト実行時に `Connection failed` エラー
+
+`run_tests` ツールの実行が次のエラーを返します：
+```
+Error:
+Connection failed: Unknown error
+```
+
+このエラーは、Play Modeに切り替わる際にドメインがリロードされるため、ブリッジ接続が失われることで発生します。  
+回避方法は、**Edit > Project Settings > Editor > "Enter Play Mode Settings"** で **Reload Domain** をオフにすることです。
+
 ## 貢献
 
 貢献は大歓迎です！詳細については[貢献ガイド](CONTRIBUTING.md)をお読みください。

README.md

@@ -327,6 +327,19 @@ Don't forget to shutdown the server with `Ctrl + C` before closing the terminal
 - Verify that the menu item is available in the current context
 </details>
 
+<details>
+<summary><span style="font-size: 1.1em; font-weight: bold;">Connection failed when running Play Mode tests</span></summary>
+
+The `run_tests` tool returns the following response:
+```
+Error:
+Connection failed: Unknown error
+```
+
+This error occurs because the bridge connection is lost when the domain reloads upon switching to Play Mode.  
+The workaround is to turn off **Reload Domain** in **Edit > Project Settings > Editor > "Enter Play Mode Settings"**.
+</details>
+
 ## Support & Feedback
 
 If you have any questions or need support, please open an [issue](https://github.com/CoderGamester/mcp-unity/issues) on this repository.

README_zh-CN.md

@@ -246,6 +246,17 @@ MCP Unity 通过将 Unity `Library/PackedCache` 文件夹添加到您的工作
 - 检查菜单项是否需要确认
 - 验证菜单项在当前上下文中是否可用
 
+#### 运行播放模式测试时连接失败
+
+`run_tests` 工具返回以下响应：
+```
+Error:
+Connection failed: Unknown error
+```
+
+发生此错误的原因是在切换到播放模式触发域重新加载时，桥接连接会丢失。  
+解决方法是在 **Edit > Project Settings > Editor > "Enter Play Mode Settings"** 中关闭 **Reload Domain**。
+
 ## 贡献
 
 欢迎贡献！请阅读我们的[贡献指南](CONTRIBUTING.md)以获取更多信息。

Server/build/tools/runTestsTool.js

@@ -57,6 +57,9 @@ async function toolHandler(mcpUnity, params) {
     const testResults = response.results || [];
     const testCount = response.testCount || 0;
     const passCount = response.passCount || 0;
+    const failCount = response.failCount || 0;
+    const inconclusiveCount = response.inconclusiveCount || 0;
+    const skipCount = response.skipCount || 0;
     // Format the result message
     let resultMessage = `${passCount}/${testCount} tests passed`;
     if (testCount > 0 && passCount < testCount) {
@@ -69,14 +72,16 @@ async function toolHandler(mcpUnity, params) {
         content: [
             {
                 type: 'text',
-                text: response.message || resultMessage
+                text: resultMessage || response.message
             },
             {
                 type: 'text',
                 text: JSON.stringify({
                     testCount,
                     passCount,
-                    failCount: testCount - passCount,
+                    failCount,
+                    inconclusiveCount,
+                    skipCount,
                     results: testResults
                 }, null, 2)
             }

Server/src/tools/runTestsTool.ts

@@ -75,6 +75,9 @@ async function toolHandler(mcpUnity: McpUnity, params: any): Promise<CallToolRes
   const testResults = response.results || [];
   const testCount = response.testCount || 0;
   const passCount = response.passCount || 0;
+  const failCount = response.failCount || 0;
+  const inconclusiveCount = response.inconclusiveCount || 0;
+  const skipCount = response.skipCount || 0;
 
   // Format the result message
   let resultMessage = `${passCount}/${testCount} tests passed`;
@@ -89,14 +92,16 @@ async function toolHandler(mcpUnity: McpUnity, params: any): Promise<CallToolRes
     content: [
       {
         type: 'text',
-        text: response.message || resultMessage
+        text: resultMessage || response.message
       },
       {
         type: 'text',
         text: JSON.stringify({
           testCount,
           passCount,
-          failCount: testCount - passCount,
+          failCount,
+          inconclusiveCount,
+          skipCount,
           results: testResults
         }, null, 2)
       }