test(e2e): extend visual-regression, performance, mobile, and user-journeys suites - #1433
Open
scriptnovaa wants to merge 1 commit into
Open
Conversation
…ys suites Closes solutions-plug#1420, solutions-plug#1421, solutions-plug#1422, solutions-plug#1423 - visual-regression.spec.ts: wait for document.fonts.ready before every full-page screenshot to absorb font-loading timing flakiness across local/CI/Docker; add Statistics section baselines (loaded + loading state) as the one real data-fetching page live today. - performance.spec.ts: add LCP/TTI budget assertions (env-configurable) for the statistics data page, seeded with a realistic non-trivial dataset. - mobile.spec.ts: fix touch-target assertions to the WCAG 2.5.5 44x44px minimum (was checking height >= 40 only) at mobile viewport; add a feature-detected mobile nav toggle test; add a mobile browse -> connect wallet -> bet journey (wallet/bet legs mocked at the API level, browse through the real UI). - user-journeys.spec.ts: add a full cross-epic journey (browse -> connect wallet -> bet -> view statistics -> subscribe newsletter) that asserts state from the bet step is reflected in both the statistics section and the user's bet history, not just that each step passes in isolation. Markets list/detail pages and wallet-connect/betting UI (solutions-plug#56-solutions-plug#58, solutions-plug#68, solutions-plug#76) haven't shipped yet, so those legs are covered at the API-mock level following the existing pattern in this suite (Betting Flow, market-creation spec); the new describe blocks are where real-UI coverage gets added once those routes exist.
|
@scriptnovaa Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Closes #1420
Closes #1421
Closes #1422
Closes #1423
Summary
Extends the four e2e suites recreated for the frontend rebuild to cover what's live today, and sets up the seams for what isn't yet:
document.fonts.readybefore every full-page screenshot (fixes the font-loading timing flakiness called out in the issue); add Statistics-section baselines (loaded + loading state) — the one real data-fetching page live today, and the natural spot to add markets list/detail baselines once those routes ship.Note on scope
Markets list/detail pages and wallet-connect/betting UI (#56–#58, #68, #76) haven't landed in this repo yet — only the landing page and the Statistics section exist. Those legs are exercised at the API-mock level, following the same pattern already used by the existing "Betting Flow" and
market-creation.spec.tssuites. The new describe blocks are where real-UI coverage gets added once those routes ship.Testing
Not run in this environment (no
node_modules/browsers installed); per request, test execution was skipped.🤖 Generated with Claude Code