exclude browser stress tests from default ci#118
Conversation
There was a problem hiding this comment.
Pull request overview
This PR reclassifies the hosted-browser 1 GiB Playwright “large flow” integration tests as an explicit stress lane and removes them from the default GitHub Actions CI paths to keep build-and-test/Release runtimes and stability aligned with historical expectations.
Changes:
- Tagged the two hosted-browser 1 GiB Playwright flows with
Category=BrowserStress. - Updated
build-and-testandReleaseworkflows to excludeCategory=BrowserStressby default. - Documented how to include/exclude
BrowserStresslocally and clarified CI intent in docs.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserWasmStorageIntegrationTests.cs | Marks the WASM-hosted 1 GiB flow as BrowserStress for selective execution. |
| Tests/ManagedCode.Storage.Tests/Storages/Browser/BrowserServerStorageIntegrationTests.cs | Marks the Server-hosted 1 GiB flow as BrowserStress for selective execution. |
| .github/workflows/ci.yml | Excludes BrowserStress tests from the default CI test run (and coverage collection). |
| .github/workflows/release.yml | Excludes BrowserStress tests from the Release workflow test run. |
| docs/Testing/strategy.md | Adds documentation for the BrowserStress category and how to run/skip it. |
| docs/Development/setup.md | Notes that CI excludes BrowserStress and how to run it explicitly. |
| AGENTS.md | Records the repo preference to avoid CI runtime regressions from long browser stress lanes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #118 +/- ##
==========================================
- Coverage 57.27% 48.55% -8.72%
==========================================
Files 133 153 +20
Lines 6540 7702 +1162
Branches 922 1029 +107
==========================================
- Hits 3746 3740 -6
- Misses 2362 3528 +1166
- Partials 432 434 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
Verification