bitterfox / slack-java-api

Provide full api of slack via java
1 stars 0 forks source link

fail to test #71

Closed pgmot closed 9 years ago

pgmot commented 9 years ago

そもそもテストをするときにサーバ叩きに行くのあまり行儀が良くない. モックを作ってJSON食わした時にちゃんと動くかという手をテストすべき

bitterfox commented 9 years ago

そのあたりよくわからないので,いい感じにしてもらいたいです.

bitterfox commented 9 years ago

ちなみに,例外周りはサーバの実装に合わせたいので,リクエスト送ってるんですが,それもあんまり良くないですか?

pgmot commented 9 years ago

毎回サーバ叩くの良くないからそのJSONと合致してるかみたいな挙動チェックするほうがええで

bitterfox commented 9 years ago

そのJSONというと・・・?

pgmot commented 9 years ago

実際に取ってきておく

bitterfox commented 9 years ago

なるほど.

pgmot commented 9 years ago

実装としては,テスト用のApiとGetApiRequestを作って差し替えて,いい感じのApiResultを返すようにしたら内部実装変えなくていいと思う

bitterfox commented 9 years ago

わりかし大きなリファクタリングが必要そうなので,明日以降順次変えていきます.

とりま,実装は思いつきました.

bitterfox commented 9 years ago

ちなみに,そうした場合,実際に通信してデータとれてこれているか確認できないですがいいんです?

pgmot commented 9 years ago

ここでテストすべきなのは,あくまでもレスポンスに対して正しい挙動を行うことをテストすべきなのであって, サーバと通信してデータを取れてこれているかというのは今テストすべきではない. サーバと通信してテストするのは一つのテストでいいと思う.

自社開発とかでテストサーバ用意できるならサーバ叩くけど,slackのサーバ叩きまくるのはかなり行儀が悪いし気軽にテストできない.

pgmot commented 9 years ago

モック.この辺りのライブラリ使うと簡単だと思います. http://qiita.com/mstssk/items/98e597c13f12746c907d

あと流れる話し合いはgitter使いましょう