solidjs / solid / 41145071754

78%

main: 89%

push

github

test(signals): make two timing-sensitive tests robust on loaded CI runners (#4516)

* test(signals): give the tree-shaking suite a 30 s timeout

Every test in treeshake.test.ts runs a full Vite build of a fixture. The
first one pays the bundler's cold start, and under the whole suite's load
on CI it has run past vitest's 5 s default (5,295 ms on #4510, and on
other PRs earlier today) without any assertion failing.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015hPxMkC3jg3Uivh65gvaf2

* test(signals): widen the attribution self-time gap to 40 ms

"measures self-time and aggregates costs by scope and root write"
ranks a 10 ms memo above a cheap effect by wall-clock self-time. On this
PR's CI run a scheduling pause inside the effect measured 17.9 ms against
the memo's 10.1 ms. A 40 ms spin keeps the ranking meaningful under load.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015hPxMkC3jg3Uivh65gvaf2

---------

Co-authored-by: Claude <noreply@anthropic.com>

790 of 1090 branches covered (72.48%)

Branch coverage included in aggregate %.

1126 of 1363 relevant lines covered (82.61%)

29.92 hits per line