fix: SceneView test asserts both environments; workflow checks failure count

- StatsSetSceneDebug_ValidMode test now asserts success when SceneView
  exists and correctly asserts failure when it doesn't (batch mode),
  instead of going inconclusive.
- Workflow now parses NUnit XML for failed=0 rather than trusting the
  runner exit code, which returns 2 for inconclusive tests (URP,
  ProBuilder, Volumes not installed in CI).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: dsarno

.github/workflows/unity-tests.yml

@@ -76,6 +76,7 @@ jobs:
         if: steps.detect.outputs.unity_ok == 'true'
         uses: game-ci/unity-test-runner@v4
         id: tests
+        continue-on-error: true
         env:
           UNITY_EMAIL: ${{ secrets.UNITY_EMAIL }}
           UNITY_PASSWORD: ${{ secrets.UNITY_PASSWORD }}
@@ -86,6 +87,25 @@ jobs:
           unityVersion: ${{ matrix.unityVersion }}
           testMode: ${{ matrix.testMode }}
 
+      - name: Check test results
+        if: steps.detect.outputs.unity_ok == 'true'
+        run: |
+          RESULTS_XML=$(find ${{ steps.tests.outputs.artifactsPath }} -name '*.xml' 2>/dev/null | head -1)
+          if [ -z "$RESULTS_XML" ]; then
+            echo "::error::No test results XML found — Unity may have crashed"
+            exit 1
+          fi
+          FAILED=$(grep -oP 'failed="\K[0-9]+' "$RESULTS_XML" | head -1)
+          PASSED=$(grep -oP 'passed="\K[0-9]+' "$RESULTS_XML" | head -1)
+          TOTAL=$(grep -oP 'total="\K[0-9]+' "$RESULTS_XML" | head -1)
+          INCONCLUSIVE=$(grep -oP 'inconclusive="\K[0-9]+' "$RESULTS_XML" | head -1)
+          SKIPPED=$(grep -oP 'skipped="\K[0-9]+' "$RESULTS_XML" | head -1)
+          echo "Results: $PASSED passed, $FAILED failed, $INCONCLUSIVE inconclusive, $SKIPPED skipped (total: $TOTAL)"
+          if [ "$FAILED" != "0" ]; then
+            echo "::error::$FAILED test(s) failed"
+            exit 1
+          fi
+
       - uses: actions/upload-artifact@v4
         if: always() && steps.detect.outputs.unity_ok == 'true'
         with:

TestProjects/UnityMCPTests/Assets/Tests/EditMode/Tools/ManageGraphicsTests.cs

@@ -586,13 +586,15 @@ public void StatsGetMemory_ReturnsMemoryInfo()
         [Test]
         public void StatsSetSceneDebug_ValidMode_Succeeds()
         {
-            Assume.That(_hasSceneView, "No SceneView in batch mode — skipping.");
             var result = ToJObject(ManageGraphics.HandleCommand(new JObject
             {
                 ["action"] = "stats_set_scene_debug",
                 ["mode"] = "Wireframe"
             }));
-            Assert.IsTrue(result.Value<bool>("success"), result.ToString());
+            if (_hasSceneView)
+                Assert.IsTrue(result.Value<bool>("success"), result.ToString());
+            else
+                Assert.IsFalse(result.Value<bool>("success"));
         }
 
         [Test]