Separate test report collection from the main target (#5061)

jaegertracing · Dec 30, 2023 · 5082239 · 5082239
1 parent 84e7b69
commit 5082239
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 1 deletion.
diff --git a/.github/workflows/ci-unit-tests-go-tip.yml b/.github/workflows/ci-unit-tests-go-tip.yml
@@ -23,9 +23,17 @@ jobs:
     - name: Install Go Tip
       uses: ./.github/actions/setup-go-tip
 
+    - name: Install test deps
+      # even though the same target runs from test-ci, running it separately makes for cleaner log in GH workflow
+      run: make install-test-tools
+
     - name: Run unit tests
       run: make test-ci
 
+    - name: Prepare unit tests report
+      if: always()
+      run: make test-report
+
     - name: Publish Unit Test Summary 📑
       uses: EnricoMi/publish-unit-test-result-action@v2
       if: always()

diff --git a/.github/workflows/ci-unit-tests.yml b/.github/workflows/ci-unit-tests.yml
@@ -32,9 +32,17 @@ jobs:
         go-version: 1.21.x
         cache-dependency-path: ./go.sum
 
+    - name: Install test deps
+      # even though the same target runs from test-ci, running it separately makes for cleaner log in GH workflow
+      run: make install-test-tools
+
     - name: Run unit tests
       run: make test-ci
 
+    - name: Prepare unit tests report
+      if: always()
+      run: make test-report
+
     - name: Publish Unit Test Summary 📑
       uses: EnricoMi/publish-unit-test-result-action@v2
       if: always()

diff --git a/Makefile b/Makefile
@@ -453,7 +453,7 @@ install-ci: install-test-tools install-build-tools
 
 .PHONY: test-ci
 test-ci: GOTEST := $(GOTEST_QUIET) -json
-test-ci: install-test-tools build-examples cover test-report
+test-ci: install-test-tools build-examples cover
 
 .PHONY: test-report
 test-report: