Fix test count

Author: nowsprinting

Editor/Tools/RunTestsTool.cs

@@ -136,8 +136,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",
@@ -169,6 +172,9 @@ public void RunFinished(ITestResultAdaptor result)
                     ["message"] = summary["message"].Value<string>(),
                     ["testCount"] = summary["testCount"],
                     ["passCount"] = summary["passCount"],
+                    ["failCount"] = summary["failCount"],
+                    ["skipCount"] = summary["skipCount"],
+                    ["inconclusiveCount"] = summary["inconclusiveCount"],
                     ["results"] = summary["results"]
                 });
             }

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)
       }