Skip to content

test: update ResetPasswordForm tests for token-based flow#140

Closed
jakebromberg wants to merge 0 commit intomainfrom
fix/reset-password-form-test
Closed

test: update ResetPasswordForm tests for token-based flow#140
jakebromberg wants to merge 0 commit intomainfrom
fix/reset-password-form-test

Conversation

@jakebromberg
Copy link
Copy Markdown
Member

@jakebromberg jakebromberg commented Feb 3, 2026

Closes #241

Summary

Update tests to match current component behavior:

  • Component now takes token prop instead of username
  • Hidden input is token not username
  • No "Code" field - validation is via token presence
  • Add test for passwords not matching
  • Add test for missing token scenario

Test plan

  • All 11 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: 837f2e2
Status: ✅  Deploy successful!
Preview URL: https://88d7fb86.dj-site.pages.dev
Branch Preview URL: https://fix-reset-password-form-test.dj-site.pages.dev

View logs

@jakebromberg jakebromberg force-pushed the fix/reset-password-form-test branch 2 times, most recently from 0c3e746 to e6fcd0e Compare March 2, 2026 15:36
@jakebromberg jakebromberg force-pushed the fix/reset-password-form-test branch from e6fcd0e to 837f2e2 Compare March 9, 2026 01:30
@jakebromberg jakebromberg force-pushed the fix/reset-password-form-test branch from 837f2e2 to f5f6db1 Compare March 19, 2026 00:14
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.

ResetPasswordForm tests assume username-based flow instead of token-based

2 participants