try check-with-coverage

Author: aykut-bozkurt

.github/workflows/ci.yml

@@ -138,20 +138,20 @@ jobs:
           make check-format
           make check-lint
 
-      # - name: Run tests with coverage
-      #   if: ${{ env.PG_MAJOR }} == 17 || ${{ matrix.arch == 'x64' }}
-      #   run: |
-      #     make check-with-coverage
-
       - name: Run tests without coverage
-        if: ${{ env.PG_MAJOR }} != 17 || ${{ matrix.arch != 'x64' }}
+        if: ${{ env.PG_MAJOR != '17' || matrix.arch != 'x64' }}
         run: |
           make check
 
-      # - name: Upload coverage report to Codecov
-      #   if: ${{ env.PG_MAJOR }} == 17 && ${{ matrix.arch == 'x64' }}
-      #   uses: codecov/codecov-action@v4
-      #   with:
-      #     fail_ci_if_error: true
-      #     files: ./lcov.info
-      #     token: ${{ secrets.CODECOV_TOKEN }}
+      - name: Run tests with coverage
+        if: ${{ env.PG_MAJOR == '17' && matrix.arch == 'x64' }}
+        run: |
+          make check-with-coverage
+
+      - name: Upload coverage report to Codecov
+        if: ${{ env.PG_MAJOR == '17' && matrix.arch == 'x64' }}
+        uses: codecov/codecov-action@v4
+        with:
+          fail_ci_if_error: true
+          files: ./lcov.info
+          token: ${{ secrets.CODECOV_TOKEN }}

Makefile

@@ -47,11 +47,11 @@ check: build stop-containers start-containers
 	cargo pgrx test pg$(PG_MAJOR) --no-default-features
 
 check-with-coverage: init-pgrx stop-containers start-containers
-	cargo llvm-cov show-env --export-prefix > llvm-cov.env
-	. ./llvm-cov.env
-	cargo llvm-cov clean
-	cargo build --features "pg$(PG_MAJOR), pg_test" --no-default-features
-	cargo pgrx test pg$(PG_MAJOR) --no-default-features
+	cargo llvm-cov show-env --export-prefix > llvm-cov.env && \
+	. ./llvm-cov.env && \
+	cargo llvm-cov clean && \
+	cargo build --features "pg$(PG_MAJOR), pg_test" --no-default-features && \
+	cargo pgrx test pg$(PG_MAJOR) --no-default-features && \
 	cargo llvm-cov report --lcov > lcov.info
 
 check-s3: build stop-minio start-minio