chore: improve test workflow and coverage reports

.github/workflows/unit-test.yml → .github/workflows/pr-test.yml

@@ -1,14 +1,14 @@
-name: Unit Tests
+name: Pull Request Tests
 on:
-  push:
-    branches: [main]
   pull_request:
+    # TODO: remove it
+    branches: [main]
 
 permissions:
   contents: read
 
 jobs:
-  build-and-cache:
+  compile:
     strategy:
       fail-fast: false
     runs-on: ubuntu-latest
@@ -26,19 +26,20 @@ jobs:
           node-version: 18
       - name: Install
         run: npm ci
-      - name: Build
-        run: npm run compile
+      # TODO: check if compiling only the affected modules is correct
+      - name: Compile (Delta)
+        run: npm run compile:ci:affected
       - name: Upload Build Artifacts
         uses: actions/upload-artifact@v4
         with:
-          name: tests-build-cache-${{ github.run_number }}
+          name: compile-cache-${{ github.run_number }}
           path: .nx
           include-hidden-files: true
           if-no-files-found: error
           retention-days: 1
 
   unit-test:
-    needs: build-and-cache
+    needs: compile
     strategy:
       fail-fast: false
       matrix:
@@ -185,26 +186,27 @@ jobs:
       - name: Download Build Artifacts
         uses: actions/download-artifact@v4
         with:
-          name: tests-build-cache-${{ github.run_number }}
+          name: compile-cache-${{ github.run_number }}
           path: .nx
-      - name: Build
-        run: npm run compile
-      - name: Unit tests (Full)
-        if: matrix.code-coverage
-        run: npm run test
+      - name: Compile (Delta)
+        run: npm run compile:ci:affected
       - name: Unit tests (Delta)
-        if: ${{ !matrix.code-coverage }}
-        run: npm run test:ci:changed
-      - name: Report Coverage
-        if: ${{ matrix.code-coverage && !cancelled()}}
-        uses: codecov/codecov-action@v5
-        env:
-          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+        run: npm run test:ci:affected
+      - name: Test All Versions (Delta)
+        run: npm run test-all-versions:ci:affected
+      - name: Upload Test Artifacts
+        uses: actions/upload-artifact@v4
         with:
-          verbose: true
+          name: tests-coverage-cache-${{ github.run_number }}-${{ matrix.node }}
+          include-hidden-files: true
+          if-no-files-found: error
+          retention-days: 1
+          path: |
+            !node_modules
+            **/.nyc_output/**
 
   browser-test:
-    needs: build-and-cache
+    needs: compile
     strategy:
       fail-fast: false
       matrix:
@@ -226,7 +228,7 @@ jobs:
       - name: Download Build Artifacts
         uses: actions/download-artifact@v4
         with:
-          name: tests-build-cache-${{ github.run_number }}
+          name: compile-cache-${{ github.run_number }}
           path: .nx
       - name: Build
         run: npm run compile
@@ -238,3 +240,54 @@ jobs:
           CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
         with:
           verbose: true
+
+  test-coverage-report:
+    runs-on: ubuntu-latest
+    needs: unit-test
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 18
+      - name: Install
+        run: npm ci
+      # NOTE: keep this in sync with the node versions from `unit-test` job
+      - name: Download Test Artifacts (18)
+        uses: actions/download-artifact@v4
+        with:
+          name: tests-coverage-cache-${{ github.run_number }}-18
+          path: .
+      - name: Download Test Artifacts (18.19.0)
+        uses: actions/download-artifact@v4
+        with:
+          name: tests-coverage-cache-${{ github.run_number }}-18.19.0
+          path: .
+      - name: Download Test Artifacts (20)
+        uses: actions/download-artifact@v4
+        with:
+          name: tests-coverage-cache-${{ github.run_number }}-20
+          path: .
+      - name: Download Test Artifacts (20.6.0)
+        uses: actions/download-artifact@v4
+        with:
+          name: tests-coverage-cache-${{ github.run_number }}-20.6.0
+          path: .
+      - name: Download Test Artifacts (22)
+        uses: actions/download-artifact@v4
+        with:
+          name: tests-coverage-cache-${{ github.run_number }}-22
+          path: .
+      - name: Merge coverage (Delta)
+        run: npm run test-merge-coverage:ci:affected
+      - name: Check coverage
+        run: find . | grep coverage-final
+      - name: Report Coverage
+        uses: codecov/codecov-action@v5
+        env:
+          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+        with:
+          verbose: true
+          dry_run: true