Lit-innosence / tus_yuurikai_system

MIT License
1 stars 0 forks source link

Refactor/mail-auth #12

Closed Dot-P closed 2 weeks ago

Dot-P commented 2 weeks ago

メール認証部分のリファクタ

概要

Lit-innosence commented 2 weeks ago

結合テストについて

token周りが不安なので、結合テストは行うべきだと思います。 その場合、tokenの品質を保証するため、メール送信周りの処理を省く形で結合テストを記述するべきであるかと。

Dot-P commented 2 weeks ago

メール送信周りを除きながらトークンのチェックをするのが非常に困難だと思っています。 具体的には、

  1. トークンがランダムであるのでテストを組みづらい(.eqなどが使えない)
  2. メール送信部分を取り除くと、テストカバレッジが大幅に減少する

つきましては、もう少し具体的な内容について説明をお願いしたいです。