From e24b59e9395b6a0ee6e9438276872074a925ba55 Mon Sep 17 00:00:00 2001
From: spacek531 <spacek531@comcast.net>
Date: Thu, 2 Nov 2023 10:43:00 -0700
Subject: [PATCH] Upload macos-x64 artifacts before tests (#20936)

This allows for artifact upload even when tests fail.

Other targets already upload artifacts before tests.
---
 .github/workflows/ci.yml | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 946b83d55f..13054f3e45 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -191,14 +191,6 @@ jobs:
         run: |
           HOMEBREW_NO_ANALYTICS=1 brew install ninja pkg-config
           . scripts/setenv -q && build -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=on ${{ matrix.build_flags }}
-      - name: Run Tests
-        if: ${{matrix.run_tests}}
-        run: . scripts/setenv -q && run-tests
-      - name: Test Summary
-        uses: test-summary/action@v2
-        with:
-          paths: "artifacts/test-**.xml"
-        if: ${{matrix.run_tests}}
       - name: Build artifacts
         run: |
           . scripts/setenv
@@ -213,6 +205,14 @@ jobs:
           name: OpenRCT2-${{ runner.os }}-${{ matrix.arch }}-cmake
           path: artifacts/openrct2-macos.zip
           if-no-files-found: error
+      - name: Run Tests
+        if: ${{matrix.run_tests}}
+        run: . scripts/setenv -q && run-tests
+      - name: Test Summary
+        uses: test-summary/action@v2
+        with:
+          paths: "artifacts/test-**.xml"
+        if: ${{matrix.run_tests}}
   macos-universal:
     name: macOS universal app bundle
     runs-on: macos-latest