ぼくらのかんがえたさいきょうのfactory_bot

Transcript

1. ZEALS Co.,Ltd ぼくらのかんがえたさいきょうの factory_bot RSpec テクノロジー開発部 takakuda
2. 前職：雛人形売り - Ruby - Rails ZEALS Rails Engineer takakuda ：takakuda : @kutaike1504
3. fanpの紹介
4. factory_bot
5. ZEALSのfactory_botの歴史
6. 今日話すこと - default時代 - 一発作成時代 - 今
7. default時代 普通にfactory_botを使っていた after(:create)などはほとんどない traitが少しあるくらい
8. letがめちゃくちゃ多くなる - databaseが結構正規化されている - テスト対象のデータを作成するのが大変 - 可読性が低く、無理やりテストを通すようになる （それって本当にテストなの？状態）
9. 無理やりデータを揃える
10. letが8個くらいつながってしまうと どんなテストデータが作成されているのかを 把握することが困難 テストデータの帳尻合わせるためにbeforeでupdateを 行う
11. テストカバレッジ98%(…すごい) …のにバグが発生し続ける -> テストで作成されるデータが間違ってるんじゃない？ -> まちがったデータで作成されたテストをひたすら通し ていた
12. 一発作成時代 - let書きまくり時代からの解放 - association, after(:create), traitsをフル活用 - 正しいテストデータを作成できるようにする - メンバーのRSpecを書く効率を上げたかった
13. factory_botでむすこを作 成すると、一番上のじい ちゃんまで作成されるよう に変更
14. 良かった点 - letの数が確実に減った(可読性 ↑) - before内でupdateを使うことがなくなる - テストデータが齟齬ることがなくなる
15. 実際にconsoleで見せます！
16. 悪かった点 - かなり工数かかる(1ヶ月 factory_botリファクタ) - メンテ、共有のコスト - 全部作成してしまうのでテストに関係のないデータま でも作成してしまう - 全部作成する必要があるのか？問題
17. 全部作らなければいけない理由 認証
18. 今
19. 認証を回避
20. テストのためにソースコードに変更を加えるようなことは しないほうが良い
21. 認証部分をstubしちゃう
22. factory_botで全部作る必要 がなくなる
23. shared_context テストにおける条件を共有する
24. - やりすぎfactory_botをやめる - 認証部分はstub - 複雑なテスト前提条件shared_context
25. DRYに書きすぎて読みづらく ならないように バランスを考えて読みやすさ を重視する
26. まとめ
27. これからも改善続けます…！ factory_botのベスト・プラクティ ス的なの持ってる方は是非教え てください！
29. Thank you!!