Uh oh!
There was an error while loading. Please reload this page.
test(e2e): live summary-rollup spec + exclude live specs from default config - #1523
Merged
Conversation
… config Adds e2e/live/summary-rollup.spec.ts verifying server-side roll-up summary fields (showcase_project.total_estimate=SUM, task_count=COUNT) end-to-end against the real SQL driver via /api/v1/batch — SUM/COUNT on atomic create, recompute to 0 on child delete. Also adds testIgnore '**/live/**' to the default playwright.config.ts so the mocked CI run (vite preview :4173, no backend) skips the live specs, which require the real stack + the live config's auth global-setup. Run them with `pnpm test:e2e:live`. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The latest updates on your projects. Learn more about Vercel for GitHub. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
e2e/live/summary-rollup.spec.ts— verifies the new server-side roll-upsummaryfields (framework PR fix(plugin-list): translate ListView empty-state title and message #1610) end-to-end against the real SQL driver via/api/v1/batch:showcase_project.total_estimate(SUM) +task_count(COUNT) computed on atomic create, and recomputed to0when the child is deleted. 2 passing against the live stack.testIgnore: ['**/live/**']to the defaultplaywright.config.tsso the mocked CI run (vite preview on :4173, no backend) skips the live specs. Live specs need the real stack (backend :3000 + console :5180) and the live config's auth global-setup; run them withpnpm test:e2e:live. This also retroactively keeps the master-detail live spec (added in fix(master-detail): reliable submit + durable live e2e harness #1521) out of the CI run.Testing
pnpm test:e2e:live→ master-detail (2) + summary-rollup (2) all green against the live stack.npx playwright test --list(default) confirms live specs are excluded; smoke/console specs unaffected.Pairs with framework PR #1610 (server-side summary computation).
🤖 Generated with Claude Code