ci: test

Author: nicoecheza

.github/workflows/visual-regression.yml

@@ -65,7 +65,11 @@ jobs:
       # Used as the explorer-automation tests-ref. metaforge falls back to main
       # if the branch doesn't exist on explorer-automation's origin.
       TEST_REF: ${{ github.event.pull_request.head.ref || github.ref_name }}
-      TESTS_REF: ${{ github.event.pull_request.head.ref || github.ref_name }}
+      # TEMPORARY (smoke-test): the explorer-automation visual setup lives on
+      # feat/snapshots-comparison (PR #2) until merged to main. Once that PR
+      # lands, switch this back to ${{ github.event.pull_request.head.ref }}
+      # so per-branch test pairing works for real PRs.
+      TESTS_REF: feat/snapshots-comparison
       PR_NUMBER: ${{ github.event.pull_request.number }}
       # TEMPORARY (workflow smoke-test): hardcoded direct URL of a known-green
       # Unity Cloud Build for this branch, so we don't block on the in-progress
@@ -122,6 +126,12 @@ jobs:
           xattr -dr com.apple.quarantine /Applications/AltTesterDesktop.app || true
           ls /Applications/AltTesterDesktop.app/Contents/MacOS/AltTesterDesktop
 
+          # Allure CLI: metaforge generates the HTML report with it after each
+          # run. Pre-install globally so `mf explorer test --non-interactive`
+          # doesn't hit the "install allure?" prompt.
+          npm install -g allure-commandline
+          allure --version
+
           gh --version
 
       - name: Verify MetaForge access
@@ -155,8 +165,15 @@ jobs:
 
       - name: Configure AltTester license
         env:
-          ALTTESTER_LICENSE: ${{ secrets.ALTTESTER_LICENSE }}
-        run: mf alttester set-license "$ALTTESTER_LICENSE"
+          # NB: secret name uses British spelling (LICENCE) — set by devops.
+          ALTTESTER_LICENSE: ${{ secrets.ALTTESTER_LICENCE }}
+        run: |
+          set -euo pipefail
+          if [ -z "${ALTTESTER_LICENSE:-}" ]; then
+            echo "::error::Missing ALTTESTER_LICENCE secret. Add it under repo Settings → Secrets → Actions."
+            exit 1
+          fi
+          mf alttester set-license "$ALTTESTER_LICENSE"
 
       - name: Pre-install Explorer build
         env:
@@ -196,9 +213,16 @@ jobs:
           # Surface metaforge logs at info level so failures are diagnosable
           # without needing to download artifacts.
           MF_LOG_LEVEL: info
+          # metaforge clones explorer-automation via `gh repo clone`, which
+          # needs GH_TOKEN to authenticate against the private repo.
+          GH_TOKEN: ${{ secrets.REPOS_READ_ONLY_TOKEN }}
         run: |
           set -euo pipefail
-          mf explorer server start
+          # Pass --tests-ref to both commands so the managed clone of
+          # explorer-automation lands on the right branch up front.
+          # `server start` is what triggers the clone + _host build; if it
+          # defaults to main, the scenes/ workspace may be missing.
+          mf explorer server start --tests-ref "${TESTS_REF}"
           # Smoke-test: same direct URL as the install step above. Once
           # validated, swap back to `"${TEST_REF}"` for per-SHA resolution.
           mf explorer test "${BUILD_URL}" \