Freegle / Iznik
72%
master: 70%

LAST BUILD BRANCH: feature/eee-identification
DEFAULT BRANCH: master
Repo Added
Files 1131
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH fix/queue-fatal-error-sentry
branch: fix/queue-fatal-error-sentry
CHANGE BRANCH
x

coverage: 71.648% (+0.03%) from 71.614%
5841

Pull #249

circleci

edwh 
fix(ci): revert PW_WORKERS and test-skipping logic causing Playwright failures

Root cause analysis (job #5815):
1. **PW_WORKERS 1→4 caused resource contention**: Increased parallelism from 1 to 4 on the CI cloud machine triggered 'Failed to fetch' errors in Playwright tests. Browser console logs show TypeError: Failed to fetch when fetching from API, indicating the server couldn't handle the concurrent load or network timeouts increased.
2. **Test-skipping logic caused false test coverage**: Changed from 'always run all suites' to selective skipping based on changed files. This would cause feature branch PRs to skip test suites (e.g., Playwright for Go-only changes), breaking Coveralls carryforward and showing false coverage decreases.

Fixes:
- Revert PW_WORKERS to 1 for cloud machines (safe value proven in production)
- Revert to always running all test suites on every branch (prevents Coveralls false positives)
- Self-hosted runner keeps its dedicated PW_WORKERS setting (if any)

The PR's actual code changes (fatal error handler in queue:background-tasks) remain unchanged and are unrelated to the test failures.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Pull Request #249: fix(batch): report PHP fatal errors (incl. OOM) to Sentry on shutdown

13356 of 20230 branches covered (66.02%)

Branch coverage included in aggregate %.

17 of 22 new or added lines in 1 file covered. (77.27%)

20 existing lines in 5 files now uncovered.

96359 of 132901 relevant lines covered (72.5%)

21.7 hits per line

Relevant lines Covered
132901 RELEVANT LINES 96359 COVERED LINES
21.7 HITS PER LINE
Source Files on fix/queue-fatal-error-sentry

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
5841 fix/queue-fatal-error-sentry fix(ci): revert PW_WORKERS and test-skipping logic causing Playwright failures Root cause analysis (job #5815): 1. **PW_WORKERS 1→4 caused resource contention**: Increased parallelism from 1 to 4 on the CI cloud machine triggered 'Failed to fetch... Pull #249 edwh circleci
71.65
5815 fix/queue-fatal-error-sentry test: extend test 4.1 timeout to 20 min (signUpViaHomepage is slow in CI) signUpViaHomepage goes through the full UI signup wizard — multiple form steps and page navigations each with 45-90s timeouts in CI. Under load this can consume most of the... Pull #249 edwh circleci
71.57
5796 fix/queue-fatal-error-sentry fix(ci): use PORT_FREEGLE_PROD_LOCAL/PORT_MODTOOLS_PROD_LOCAL vars in health check Hardcoded ports 3012/3013 never matched CI runner's actual ports (12012/12013), causing the 'Wait for production containers to be healthy' step to always hang unti... Pull #249 edwh circleci
71.61
5778 fix/queue-fatal-error-sentry fix(ci): use PORT_FREEGLE_PROD_LOCAL/PORT_MODTOOLS_PROD_LOCAL vars in health check Hardcoded ports 3012/3013 never matched CI runner's actual ports (12012/12013), causing the 'Wait for production containers to be healthy' step to always hang unti... Pull #249 edwh circleci
67.84
5755 fix/queue-fatal-error-sentry fix(batch): report PHP fatal errors (incl. OOM) to Sentry on shutdown PHP fatal errors bypass try/catch so the existing Throwable handler in ProcessBackgroundTasksCommand never fires. Registers a shutdown function with a 2 MB memory reserve that... Pull #249 edwh circleci
66.69
See All Builds (1006)