テストコード文化を創る

サービス立ち上げから約2年間のテストコード (RSpec) の変遷

Transcript

1. テストコード文化を創る Ryo Yoneyama LiB, Inc.
2. “Geek Suit” Ryo Yoneyama 株式会社LiB（リブ）
4. なぜ、プログラミングはつらいのか？
5. プログラミングの悲しい事実 - コードを書いた分だけバグが出る - コードは触らないと腐っていく - 今は素晴らしいコードもいつか邪魔になる - 仕様変更は簡単だけど、同じ手軽さでコードは変わらない
6. テストを書いても変わらない悲しい事実 - テストを書いてもバグは出る - テストが書いてあってもコードは腐敗していく - テストが書いてあっても、素晴らしいコードを維持できるわけではない - テストがあっても仕様変更に伴うコードの改修は容易じゃない
7. テストはモニタリング手法であり、 コードの品質は設計と実装が決める
8. あなたは、なぜテストを書かないのか？ - テストを書く時間がない - 実装コードがダメだから、テストが書けない（書きづらい） - まだ仕様が曖昧だからテストが書けない - テストを書くより実装の方が楽しい - ちゃんと動いてるから良くね？
9. なぜ、テストを書くのか？
10. ビジネスが変わればコードも変わる ↓ 終わらないリファクタリングの始まり
11. 『リファクタリング』を 支えるテストを書こう
12. LiBzCAREER の変遷
13. Phase 1 : 初期ローンチ Phase 2 : ひとりで開発 Phase 3 : チームで開発
14. 初期ローンチ ; rake stats
15. 俺が書いたコードが 正しく動いているはずがない！ けど、テストを書く時間がない
16. Not RSpec but factory_girl
17. ！ ランダムにデータ生成 ！ ！
18. 正しいデータを ランダムに大量生成して ポチポチと手動テスト 一部Spec はありますが、 メインは手動テストでした・・・
19. [WARNING] たまにfail するテスト 放置すると テストがfail することに慣れてしまうので注意！
20. ひとりで開発 ; rake stats
21. Controllers Views Models Routes
22. Controller でrender_views を使う ！
23. Controller を叩けば 少ないコードで 広い範囲のコードが動く
24. そのテストコードは正しいのか？ - そもそもテスト駆動じゃない - ユニットテストじゃない - render_views しているのでテストの実効速度は遅い - 間接的にModel やView も動くがテストケースを網羅しているわけじゃない
25. コードの品質モニタリングのために テストを書いている。 細かいことは置いといて、 ある程度デグレに気づけたらOK! （ドヤッ
26. [WARNING] 実装変更に弱いテストコード - いろんなコードに依存してるため、ちょっと変えるとテストが落ちまくる - 実装コードがダメだから、テストコードの依存関係を取り除けない そのテストコード捨てましょう！ テストがなくても、サービスは動きます。
27. チームで開発 ; rake stats
28. Model のSpec を継ぎ足しながら、 テストの独立性を高めてます・・・ 紆余曲折あったけど、 『テストコードはあるのが当たり前』ということが チームの共通認識になったのは良かったかと。
29. RuboCop やBrakeman でコード品質の劣化を防ぐ
30. ビジネスにコードをフィットさせよう