Improve coverage reporting

1. Improve make coverage to take integration test into account 2. Upload coverage report as an artifact of CI Signed-off-by: Yutaro Hayakawa <[email protected]>
YutaroHayakawa · May 26, 2024 · 64fe96a · 64fe96a
1 parent 5c52f3a
commit 64fe96a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
@@ -18,8 +18,11 @@ jobs:
       with:
         go-version-file: ./go.mod
 
-    - name: Run unit tests
-      run: go test -v
+    - name: Run tests
+      run: go test -v -exec "sudo" -coverpkg=./... -coverprofile=coverage.out ./...
 
-    - name: Run integration tests
-      run: go test -exec "sudo -E" -v ./integration_tests
+    - name: Upload coverage report
+      uses: actions/upload-artifact@v4
+      with:
+        name: coverage.html
+        path: coverage.html
diff --git a/Makefile b/Makefile
@@ -16,5 +16,5 @@ check-deepcopy:
 	git diff --exit-code zz_generated_deepcopy.go || echo "deepcopy is not up to date. Please commit the changes."; git diff zz_generated_deepcopy.go
 
 coverage:
-	go test -cover -coverprofile=coverage.out -v
+	go test -v -exec "sudo" -coverpkg=./... -coverprofile=coverage.out ./...
 	go tool cover -html=coverage.out -o coverage.html