Closed shikata1112 closed 1 year ago
Name | Link |
---|---|
Latest commit | 5c1bdb27db116cfad6509a1ff05f0adda8112066 |
Latest deploy log | https://app.netlify.com/sites/lovely-buttercream-3a7f81/deploys/64e37a4557da85000857b496 |
Deploy Preview | https://deploy-preview-113--lovely-buttercream-3a7f81.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
📝 うれっとさん @RouxRhett と一緒にレビューする
何を解決するのか(関連するissueへのリンク等)
取得できるJournalのレコードの値まで検証できる。
実装の内容や理由(コードからは読み取れない情報を書きましょう)
クエリで取得できるJournalの値が期待通りかを検証するため、期待する値をベタ下書きして 検証するようにした。
Journalのレコードを作成する際、journal1, journal2と冗長な書き方をしているがDRYであることよりも テストにおいては可読性が高く、正しく検証できていることを優先した。 Journalのid, user_idは自動発番される値のためベタ書きにしていない。
伊藤さんの参考記事
不安なところ・レビューしてもらいたいところ
検証できているかどうか