Update cmake-multi-platform.yml

Author: orange-cpp

.github/workflows/cmake-multi-platform.yml

@@ -1,101 +1,96 @@
-name: Omath CI
+name: Omath CI (Arch Linux / macOS / Windows)
 
 on:
   push:
-    branches: [ main ]        # change if your default branch is not "main"
+    branches: [ main ]
   pull_request:
     branches: [ main ]
 
 concurrency:
   group: ci-${{ github.ref }}
   cancel-in-progress: true
 
-################################################################################
-# 1)  ARCH LINUX  – runs inside archlinux:latest container
-################################################################################
+##############################################################################
+# 1)  ARCH LINUX  –  Clang / Ninja  (runs in an official Arch container)
+##############################################################################
 jobs:
   arch-build-and-test:
-    runs-on: ubuntu-latest
-    container: archlinux:latest
     name: Arch Linux (Clang)
+    runs-on: ubuntu-latest          # host runner
+    container: archlinux:latest     # actual build runs inside this container
 
     steps:
-      # Install Git, Clang, CMake, Ninja, make, etc. *before* we checkout
-      - name: Install build tools
+      # --- Install tool-chain **before** checkout so Git is available --------
+      - name: Install basic tool-chain with pacman
         shell: bash
         run: |
           pacman -Sy --noconfirm archlinux-keyring
           pacman -Syu --noconfirm --needed \
                  git base-devel clang cmake ninja
 
-      # Normal checkout with submodules now works because Git is present
-      - uses: actions/checkout@v4
+      # --- Now we can safely clone, incl. sub-modules ------------------------
+      - name: Checkout repository (with sub-modules)
+        uses: actions/checkout@v4
         with:
           submodules: recursive
 
-      - name: Configure (CMake preset: linux-release)
+      # --- Configure via CMake preset ---------------------------------------
+      - name: Configure (cmake --preset)
         shell: bash
         run: cmake --preset linux-release -DOMATH_BUILD_TESTS=ON
 
+      # --- Build ------------------------------------------------------------
       - name: Build
         shell: bash
         run: cmake --build cmake-build/build/linux-release
 
-      - name: Run unit-tests from out/Release
+      # --- Test -------------------------------------------------------------
+      - name: Test
         shell: bash
-        run: |
-          ./out/Release/unit_tests
+        working-directory: cmake-build/build/linux-release
+        run: ctest --output-on-failure
+
 
-################################################################################
-# 2)  macOS  – latest Homebrew LLVM/Clang
-################################################################################
+##############################################################################
+# 2)  macOS  –  Clang / Ninja  (native runner)
+##############################################################################
   macos-build-and-test:
+    name: macOS (Clang)
     runs-on: macos-latest
-    name: macOS (latest Clang)
 
     steps:
-      - uses: actions/checkout@v4
+      - name: Checkout repository (with sub-modules)
+        uses: actions/checkout@v4
         with:
           submodules: recursive
 
       - name: Install Ninja
         uses: seanmiddleditch/gha-setup-ninja@v4
 
-      - name: Install latest LLVM/Clang with Homebrew
-        run: |
-          brew update
-          brew install llvm
-          echo "LLVM_PREFIX=$(brew --prefix llvm)" >> "$GITHUB_ENV"
-
-      - name: Configure (preset: darwin-release) using Homebrew Clang
+      - name: Configure (cmake --preset)
         shell: bash
-        env:
-          CC:  ${{ env.LLVM_PREFIX }}/bin/clang
-          CXX: ${{ env.LLVM_PREFIX }}/bin/clang++
-        run: |
-          cmake --preset darwin-release \
-                -DCMAKE_C_COMPILER="$CC" \
-                -DCMAKE_CXX_COMPILER="$CXX" \
-                -DOMATH_BUILD_TESTS=ON
+        run: cmake --preset darwin-release -DOMATH_BUILD_TESTS=ON
 
       - name: Build
         shell: bash
         run: cmake --build cmake-build/build/darwin-release
 
-      - name: Run unit-tests from out/Release
+      - name: Test
         shell: bash
-        run: |
-          ./out/Release/unit_tests
+        working-directory: cmake-build/build/darwin-release
+        run: ctest --output-on-failure
+
 
-################################################################################
-# 3)  WINDOWS  – MSVC
-################################################################################
+##############################################################################
+# 3)  Windows  –  MSVC / Ninja  (native runner)
+##############################################################################
   windows-build-and-test:
-    runs-on: windows-latest
     name: Windows (MSVC)
+    runs-on: windows-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - name: Checkout repository (with sub-modules)
+        uses: actions/checkout@v4
         with:
           submodules: recursive
 
@@ -105,15 +100,15 @@ jobs:
       - name: Set up MSVC developer command-prompt
         uses: ilammy/msvc-dev-cmd@v1
 
-      - name: Configure (preset: windows-release)
+      - name: Configure (cmake --preset)
         shell: bash
         run: cmake --preset windows-release -DOMATH_BUILD_TESTS=ON
 
       - name: Build
         shell: bash
         run: cmake --build cmake-build/build/windows-release
 
-      - name: Run unit-tests from out\\Release
+      - name: Test
         shell: bash
-        run: |
-          ./out/Release/unit_tests.exe
+        working-directory: cmake-build/build/windows-release
+        run: ctest --output-on-failure