privateメソッドのテストについて

irof @irof

テストを書きたいと思うほどの責務をprivateメソッドに持たせてるのはどうなんだ。とは思う。 #tddbc

瀬良 @shela_

@irof publicからprivateを含めて検証するのと、private単体だけで検証するのであれば、先にprivate単体で検証しておいた方が安心感があると思う

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@irof public と private の境目はなに？

irof @irof

@kyon_mm @shela_ privateは内部の共通化とか処理の固まりに名前をつけるとかのメソッド抽出で出てきて、最初からprivateにするのはあんまないと思ってる。

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@irof ある共通的な処理の固まりがprivateならテストしたくなるほどの責務を持たせる場合があるんじゃないの？

irof @irof

@kyon_mm それ書いた時点でpublic経由のテストは書き終えてます。

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@irof っていうことはprivateから始まるプロダクトコードはTDDでは書いてはいけないということでよい？

irof @irof

@kyon_mm privateから始まるって状況がわからない・・・

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@irof 設計を先に済ませていると、はじめから「あー、この処理絶対に共通的に必要じゃん。privateで定義すればいいよね。」ってなる場合。

瀬良 @shela_

@irof @kyon_mm publicの処理で絶対にprivateには入らない値での検証がどうなるか。privateを将来再利用する機会が来た場合に初めてその値を検証することになるので、その時までバグが発覚しないことになりそう。

irof @irof

@shela_ その「将来再利用したとき」でいいと思います。

irof @irof

@kyon_mm んー……責務持ち過ぎ感はありますが。それだけ核になることならそこから手をつけると思うので、多分先に書くから行けるんじゃないかなと。

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@irof いや、結果として同じコードになるなら責務持ち過ぎっておかしくないですか？ 先に書くからっていうのは、先にprivateにしていても、それをつかうpublicを書くであろうそこからテスト通せる的な意味です？

irof @irof

@kyon_mm 「メソッド単体でテストしたい」気持ちが強いなら、やっぱ中に隠すにはやり過ぎだと思う。

瀬良 @shela_

@irof @kyon_mm それは必要なアクセス範囲を超えて公開することになるから設計としてよくないかと

irof @irof

@shela_ @kyon_mm かもしれないですね。（対象が明確でないですし十把一絡げに言いたくないです。）

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@irof 「publicとprivateの境目は<例として>共通処理である」「共通処理にはテストが必要だと思うが、それが予見できている場合に最初からprivateにする方針ではどうするのか？」っていう流れなんだけどそこで「メソッド単体でテストしたい」は別じゃない？

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@irof 「メソッド単体でテストしたい」かどうかではなくって、論点としては「初めからprivateにするということはTDDではゆるさない(アンチパターンくらいでいいかしれない)」のかどうかです。別にprivateでテストしたいとかそういう意味ではない。

irof @irof

@kyon_mm その流れでいくと、たぶん共通処理をpublicに書いてからメソッド抽出しますが、「public経由でテストできてんじゃん」です。最初からprivateで書いて、「privateにテスト書く書かない？」の話をするのはなんかコレジャナイ感。

irof @irof

@kyon_mm 「アンチパターンである！」と言い切りたくはないですが、私は積極的にはしないです。

しんすく(け) さん。 @snsk

@irof @kyon_mm こういうの死ぬほど参考になりますm(_ _)m

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@irof なるほどね。だいたいわかった。そしておおいに同意だ。

irof @irof

@kyon_mm 途中中途半端に変なこと言ったっぽくてなんかごめんなさいな気分（・ω・`）

きょん＠アジャイルコーチ、システムアーキテクト @kyon_mm

@snsk たのしいっすよねー

irof @irof

とりあえずモヤモヤを言葉にできたのでありがたい。心臓には悪いw