JJUG CCC 2014 fall 「私がTDD出来ないのはどう考えてもお前らが悪い！」～エンタープライズJava開発でのTDD適用の勘所～

1. #ccc_r11 私がTDD出来ないのはどう考えてもお前らが悪い！～エンタープライズJava開発におけるTDD適用の勘所～ 2014/11/15 大中浩行 #ccc_r11

3. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 TDD三原則 (Uncle Bob) • 失敗する単体テストのコードを書く前に、製品のコードを書いてはいけない • コンパイルが通り、適切に失敗する単体テストが通るまでは、次の単体テストを書いてはならない • 現在失敗する単体テストが通るまで、次の製品コードを書いてはならない「Clean Code アジャイルソフトウェア達人の技」

33. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 100点を目指さない。優先順位をつけるユニットテストの5つの規則(F.I.R.S.T) • 再現性がある (Repeatable) • 独立している(Independent) ※ほか、高速である(Fast)/自己検証可能(Self-Validating)/適時性(Timely) 「Clean Code アジャイルソフトウェア達人の技」

39. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 どうしてうまくいった？ • RESTfulのAPIを定義した上で、フロントエンドとサーバサイドでそれぞれ実装して、それを結合するというプロセスだったので、ユニットテストでサーバーサイドの動作を確認するというアプローチが自然だった • Controller層とService層もModuleを分割していたので、同じ事が言える

49. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 • Java開発の場合は、スローテストの原因になるのは、コンテナの立ち上げ • ユニットテストの望ましい姿(独立性)からは外れますが、Spring Frameworkの SpringJUnit4ClassRunnnerはContainerの Contextをキャッシュする実装になっているので、初期化コストはおさえられる。

52. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 XMLだけど自動生成すれば問題ないよねっ • 本当はYAMLなりJSONにしたかったのですが、 DBUnitとの相性との関係で。 • テストデータ作成時は開発者のテストデータに一度投入した後、mavenプラグインでエクスポートする方法を採用 • ただ、mavenでエクスポートするのはあまり使われてないみたい..

59. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 Spring によるRESTful APIのテスト mockMvc.perform(get("/company/fieldnotes.jp/users/hiroy uki@fieldnotes.jp")) .andExpect(jsonPath("$.id",is("hiroyuki@fieldnotes.jp"))) .andExpect(jsonPath("$.firstName",is("Hiroyuki"))) .andExpect(jsonPath("$.lastName",is("Ohnaka"))) .andExpect(status().isOk());

60. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 モックした場合のコンテナー汚染問題 • Dependency Injectionしたコンポーネント内のオブジェクトをモックした場合、Springのコンテナー内のオブジェクトが汚染される問題 • モックを使うテストケースに@DirtyContextをつければ回避できるが、テストが目に見えて遅くなるため @After のメソッドの中でオブジェクトを差し戻すように修正

70. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 describe service('jboss') do it { should be_enabled } it { should be_running } end describe command("wget -O - http://localhost:8080/service/status") do it { should return_exit_status 0 } end describe "setting.propetiesがデプロイされていること" do describe command("diff #{RESOURCES}/setting.properties #{JBOSS_HOME}/properties/setting.properties") do it { should return_exit_status 0 } end end

73. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka 荒みきったコード疲弊しきった現場爆弾処理のようなリリース http://www.flickr.com/photos/okinawa-soba/2951808529/ http://www.flickr.com/photos/22719239@N04/2246462044/ http://ja.wikipedia.org/wiki/%E3%83%95%E3%82%A1%E3%82%A4%E3%83%AB:Bomb_neutralizing_EOD_9.jpg

75. #ccc_r11 Copyright 2014 Hiroyuki Ohnaka #ccc_r11 もうウンザリです。何も改善できません。「しかし、私たちにはプログラミングをする本当の理由があるはずです。」「もし、自分や同僚がプログラミングを楽しめるなら、どんなシステムに取り組んでいるかは重要ではありません。そのシステムに対してきちんとした仕事ができるはずで、そうでなければ待っているのは落胆です。そうなってしまっては何の楽しみもありませんし、私たちはそんな目に遭うべきではありません。」「レガシーコード改善ガイド」

https://twitter.com	149
http://chichi1091.hatenablog.jp	10
https://www.chatwork.com	1

SlideShare for iOS

Views

Actions

Embeds 160

Report content

Transcript