『JUnit実践入門』写経・実践会 in 横浜 #4 でLTしたネタを、 TDD Boot Camp Tokyo 2013-03にあわてて焼きなおしてLTした際の資料。重複多いですが備忘の意味でアップ。

1. TDDでレガシーコードに立ち向かう
#tddbc Tokyo 2013/03/16
@PoohSunny

2. 自己紹介
【@PoohSunny】
・㈱ヒューマンテクノロジーズ
・エンジニア
・最近のミッションはもっぱら
「テストのないレガシーコードにテストで
立ち向かう」
・時として魔改造も厭わない黒魔術師

3. ことしのよめ
※あわてて追記

4. 今回のお話
実践！ レガシーコード改善ガイド！
※一部書籍の内容は意図的に無視します。

5. レガシーコード...
サンプル（※フィクションです）
public void legacyMethod(HttpServletRequest request){
String code = (String) request.getAttribute("code");
TargetData data = new TargetDataDAO.getTargetData(code);
Integer comppanyId = data.getCompanyId();
TargetCompanyData companyData = new TargetCompanyDAO.
getTargetCompanyData(comppanyId);
......
//以下1000行

6. レガシーコード...
サンプル（※フィクションです）
引数が多かったり
重かったり
public void legacyMethod(HttpServletRequest request){
String code = (String) request.getAttribute("code");
TargetData data = new TargetDataDAO.getTargetData(code);
Integer comppanyId = data.getCompanyId();
TargetCompanyData companyData = new TargetCompanyDAO.
getTargetCompanyData(comppanyId);
......
//以下1000行

7. レガシーコード...
サンプル（※フィクションです）
引数が多かったり
重かったり
public void legacyMethod(HttpServletRequest request){
DBアクセスが
密結合だったり
String code = (String) request.getAttribute("code");
TargetData data = new TargetDataDAO.getTargetData(code);
Integer comppanyId = data.getCompanyId();
TargetCompanyData companyData = new TargetCompanyDAO.
getTargetCompanyData(comppanyId);
......
//以下1000行

8. レガシーコード...
サンプル（※フィクションです）
引数が多かったり
重かったり
public void legacyMethod(HttpServletRequest request){
DBアクセスが
密結合だったり
String code = (String) request.getAttribute("code");
TargetData data = new TargetDataDAO.getTargetData(code);
Integer comppanyId = data.getCompanyId();
TargetCompanyData companyData = new TargetCompanyDAO.
長かったり
getTargetCompanyData(comppanyId);
......
//以下1000行

9. レガシーコードにTDD！
ハマルこと
クラスが重すぎて
そもそもテストが書けない
(Mock使うのもむずい)
先にテストを書くとか無理。

10. 対策１
スプラウトメソッド
不具合があった場合などに、その部分だけ
「メソッドの抽出」リファクタを実行
→そこにテストを書く
＜悩ましいポイント＞
・カバレッジが上がってこない
→開発者のモチベーションが上がりにくい
・既存動作の担保ができない
→開発者の「不安」の解消ができない

11. 対策２
Seleniumで外から攻める
まずはSeleniumでラップ
→しかるのちにユニットテストを書く！
＜困ったポイント＞
中がレガシー → 画面は沢山のパラメータ
・ブラックボックステストを安心して書けるようになる
までは途方もない時間

12. というわけで工夫してあげる
DBUnit（JAVA）を使い、
DBアクセスを実際にして
テストをしてあげればいい！

13. レガシーコード改善ガイドは
DBアクセスするテストを特に嫌う！
∵遅いから
→気にせずDBアクセスしてしまえ！
いまは速度は金で買う時代！！！

14. DBアクセスして幸せになれた
・DBにアクセスするので、特に不具合修正のときに
ちゃんと「正しくはこう動くんだよー！！」というテスト
を書けるようになった！
・しかも、一度テストを書いたところって、時間がた
つといい感じにバグってくれる
→みんな、レガシーコードでもTDDできるよ！！