Workflow cleanup and removed duplicate steps

Open-CMSIS-Pack · Jul 30, 2023 · ef3952b · ef3952b
1 parent 7ac62d4
commit ef3952b
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -59,6 +59,7 @@ jobs:
           go-package: ./...
 
   test:
+    continue-on-error: true
     strategy:
       matrix:
         platform: [ubuntu-latest, windows-latest, macos-latest]
@@ -81,6 +82,10 @@ jobs:
         run: |
           mkdir -p build
           go test -v ./... > build/cbuildtests-${{ matrix.platform }}-amd64.txt
+
+      - name: Generate unit test report
+        if: success() || failure()
+        run: |
           go-junit-report -set-exit-code -in build/cbuildtests-${{ matrix.platform }}-amd64.txt -iocopy -out build/cbuildtests-${{ matrix.platform }}-amd64.xml
 
       - name: Install qemu (for Linux-Arm64)
@@ -99,14 +104,18 @@ jobs:
           go-junit-report -set-exit-code -in build/cbuildtests-${{ matrix.platform }}-arm64.txt -iocopy -out build/cbuildtests-${{ matrix.platform }}-arm64.xml
 
       - name: Archive unit test results
+        if: ${{ always() }}
         uses: actions/upload-artifact@v3
         with:
           name: unit-test-result-${{ matrix.platform }}
           path: ./build/cbuildtests-*.xml
           if-no-files-found: error
 
   publish-test-results:
-    if: github.event_name != 'release'
+    if: ${{ always() && github.event_name != 'release' }} 
+    permissions:
+      checks: write
+      pull-requests: write
     name: "Publish Tests Results"
     needs: [ test ]
     runs-on: ubuntu-latest
@@ -132,9 +141,12 @@ jobs:
       - name: publish test results
         uses: EnricoMi/publish-unit-test-result-action/composite@v2
         with:
-          commit: ${{ github.event.workflow_run.head_sha }}
+          files: "testreports/*.xml"
           report_individual_runs: true
-          junit_files: "testreports/*.xml"
+        # with:
+        #   commit: ${{ github.event.workflow_run.head_sha }}
+        #   report_individual_runs: true
+        #   junit_files: "testreports/*.xml"
 
   coverage:
     if: github.event_name != 'release'

diff --git a/pkg/utils/configs_test.go b/pkg/utils/configs_test.go
@@ -30,7 +30,7 @@ func TestGetInstallConfigs(t *testing.T) {
 	assert := assert.New(t)
 	t.Run("test get install configs with CMSIS_BUILD_ROOT", func(t *testing.T) {
 		err := os.Setenv("CMSIS_BUILD_ROOT", testRoot+"/run/bin")
-		assert.Nil(err)
+		assert.Error(err)
 		configs, err := GetInstallConfigs()
 		assert.Nil(err)
 		assert.NotEmpty(configs.BinPath)