Only enable test coverage for non-MSVC chains

Author: pcolby

.github/workflows/build.yaml

@@ -514,16 +514,16 @@ jobs:
              ${{ (startsWith(matrix.qt, '5.10.') || startsWith(matrix.qt, '5.11.')) && 'qt' ||
                  (startsWith(matrix.qt, '5') && 'qtcore qtbluetooth' || 'qtcore') }}
           doc-modules: ${{ startsWith(matrix.qt, '6') && 'qtbluetooth' || '' }}
-      - run: gcov.exe --version
-      - run: C:\msys64\usr\bin\pacman --sync --noconfirm --verbose mingw-w64-x86_64-lcov
-      - run: dir C:\msys64 /s
       - name: Upload aqtinstall log file
         if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: aqtinstall-log-win-${{ matrix.arch }}-${{ matrix.toolchain }}-${{ matrix.qt }}
           path: aqtinstall.log
           if-no-files-found: error
+      - name: Install LCOV
+        if: toolchain != 'msvc'
+        run: C:\msys64\usr\bin\pacman --sync --noconfirm --verbose mingw-w64-x86_64-lcov
       - name: Configure cross-compilation
         if: matrix.arch == 'arm64'
         shell: bash
@@ -539,11 +539,12 @@ jobs:
       - name: Build
         env:
           CMAKE_MESSAGE_LOG_LEVEL: VERBOSE
+          ENABLE_COVERAGE: ${{ matrix.os == 'msvc' && 'false' || 'true' }}
           PROJECT_BUILD_ID: ${{ github.run_number }}.win.${{ matrix.arch }}.${{ matrix.toolchain }}.qt-${{ matrix.qt }}
         run: |
           cmake -D CMAKE_BUILD_TYPE=Release ^
                 -D CODECOV_LCOV=C:\msys64\mingw64\bin\lcov ^
-                -D ENABLE_COVERAGE=true ^
+                -D ENABLE_COVERAGE=%ENABLE_COVERAGE% ^
                 -D QT_HOST_PATH=%qtHostPath% ^
                 -D QT_INSTALL_DOCS=%RUNNER_WORKSPACE%\Qt\Docs\Qt-${{ matrix.qt }} ^
                 -G "${{ matrix.generator }}" -S "%GITHUB_WORKSPACE%" -B "%RUNNER_TEMP%"
@@ -564,6 +565,7 @@ jobs:
         if: matrix.arch != 'arm64'
         run: ctest --output-on-failure --test-dir "%RUNNER_TEMP%" --verbose
       - name: Collate test coverage
+        if: matrix.toolchain == 'msvc'
         run: cmake --build "%RUNNER_TEMP%" --target coverage
       - name: Upload test results
         if: matrix.arch != 'arm64' && fromJSON(steps.post-build.outputs.tap)

test/CMakeLists.txt

@@ -42,8 +42,8 @@ if (ENABLE_COVERAGE AND CMAKE_CXX_COMPILER_ID MATCHES "Clang$")
 endif()
 
 if (ENABLE_COVERAGE AND CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
-  find_program(CODECOV_GCOV NAMES gcov gcov.exe)
-  find_program(CODECOV_LCOV NAMES lcov lcov.exe)
+  find_program(CODECOV_GCOV gcov)
+  find_program(CODECOV_LCOV lcov)
   if (CODECOV_LCOV)
     message(STATUS "Found gcov ${CODECOV_GCOV}")
     message(STATUS "Found lcov ${CODECOV_LCOV}")