yuki-snow1823 / diary

Diary of development with an engineer friend
https://diary-red.vercel.app
3 stars 0 forks source link

新規登録のテストを追加 #110

Closed yuki-snow1823 closed 1 year ago

yuki-snow1823 commented 1 year ago

何を解決するのか(関連するissueへのリンク等)

https://github.com/yuki-snow1823/diary/issues/57

実装の内容や理由(コードからは読み取れない情報を書きましょう)

まずはテストを追加して動作することを確認しました。 このPRがマージされた後、フロントエンドを作成します。

不安なところ・レビューしてもらいたいところ

FactoryBotを使用してbuildし(その時点ではレコードを記録しない)そのnameやemailを使うという方法もあります。 こんな感じ(spec/requests/users/auth/registrations_spec.rb を作成の箇所)です。

一方で、登録というのは適切な値をセットして、POSTリクエストを送る行為なので、FactoryBotのbuild云々などを極力無くしてテストしたかったため、普通にPOSTする感じにしています。

netlify[bot] commented 1 year ago

Deploy Preview for lovely-buttercream-3a7f81 canceled.

Name Link
Latest commit b82f5f28962886918c85467de87019ce7527b3d4
Latest deploy log https://app.netlify.com/sites/lovely-buttercream-3a7f81/deploys/64d75b8505b6a80008961848