Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ログインページのテストコード作成と、actの記述方法を変更 #24

Merged
merged 3 commits into from
Oct 10, 2024

Conversation

kameiryohei
Copy link
Owner

@kameiryohei kameiryohei commented Oct 10, 2024

実装意図

  • テストの網羅性を向上させるために、/app/user/login/page.tsxのコンポーネントに対して単体テストを作成

実装詳細

  • テストした項目を以下に示す
    • 画面に必要な要素が表示されていることの確認テスト
    • メールアドレスとパスワードが未入力の時のテスト
    • 適切なメールアドレス,パスワードの形でないものが入力された時のテスト
    • 正しくログインができた時ののテスト

詰まった点

  • 正しくログインができた時のテストで、成功した時の処理でtoastを表示しているがtoastはDOM要素ではないため、要素の取得ができなかった。

  • モックして挙動を確認できるか試したが、難しそうだったので一旦「ログイン中・・・」という文字列が出てくることを確認としてテストを通過させるようにした。

    • しかし、テストとしては十分ではないと考えているため、アプリ側のロジックの変更などをしてログインしたことを確認した方が良いと感じた。
  • actに関しては以下の記事を参考

  • actでラップしなればならないというコンソールエラーが出るが、検索して簡単なアプローチでエラーを消すことができるものを見つけ、適応させた

  • 追記(2024/10/11)

    • 以下のサイトを参考にmockの書き方をやってみたがエラーでテストが通らなかった。
    • 具体的にNumber of calls: 0と表示され期待した値が返ってきていないことがわかる。
    • https://blog.shinki.net/posts/jest-next-router-mock

Copy link

vercel bot commented Oct 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
class-planner ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 10, 2024 1:39pm

@kameiryohei kameiryohei changed the title Create test login page ログインページのテストコード作成と、actの記述方法を変更 Oct 10, 2024
@kameiryohei kameiryohei merged commit 3b329c7 into main Oct 10, 2024
3 checks passed
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