Skip to content

Conversation

@ryota-murakami
Copy link
Contributor

Summary

  • Add Storybook stories for DeleteAccountDialog with 4 stories: Default, WithConfirmationTyped, PartialInput, Closed
  • Add E2E tests for account deletion flow: dialog display, button states, input validation, cancel flow
  • UI-only validation (no actual deletion — server action needs real session)

Closes #70

Test plan

  • pnpm test — Storybook stories pass
  • pnpm e2e — New E2E tests pass
  • Verify dialog renders in Storybook

Add Storybook stories and E2E tests for DeleteAccountDialog:
- 4 stories: Default, WithConfirmationTyped, PartialInput, Closed
- 5 E2E tests: dialog display, button states, input validation, cancel flow
@vercel
Copy link
Contributor

vercel bot commented Feb 12, 2026

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

Project Deployment Actions Updated (UTC)
gitbox Ready Ready Preview, Comment Feb 12, 2026 1:11pm

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Warning

Rate limit exceeded

@ryota-murakami has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 20 minutes and 12 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch test/issue-70-account-deletion-coverage

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@morph-subagents
Copy link

🤖 Morph Preview Test

Looks like you hit your rate limits!

Please upgrade your limits here, or wait a few minutes and try again.

If you need help, reach out to support@morphllm.com.


Automated testing by Morph

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.36%. Comparing base (1eb53df) to head (0630d36).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
- Coverage   74.46%   74.36%   -0.10%     
==========================================
  Files         120      121       +1     
  Lines        3881     3909      +28     
  Branches     1063     1035      -28     
==========================================
+ Hits         2890     2907      +17     
- Misses        968      979      +11     
  Partials       23       23              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link

🧪 E2E Coverage Report (Sharded: 12 parallel jobs)

Metric Coverage
Lines 94.38%
Functions 17.67%
Branches 16.88%
Statements 30.46%

📊 Full report available in workflow artifacts

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.

Add test coverage for account deletion flow

2 participants