Skip to content

Add mobile navigation smoke coverage - #386

Open
kitWarse wants to merge 1 commit into
SO4-Markets:mainfrom
kitWarse:test/335-mobile-nav-routes
Open

Add mobile navigation smoke coverage#386
kitWarse wants to merge 1 commit into
SO4-Markets:mainfrom
kitWarse:test/335-mobile-nav-routes

Conversation

@kitWarse

Copy link
Copy Markdown

Summary

  • Added a dedicated mobile Playwright project for the mobile navigation smoke test.
  • Added e2e/mobile-nav.spec.ts to open the mobile menu, navigate from Trade to Pools, and assert the route content changes.
  • Stubbed external price/chart network calls so the smoke test stays focused on layout and navigation behavior.

Issue

Changes

  • Added mobile-chromium project using Playwright's Pixel 5 device profile.
  • Excluded the mobile-only spec from the existing desktop Chromium project.
  • Added stable role/text locator assertions for the mobile menu, route link, Trade content, and Pools content.

Validation

  • Not run locally per request.

Notes

  • Full CI/e2e validation was not attempted.
  • No dependency or lockfile changes were committed.

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

@kitWarse is attempting to deploy a commit to the Ijai's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

@kitWarse 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! 🚀

Learn more about application limits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Playwright: mobile viewport navigation smoke

1 participant