Merge pull request #316 from osociety/improved-workflow

windows if then else fix

Author: git-elliot

.github/workflows/flutter_test_main.yml

@@ -101,6 +101,21 @@ jobs:
           sudo apt-get update
           sudo apt-get install -y ${{ matrix.packages }}
 
+      - name: Install lcov (Linux)
+        if: matrix.os == 'ubuntu'
+        run: |
+          sudo apt-get install -y lcov
+
+      - name: Install lcov (macOS)
+        if: matrix.os == 'macos'
+        run: |
+          brew install lcov
+
+      - name: Install lcov (Windows)
+        if: matrix.os == 'windows'
+        run: |
+          choco install lcov
+
       - name: Download dependencies
         run: flutter pub get
 
@@ -111,7 +126,12 @@ jobs:
         run: flutter test --coverage
 
       - name: Preserve unit test coverage
-        run: mv coverage/lcov.info coverage/unit_${{ matrix.os }}.lcov.info
+        run: |
+          if [ -f coverage/lcov.info ]; then
+            mv coverage/lcov.info coverage/unit_${{ matrix.os }}.lcov.info
+          else
+            echo "Unit coverage not produced; aborting." && exit 1
+          fi
 
       - name: Run integration tests - Linux
         if: matrix.os == 'ubuntu'
@@ -131,15 +151,30 @@ jobs:
         run: flutter test integration_test/app_test.dart --coverage -d windows
 
       - name: Preserve integration test coverage
-        run: mv coverage/lcov.info coverage/integration_${{ matrix.os }}.lcov.info
+        run: |
+          if [ -f coverage/lcov.info ]; then
+            mv coverage/lcov.info coverage/integration_${{ matrix.os }}.lcov.info
+          else
+            echo "No integration coverage produced; skipping integration coverage step."
+          fi
 
       - name: Combine coverage files
         run: |
           # Combine unit and integration coverage
           cat coverage/unit_${{ matrix.os }}.lcov.info coverage/integration_${{ matrix.os }}.lcov.info > coverage/lcov.info
 
       - name: Remove generated files from coverage
-        run: lcov --remove coverage/lcov.info '**/*.g.dart' 'lib/models/drift/*' -o coverage/lcov.info
+        run: |
+          # Check if lcov is available; if not, skip filtering
+          if command -v lcov &> /dev/null; then
+            lcov --remove coverage/lcov.info \
+              '**/*.g.dart' \
+              'lib/models/drift/*' \
+              -o coverage/lcov.info
+            echo "Coverage filtered successfully."
+          else
+            echo "lcov not found; skipping coverage filtering. Coverage report will include generated files."
+          fi
 
       - name: Upload combined coverage to CodeCov
         if: always()