Skip to content

fix(ci): split coverage step to avoid Linux hang #94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JerrettDavis merged 3 commits into from
Apr 29, 2026
Merged

fix(ci): split coverage step to avoid Linux hang #94

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8dd0970
fix(ci): add --no-build to test step to prevent rebuild race condition
Apr 25, 2026
bbab3c6
fix(ci): collect coverage per-project to fix solution-level test-host…
Apr 25, 2026
85b1650
fix(tests): quarantine Testcontainers Redis tests with integration trait
Apr 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
77 changes: 72 additions & 5 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,14 +45,48 @@ jobs:
- name: Build (Release)
run: dotnet build ExperimentFramework.slnx --configuration Release --no-restore /p:ContinuousIntegrationBuild=true /p:BuildInParallel=false

- name: Test with coverage
- name: Test (verify correctness)
run: |
dotnet test ExperimentFramework.slnx \
--configuration Release \
--no-build \
-p:ExcludeE2ETests=true \
--filter "Category!=docs-screenshot&Category!=integration" \
--collect:"XPlat Code Coverage" \
--settings tests/ExperimentFramework.Tests/coverage.runsettings
--logger "console;verbosity=minimal"

- name: Collect coverage (per-project to avoid test-host crash on large projects)
shell: bash
run: |
RUNSETTINGS="tests/ExperimentFramework.Tests/coverage.runsettings"
FILTER="Category!=docs-screenshot&Category!=integration"
for proj in \
tests/ExperimentFramework.Audit.Tests \
tests/ExperimentFramework.Configuration.Tests \
tests/ExperimentFramework.Dashboard.Api.Tests \
tests/ExperimentFramework.Dashboard.Tests \
tests/ExperimentFramework.Dashboard.UI.Tests \
tests/ExperimentFramework.DataPlane.Tests \
tests/ExperimentFramework.Diagnostics.Tests \
tests/ExperimentFramework.Generators.Tests \
tests/ExperimentFramework.Governance.Persistence.Redis.Tests \
tests/ExperimentFramework.Governance.Persistence.Sql.Tests \
tests/ExperimentFramework.Governance.Persistence.Tests \
tests/ExperimentFramework.Governance.Tests \
tests/ExperimentFramework.Metrics.Exporters.Tests \
tests/ExperimentFramework.Plugins.Generators.Tests \
tests/ExperimentFramework.Plugins.Tests \
tests/ExperimentFramework.Simulation.Tests \
tests/ExperimentFramework.StickyRouting.Tests \
tests/ExperimentFramework.Tests; do
echo "--- Coverage: $(basename $proj) ---"
dotnet test "$proj" \
--configuration Release \
--no-build \
--filter "$FILTER" \
--collect:"XPlat Code Coverage" \
--settings "$RUNSETTINGS" \
--logger "console;verbosity=quiet" || echo "WARNING: $proj exited non-zero"
done

- name: Install ReportGenerator
run: dotnet tool update -g dotnet-reportgenerator-globaltool
Expand Down Expand Up @@ -336,8 +370,41 @@ jobs:
--no-build \
--verbosity normal \
-p:ExcludeE2ETests=true \
--collect:"XPlat Code Coverage" \
--settings tests/ExperimentFramework.Tests/coverage.runsettings
--filter "Category!=docs-screenshot&Category!=integration"

- name: Collect coverage (Release, per-project)
shell: bash
run: |
RUNSETTINGS="tests/ExperimentFramework.Tests/coverage.runsettings"
FILTER="Category!=docs-screenshot&Category!=integration"
for proj in \
tests/ExperimentFramework.Audit.Tests \
tests/ExperimentFramework.Configuration.Tests \
tests/ExperimentFramework.Dashboard.Api.Tests \
tests/ExperimentFramework.Dashboard.Tests \
tests/ExperimentFramework.Dashboard.UI.Tests \
tests/ExperimentFramework.DataPlane.Tests \
tests/ExperimentFramework.Diagnostics.Tests \
tests/ExperimentFramework.Generators.Tests \
tests/ExperimentFramework.Governance.Persistence.Redis.Tests \
tests/ExperimentFramework.Governance.Persistence.Sql.Tests \
tests/ExperimentFramework.Governance.Persistence.Tests \
tests/ExperimentFramework.Governance.Tests \
tests/ExperimentFramework.Metrics.Exporters.Tests \
tests/ExperimentFramework.Plugins.Generators.Tests \
tests/ExperimentFramework.Plugins.Tests \
tests/ExperimentFramework.Simulation.Tests \
tests/ExperimentFramework.StickyRouting.Tests \
tests/ExperimentFramework.Tests; do
echo "--- Coverage: $(basename $proj) ---"
dotnet test "$proj" \
--configuration Release \
--no-build \
--filter "$FILTER" \
--collect:"XPlat Code Coverage" \
--settings "$RUNSETTINGS" \
--logger "console;verbosity=quiet" || echo "WARNING: $proj exited non-zero"
done

- name: Upload coverage to Codecov
uses: codecov/codecov-action@v6
Expand Down
Loading
Loading