Skip to content

test(e2e): Add settings page email display verification#908

Merged
timothyfroehlich merged 1 commit intomainfrom
test/settings-email-display-e2e
Feb 2, 2026
Merged

test(e2e): Add settings page email display verification#908
timothyfroehlich merged 1 commit intomainfrom
test/settings-email-display-e2e

Conversation

@timothyfroehlich
Copy link
Owner

Summary

  • Add E2E test verifying that users see their email address displayed in the settings page
  • Verify the email field is disabled (read-only) as expected

Test plan

  • New test should display user email in settings passes locally
  • Existing should update profile and handle cancel test still passes
  • pnpm run check passes

Closes beads issue PinPoint-up5


Generated with Claude Code

Verify that logged-in users see their email address in the settings page
profile form, and that the email field is disabled (read-only).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 2, 2026 03:42
@vercel
Copy link

vercel bot commented Feb 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pin-point Ready Ready Preview, Comment Feb 2, 2026 3:46am

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an E2E regression test to ensure the Settings > Profile form correctly displays the logged-in user’s email and that the email field is non-editable.

Changes:

  • Added a new Playwright E2E test asserting the email input’s value matches the seeded user.
  • Added an assertion that the email field is disabled (read-only) on the settings profile form.

@timothyfroehlich timothyfroehlich merged commit 52485a6 into main Feb 2, 2026
26 checks passed
@timothyfroehlich timothyfroehlich deleted the test/settings-email-display-e2e branch February 2, 2026 03:51
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.

1 participant