Use separate CI workflow to overcome issue with Dorny

Author: abelbraaksma

.github/workflows/test-report.yaml

@@ -0,0 +1,31 @@
+name: 'Test Report'
+
+# See Dorny instructions for why we need a separate yaml for creating a test report
+# for public repositories that accept forks:
+# https://github.com/dorny/test-reporter#recommended-setup-for-public-repositories
+
+on:
+  workflow_run:
+    workflows: ['ci-test']               # runs after CI workflow
+    types:
+      - completed
+jobs:
+  test-report-release:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: dorny/test-reporter@v1
+      with:
+        artifact: test-results-release    # artifact name
+        name: Report release tests        # Name of the check run which will be created
+        path: '**/*.trx'                     # Path to test results (inside artifact .zip)
+        reporter: dotnet-trx              # Format of test results
+
+  test-report-debug:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: dorny/test-reporter@v1
+      with:
+        artifact: test-results-debug      # artifact name
+        name: Report debug tests          # Name of the check run which will be created
+        path: '**/*.trx'                     # Path to test results (inside artifact .zip)
+        reporter: dotnet-trx              # Format of test results

.github/workflows/test.yaml

@@ -2,14 +2,8 @@ name: ci-test
 
 on: [pull_request]
 
-permissions:
-  id-token: write
-  contents: read
-  checks: write
-
 jobs:
   test-release:
-    permissions: write-all
     name: Test Release Build
     runs-on: windows-latest
     steps:
@@ -24,17 +18,15 @@ jobs:
       # build it, test it
       - name: Run dotnet test - release
         run: ./build.cmd ci -release
-      - name: Publish test results - release
-        uses: dorny/test-reporter@v1
-        if: always()
+      - uses: actions/upload-artifact@v2  # upload test results
+        if: success() || failure()        # run this step even if previous step failed
         with:
-          name: Report release tests
+          name: test-results-release
           # this path glob pattern requires forward slashes!
           path: ./src/FSharp.Control.TaskSeq.Test/TestResults/test-results-release.trx
-          reporter: dotnet-trx
+
 
   test-debug:
-    permissions: write-all
     name: Test Debug Build
     runs-on: windows-latest
     steps:
@@ -49,11 +41,9 @@ jobs:
       # build it, test it
       - name: Run dotnet test - debug
         run: ./build.cmd ci -debug
-      - name: Publish test results - debug
-        uses: dorny/test-reporter@v1
-        if: always()
+      - uses: actions/upload-artifact@v2  # upload test results
+        if: success() || failure()        # run this step even if previous step failed
         with:
-          name: Report debug tests
+          name: test-results-debug
           # this path glob pattern requires forward slashes!
           path: ./src/FSharp.Control.TaskSeq.Test/TestResults/test-results-debug.trx
-          reporter: dotnet-trx

src/FSharp.Control.TaskSeq.sln

@@ -21,6 +21,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "workflows", "workflows", "{
 		..\.github\dependabot.yml = ..\.github\dependabot.yml
 		..\.github\workflows\main.yaml = ..\.github\workflows\main.yaml
 		..\.github\workflows\publish.yaml = ..\.github\workflows\publish.yaml
+		..\.github\workflows\test-report.yaml = ..\.github\workflows\test-report.yaml
 		..\.github\workflows\test.yaml = ..\.github\workflows\test.yaml
 	EndProjectSection
 EndProject