Consolidate coverage workflow into main test workflow and add framework parameter support- Remove separate test-coverage.yml workflow- Add conditional coverage steps to tests.yml workflow using coverage-check output- Add framework parameter support to Taskfile.yml- Coverage now runs only on Ubuntu x64 with .NET 8.0.x to optimize CI resources- Add .NET 9.0.x support to test workflows for C# language feature compatibility

Signed-off-by: jbrinkman <joe.brinkman@improving.com>

Author: jbrinkman

.github/workflows/test-coverage.yml

@@ -83,6 +83,10 @@ jobs:
               env:
                   DOTNET_INSTALL_DIR: ~/.dotnet
 
+            - name: Install Task for using Taskfile
+              run: |
+                  sh -c "$(curl --location https://taskfile.dev/install.sh)" -- -d -b /usr/local/bin
+
             - name: Install shared software dependencies
               uses: ./.github/workflows/install-shared-dependencies
               with:
@@ -109,6 +113,33 @@ jobs:
                   redis-server --save "" --daemonize "yes"
                   ./install_and_test.sh -no-tls -minimal -only-glide -data 1 -tasks 10 -csharp -dotnet-framework net${{ matrix.dotnet }}
 
+            - name: Check if coverage needed
+              id: coverage-check
+              run: |
+                  if [[ "${{ matrix.dotnet }}" == "8.0.x" && "${{ contains(matrix.host.RUNNER, 'ubuntu') }}" == "true" && "${{ matrix.host.ARCH }}" == "x64" ]]; then
+                    echo "coverage-enabled=true" >> $GITHUB_OUTPUT
+                  fi
+
+            - name: Build and run coverage
+              if: ${{ steps.coverage-check.outputs.coverage-enabled == 'true' }}
+              run: task coverage
+
+            - name: Upload coverage report artifact
+              if: ${{ steps.coverage-check.outputs.coverage-enabled == 'true' }}
+              uses: actions/upload-artifact@v4
+              with:
+                  name: coverage-report
+                  path: reports/
+                  retention-days: 30
+
+            - name: Upload coverage results artifact
+              if: ${{ steps.coverage-check.outputs.coverage-enabled == 'true' }}
+              uses: actions/upload-artifact@v4
+              with:
+                  name: test-results
+                  path: testresults/
+                  retention-days: 30
+
             - name: Upload test reports
               if: always()
               continue-on-error: true