目的

• AppBrew Rails Best Practiceの内容を理解した上でmake-serverのコードを書いていけるようにする

• 守れていない場合にレビューで指摘しやすくする

やり方

• みんなで一緒に読んでいく
• 1人1項目ずつ順番に読んでいく

• 適宜深ぼったり議論したりする
• ここの部分とか守れてないよねとか
• こうするとよさそうとか
 

議事録

• 不要なSQLが発行していないかみるってどうやって？
• テストとか動かすと良いよ

• 驚きの少ないコードとは？
• Railsガイド嫁

• statusコードってどうする？
• 自分たちでルール決めれば世界的に正しい

• methodにParamsにママ渡さんほうがええわよといいつつ、ActiveRecordのmethodでよく渡すものはあるがええんか？
• ActiveRecordの実装として生えているParamsを受け取るやつ（newとかcreateとか)はそのまま使えばええよ

• ネストを深くしない
• 完全木を作らないとなるとどれをテストするか選ばないといけないのむずくね？
• これよむと完全解決するらしい https://www.amazon.co.jp/dp/4798130605
• めちゃくちゃおすすめではないらしい

• 文法規則
• 例が厳しそう
• context→どういうケースか、it→どうなるかくらいでいいんジャネという話

• テストメッセージとコードを対応させる
• subjectってどういう粒度？
• 例えば job のテストは performまで含む？含まない？
• narazaka: expectに突っ込むやつをsubjectにするくらいがちょうど良さそう
• 呼び出し系はitの頭でsubjectだけかけばいいくらい

• factory bot
• traitとかつかうと後の人が幸せになる
FactoryBotでtraitを使おう - Qiita
FactoryBotでテストデータを作成するに当たって trait を押さえるのは必須かなと思います。 RSpecをこれから始める方向けにこの記事を書きます。 一例としてTaskのファクトリを作成する場面を想定して trait を使ってみます。 次にspecファイルで通常のtask、 trait を活用したtaskの二種類を書いてみます。 これは通常の書き方。 これにより、statusが todo の通常のtaskが作成可能になる。 次に traitを活用したtask（※ 想定場面はステータスが完了のタスクを作成したい時 ） これにより、statusが done、completion_dateが Time.current.yesterday のtaskが作成できる。 この時 trait を使わないと以下のようにコードが長くなります。 しかし、 trait を使えば簡潔に書ける！ この記事で traitの便利さをわかって頂けたら幸いです。 以上です！！
https://qiita.com/ShotaroHirose59/items/7c64be350676120de9bd

• rspec-request_describerを使う
• https://github.com/r7kamura/rspec-request_describer#params

• request spec書くときはstatus codeをまず確認する
• 弊社内の基準に則っているか確認するためにもよさそう
 

• parsed_bodyを使う
• https://apidock.com/rails/ActionDispatch/TestResponse/parsed_body

• request specってどこまでtestみるといいんだっけ
• serializableの中で定義したfunctionとかあればそこが正しいかはチェックする
• ただのattributes はみない

• expectはまとめる
• わかる

• テストするときは複数件作れ
• すまん

• controller-specは避ける
• controller specはdeprecated

• system-spec書かない
• まぁカロリー高いしね
 

• 宣言の有無をテストしない
• 悪い例で書かれているmatcherはこれだけど。。。http://matchers.shoulda.io/docs/v4.3.0/Shoulda/Matchers/ActiveModel.html#validate_presence_of-instance_method

ネクストアクション

• ステータスコード、どういうケース何にする？

• Railsガイト読む

• 最適なテスト勉強会

• subjectの使い方のススメを聞く