Publish test results

fredimachado · May 28, 2024 · efd6396 · efd6396
1 parent ca31756
commit efd6396
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 3 deletions.
diff --git a/.github/workflows/admin-api.yml b/.github/workflows/admin-api.yml
@@ -59,7 +59,6 @@ jobs:
         --results-directory ${{github.workspace}}/TestResults --blame \
         --logger console --logger trx --logger html \
         --logger "GitHubActions;summary.includePassedTests=true;summary.includeSkippedTests=true" \
-        '/p:CollectCoverage=true;CoverletOutputFormat="json,lcov,cobertura";MergeWith=${{github.workspace}}/TestResults/coverage.json;CoverletOutput=${{github.workspace}}/TestResults/coverage' \
         -- RunConfiguration.CollectSourceInformation=true
     
     - name: Upload Test Results

diff --git a/.github/workflows/barista-api.yml b/.github/workflows/barista-api.yml
@@ -52,7 +52,22 @@ jobs:
       run: dotnet restore
 
     - name: Test
-      run: dotnet test --results-directory ./testresults --logger "trx;LogFileName=test_results.xml" /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:CoverletOutput=./testresults/coverage/ ../**.Tests/**.csproj
+      id: test
+      # Note that the space after the last double dash (--) is intentional
+      run: >
+        dotnet test ../**.Tests/**.csproj
+        --logger console --logger trx --logger html
+        --logger "GitHubActions;summary.includePassedTests=true;summary.includeSkippedTests=true"
+        --results-directory ${{github.workspace}}/TestResults --blame
+        -- 
+        RunConfiguration.CollectSourceInformation=true
+    
+    - name: Upload Test Results
+      if: (success() || steps.test.conclusion == 'failure')
+      uses: actions/upload-artifact@v4
+      with:
+        name: TestResults
+        path: ${{github.workspace}}/TestResults
 
     - name: Build
       run: dotnet build -c Release --no-restore /p:CopyLocalLockFileAssemblies=true

diff --git a/.github/workflows/cashier-api.yml b/.github/workflows/cashier-api.yml
@@ -58,7 +58,23 @@ jobs:
       run: dotnet build -c Release --no-restore /p:CopyLocalLockFileAssemblies=true
 
     - name: Test
-      run: dotnet test ../../../NCafe.sln --results-directory ./testresults --logger "trx;LogFileName=test_results.xml" /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:CoverletOutput=./testresults/coverage/
+      id: test
+      # Note that the space after the last double dash (--) is intentional
+      run: >
+        dotnet test ../../../NCafe.sln
+        --logger console --logger trx --logger html
+        --logger "GitHubActions;summary.includePassedTests=true;summary.includeSkippedTests=true"
+        --results-directory ${{github.workspace}}/TestResults --blame
+        '/p:CollectCoverage=true;CoverletOutputFormat="json,lcov,cobertura";MergeWith=${{github.workspace}}/TestResults/coverage.json;CoverletOutput=${{github.workspace}}/TestResults/coverage' \
+        -- 
+        RunConfiguration.CollectSourceInformation=true
+    
+    - name: Upload Test Results
+      if: (success() || steps.test.conclusion == 'failure')
+      uses: actions/upload-artifact@v4
+      with:
+        name: TestResults
+        path: ${{github.workspace}}/TestResults
 
     - name: Publish
       run: dotnet publish -c Release --no-restore --no-build -o ./output