unity-cli@v1.8.2

- add additional unity utp log handling

Author: StephenHodgson

.github/workflows/build-options.json

@@ -17,6 +17,14 @@
     "6000.1.*",
     "6000.2"
   ],
+  "tests": [
+    "CompilerWarnings",
+    "CompilerErrors",
+    "BuildWarnings",
+    "BuildErrors",
+    "PlaymodeTestsErrors",
+    "EditmodeTestsErrors"
+  ],
   "include": [
     {
       "os": "ubuntu-latest",

.github/workflows/unity-build.yml

@@ -132,12 +132,12 @@ jobs:
           PACKAGE_MANAGER_LOG_PATH=$(unity-cli package-manager-logs)
           LICENSING_CLIENT_LOG_PATH=$(unity-cli licensing-client-logs)
           LICENSING_AUDIT_LOG_PATH=$(unity-cli licensing-audit-logs)
-          
+
           echo "Hub Log Path: ${HUB_LOG_PATH}"
           echo "Package Manager Log Path: ${PACKAGE_MANAGER_LOG_PATH}"
           echo "Licensing Client Log Path: ${LICENSING_CLIENT_LOG_PATH}"
           echo "Licensing Audit Log Path: ${LICENSING_AUDIT_LOG_PATH}"
-          
+
           if [ ! -f "${HUB_LOG_PATH}" ]; then
             echo "::warning:: Hub log file does not exist at ${HUB_LOG_PATH}"
             # find all info-log.json files in ~/.config/unity3d/ - print their paths
@@ -151,18 +151,25 @@ jobs:
             find ~/.config/ -type f -exec echo "{}" \;
             echo "::warning:: Hub log file does not exist at any known location"
           fi
-          
+
           if [ ! -f "${PACKAGE_MANAGER_LOG_PATH}" ]; then
             echo "::warning::Package Manager log file does not exist at ${PACKAGE_MANAGER_LOG_PATH}"
           fi
-          
+
           if [ ! -f "${LICENSING_CLIENT_LOG_PATH}" ]; then
             echo "::error::Licensing Client log file does not exist at ${LICENSING_CLIENT_LOG_PATH}"
           fi
-          
+
           if [ ! -f "${LICENSING_AUDIT_LOG_PATH}" ]; then
             echo "::error::Licensing Audit log file does not exist at ${LICENSING_AUDIT_LOG_PATH}"
           fi
+      - name: Upload UTP logs
+        if: always()
+        uses: actions/upload-artifact@v6
+        with:
+          name: utp-logs-${{ matrix.name }}
+          path: '**/*-utp-json.log'
+          if-no-files-found: ignore
       - name: Return License
         if: always()
         run: unity-cli return-license --license personal

unity-tests/BuildErrors.cs

@@ -0,0 +1,20 @@
+using UnityEditor;
+using UnityEditor.Build;
+using UnityEditor.Build.Reporting;
+
+namespace UnityCli.UtpSamples
+{
+    /// <summary>
+    /// Forces the build pipeline to fail by throwing a BuildFailedException.
+    /// Place under an Editor folder when copying into a project.
+    /// </summary>
+    public class BuildErrors : IPreprocessBuildWithReport
+    {
+        public int callbackOrder => 0;
+
+        public void OnPreprocessBuild(BuildReport report)
+        {
+            throw new System.Exception("Intentional build failure for test matrix coverage.");
+        }
+    }
+}

unity-tests/BuildWarnings.cs

@@ -0,0 +1,20 @@
+using UnityEditor;
+using UnityEditor.Build;
+using UnityEditor.Build.Reporting;
+
+namespace UnityCli.UtpSamples
+{
+    /// <summary>
+    /// Emits a build-time warning via the build pipeline (no custom UTP JSON logging).
+    /// Place under an Editor folder when copying into a project.
+    /// </summary>
+    public class BuildWarnings : IPreprocessBuildWithReport
+    {
+        public int callbackOrder => 0;
+
+        public void OnPreprocessBuild(BuildReport report)
+        {
+            UnityEngine.Debug.LogWarning("Intentional build warning for test matrix coverage.");
+        }
+    }
+}

unity-tests/CompilerErrors.cs

@@ -0,0 +1,4 @@
+// Intentional compiler error for matrix scenario coverage.
+#error Intentional compiler error: CS1029
+
+// Note: file is kept minimal so it can be copied into a project to force a build failure.

unity-tests/CompilerWarnings.cs

@@ -0,0 +1,20 @@
+using UnityEngine;
+
+namespace UnityCli.UtpSamples
+{
+    /// <summary>
+    /// Introduces a benign compiler warning (unused variable) without emitting custom logs.
+    /// </summary>
+    public class CompilerWarnings : MonoBehaviour
+    {
+        private void Awake()
+        {
+            ObsoleteApi(); // CS0618: call to obsolete member
+        }
+
+        [System.Obsolete("Intentional warning for test matrix coverage", false)]
+        private static void ObsoleteApi()
+        {
+        }
+    }
+}

unity-tests/EditmodeTestsErrors.cs

@@ -0,0 +1,16 @@
+using NUnit.Framework;
+
+namespace UnityCli.UtpSamples
+{
+    /// <summary>
+    /// Editmode test that intentionally fails to produce real test failure output.
+    /// </summary>
+    public class EditmodeTestsErrors
+    {
+        [Test]
+        public void FailsEditmodeSuite()
+        {
+            Assert.Fail("Intentional editmode failure for test matrix coverage.");
+        }
+    }
+}

unity-tests/PlaymodeTestsErrors.cs

@@ -0,0 +1,19 @@
+using System.Collections;
+using NUnit.Framework;
+using UnityEngine.TestTools;
+
+namespace UnityCli.UtpSamples
+{
+    /// <summary>
+    /// Playmode test that intentionally fails to generate real test failure output.
+    /// </summary>
+    public class PlaymodeTestsErrors
+    {
+        [UnityTest]
+        public IEnumerator FailsPlaymodeSuite()
+        {
+            yield return null;
+            Assert.Fail("Intentional playmode failure for test matrix coverage.");
+        }
+    }
+}