ci run with coverage report

Author: kevinjqliu

.github/workflows/python-ci.yml

@@ -64,8 +64,10 @@ jobs:
       run: make install-dependencies
     - name: Run linters
       run: make lint
-    - name: Run unit tests
-      run: make test
+    - name: Run unit tests with coverage
+      run: COVERAGE=1 make test
+    - name: Generate coverage report
+      run: make coverage-report
 
   integration-test:
     runs-on: ubuntu-22.04
@@ -80,26 +82,29 @@ jobs:
     - name: Install
       run: make install
 
-    - name: Run integration tests
-      run: make test-integration
+    - name: Run integration tests with coverage
+      run: COVERAGE=1 make test-integration
     - name: Show debug logs
       if: ${{ failure() }}
       run: docker compose -f dev/docker-compose.yml logs
 
-    - name: Run s3 integration tests
-      run: make test-s3
+    - name: Run s3 integration tests with coverage
+      run: COVERAGE=1 make test-s3
     - name: Show debug logs
       if: ${{ failure() }}
       run: docker compose -f dev/docker-compose.yml logs
 
-    - name: Run adls integration tests
-      run: make test-adls
+    - name: Run adls integration tests with coverage
+      run: COVERAGE=1 make test-adls
     - name: Show debug logs
       if: ${{ failure() }}
       run: docker compose -f dev/docker-compose-azurite.yml logs
 
-    - name: Run gcs integration tests
-      run: make test-gcs
+    - name: Run gcs integration tests with coverage
+      run: COVERAGE=1 make test-gcs
     - name: Show debug logs
       if: ${{ failure() }}
       run: docker compose -f dev/docker-compose-gcs-server.yml logs
+
+    - name: Generate coverage report
+      run: make coverage-report

Makefile

@@ -116,7 +116,9 @@ test-gcs: ## Run tests marked with @pytest.mark.gcs
 	$(TEST_RUNNER) pytest tests/ -m gcs $(PYTEST_ARGS)
 
 test-coverage: COVERAGE=1
-test-coverage: test test-integration test-s3 test-adls test-gcs ## Run all tests with coverage
+test-coverage: test test-integration test-s3 test-adls test-gcs coverage-report ## Run all tests with coverage and report
+
+coverage-report: ## Combine and report coverage
 	poetry run coverage combine
 	poetry run coverage report -m --fail-under=90
 	poetry run coverage html