Merge pull request #36 from microsoft/dornyFix

Fixes dorny report for external PRs.

.github/workflows/build.yml

@@ -54,13 +54,12 @@ jobs:
         CollectCoverage: true
         CoverletOutputFormat: cobertura
       run: dotnet test MSStore.CLI.sln --no-restore /p:Configuration=Release --logger trx & wait
-    - name: Display test results
-      uses: dorny/test-reporter@v1
-      if: always()
+    - name: Publish test results
+      if: ${{ !env.ACT }} && (success() || failure())
+      uses: actions/upload-artifact@v3
       with:
-        name: Test - Results - ${{ matrix.os }}
+        name: test-results-${{ matrix.os }}
         path: MSStore.CLI.UnitTests/TestResults/*.trx
-        reporter: dotnet-trx
     - name: Install ReportGenerator
       run: dotnet tool install --global dotnet-reportgenerator-globaltool --version 5.2.2
     - name: Merge code coverage files

.github/workflows/test-report.yml

@@ -0,0 +1,25 @@
+name: 'Test Report'
+on:
+  workflow_run:
+    workflows: ['CI']                     # runs after CI workflow
+    types:
+      - completed
+permissions:
+  contents: read
+  actions: read
+  checks: write
+jobs:
+  report:
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ windows-latest, ubuntu-latest, macos-latest ]
+    runs-on: ${{ matrix.os }}
+    steps:
+    - name: Display test results
+      uses: dorny/test-reporter@v1
+      with:
+        artifact: test-results-${{ matrix.os }}
+        name: Test - Results - ${{ matrix.os }}
+        path: '*.trx'
+        reporter: dotnet-trx