Skip to content

test: fix UserPasswordForm forgot link test#141

Merged
jakebromberg merged 1 commit intomainfrom
fix/user-password-form-test
Apr 21, 2026
Merged

test: fix UserPasswordForm forgot link test#141
jakebromberg merged 1 commit intomainfrom
fix/user-password-form-test

Conversation

@jakebromberg
Copy link
Copy Markdown
Member

@jakebromberg jakebromberg commented Feb 3, 2026

Closes #242

Summary

The forgot link is not disabled based on username input - it's always
enabled so users can navigate to the forgot password flow. Updated test
to reflect the actual component behavior.

Test plan

  • All 8 tests pass

Made with Cursor

Note: E2E tests depend on #271 and WXYC/Backend-Service#223. E2E will pass once those merge.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Feb 3, 2026

Deploying wxyc-dj with  Cloudflare Pages  Cloudflare Pages

Latest commit: f11a42d
Status: ✅  Deploy successful!
Preview URL: https://cc06d6f0.dj-site.pages.dev
Branch Preview URL: https://fix-user-password-form-test.dj-site.pages.dev

View logs

@jakebromberg jakebromberg force-pushed the fix/user-password-form-test branch 2 times, most recently from e713a70 to 9bd1ca9 Compare March 2, 2026 15:36
@jakebromberg jakebromberg force-pushed the fix/user-password-form-test branch from 9bd1ca9 to 283723a Compare March 9, 2026 01:30
The forgot link is not disabled based on username input - it's always
enabled so users can navigate to the forgot password flow. Updated test
to reflect the actual component behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jakebromberg jakebromberg force-pushed the fix/user-password-form-test branch from 283723a to f11a42d Compare April 18, 2026 03:59
@jakebromberg jakebromberg marked this pull request as ready for review April 21, 2026 16:24
@jakebromberg jakebromberg merged commit 5932a6a into main Apr 21, 2026
5 checks passed
@jakebromberg jakebromberg deleted the fix/user-password-form-test branch April 21, 2026 16:24
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.

UserPasswordForm test incorrectly asserts forgot link is disabled

2 participants