JUnit4のassertThat()って便利なの？

Ryo Asai @ryoasai74

初歩的な疑問だけど、JUnitのassertThat()の形式でClassクラス同士を比較することはできないのかな？この場合はassertEquals()を使う必要があるのだろうか？

🍻 @kompiro

@ryoasai74 assertThat(actual.class, is(expected.class)では不十分ですか？

Ryo Asai @ryoasai74

@kompiro is()メソッドでタイプトークンをとるメソッドがオーバーロードされていて、この比較だと、第一パラメーターが第二パラメーターで指定したクラスのインスタンスであることが判定されてしまうのですよね。 http://bit.ly/mhMie7

Ryo Asai @ryoasai74

英語ネイティブ的にはis(String.class)はis(instanceOf(String.class))と同じ意味だと解釈したいのでしょうが、これだとクラス自身が比較できないな。static importも面倒だし、個人的にはあまり便利とは思えないなassertThat()

🍻 @kompiro

@ryoasai74 ほんとですね。英語ネイティブ的というより、オブジェクト指向的な表現と僕は思いました。クラス自身を比較するのであれば、Class.getCanonicalName()とか、何らかの回避ができないですかねー。そこまで無理して使わなくてもいいともおもいますけど。

🍻 @kompiro

(クラス同士を厳密に比較するのってアンチパタンな気がしてきた)

Ryo Asai @ryoasai74

@kompiro そうなりますかね。何事もgolden hammerであってはいけませんし、いずれにしても、assertEquals()との使い分けは必要なのかもしれませんが、型安全性とエラーメッセージの可読性はhamcrestの利点かもしれません。

Tadayoshi Sato / 佐藤 匡剛 @tadayosi

@ryoasai74 今までJUnit4でテストコードをかなり書いてきましたが、正直クラスの比較をしたいと思ったことは一度もなかったです。static importはEclipseの設定次第で便利になりますよ。

Ryo Asai @ryoasai74

@tadayosi @kompiro 業務ロジックのレイヤではそうかもしれませんが、例えば、以下のようなテストはどうしますか？ https://gist.github.com/946477

Ryo Asai @ryoasai74

assertThat()はDSLの一種で上位レイヤ向けなので、リフレクションを扱うのような低水準のインフラレイヤではassertEquals()を使う比重が大きくなるということかなと理解しましたが。

Tadayoshi Sato / 佐藤 匡剛 @tadayosi

@ryoasai74 @kompiro なるほど。インフラ層のテストコードは確かにクラスを比較するケースもありますね。 assertThat(Xxx.class == aClass, is(true)) とか？

koichik @koichik

@tadayosi いつも public static Matcher<Object> sameClass(Class<?> clazz) って作ってる．．． http://ow.ly/4J1XG みたいなやつのテストでは必要．

🍻 @kompiro

@ryoasai74 @tadayosi TypeDescriptorのラーニングテストですかね。TypeDescriptorが果たすインタフェースの確認ができればいいので、isで十分いいかなと。もし内部でProxy作って返してきてたら、厳密に比較したらfailしませんか？

Ryo Asai @ryoasai74

@tadayosi なるほど、そのイディオムは使えるか。あとは、慣れの問題か。Groovyで括弧が省略できればもっと自然かもしれません。

🍻 @kompiro

(なんか、眠くて間違った事言っている気がしてきた。Proxy関係ない)

Tadayoshi Sato / 佐藤 匡剛 @tadayosi

@ryoasai74 @kompiro @koichik ググったら typeCompatibleWith っていうMatcherありますね。これはどうでしょう？　Matcher定義するのは簡単なので、最悪自分で作っちゃうのもありですね。

Ryo Asai @ryoasai74

@kompiro @tadayosi is()では以下のエラーなり、意図した比較になりません。 Expected: is an instance of java.lang.String got: <class java.lang.String>

🍻 @kompiro

@tadayosi @ryoasai74 @koichik あと、hamcrest-library っていうMatcherの拡張みたいなのにisA(Class)っていうMatcherもありました。 http://bit.ly/kHSAQm

🍻 @kompiro

(あれ？JUnitのassertThatはhamcrest由来じゃなかった？)

Ryo Asai @ryoasai74

@tadayosi @kompiro typeCompatibleWithだと意味が変わってしまうため、だめですね。代入互換性でなく、正確なクラスの比較ができなくてはなりません。

Ryo Asai @ryoasai74

@kompiro @koichik @tadayosi isA()はクラスとそのインスタンスの比較だから、is()と意味的に同じなのではないかな。英語的な読みやすさの違いだけなのでは。僕はFW開発をする関係上クラスの比較って結構必要なイメージなのですが、標準でないのは意外ですね。

🍻 @kompiro

@ryoasai74 @koichik @tadayosi isA()はソースコードみたら、is()を中で読んでいました。@koichik さんの紹介された、SeasarのMatcherがいいですね。ありがとうございます。

Ryo Asai @ryoasai74

@kompiro @koichik @tadayosi 皆さん、どうもありがとうございました。今後使えるところでは、なるべくassertThat()を使ってみようと思います。

Tadayoshi Sato / 佐藤 匡剛 @tadayosi

@ryoasai74 assertThat 私も好きで使っていますが、JavaだったらJDaveとか使ってBDDやってみるのもいいかもしれませんね。ScalaだったらSpecsがありますが。

Ryo Asai @ryoasai74

@tadayosi BDDは今のところ未開拓ですね。assertThat()もそうですが、非常に残念なことながら普通のSI案件では使いこなすのを想像できないところがありますね。小規模のアジャイルな案件でTDDでやるなら是非採用したいところです。