プライベートメソッドのテストは書かないもの？ - t-wadaのブログ

この文章の背景 この文章はプライベートメソッドのテストを書くべきか否かに関する knsmr さんのご質問に対して 2013/03/13 に QA＠IT で回答したものです。残念ながらQA＠IT のサービス終了（2020/02/28）と共にアクセスできなくなってしまったため、運営を行っていたアイティメディア株式会社様、開発を行っていた永和システムマネジメント様、そして質問をされた knsmr さんに許可とご協力をいただき、当時の回答をサルベージしてブログに転載する運びとなりました。 プライベートメソッドのテストはよく議論になるテーマですので、当時の回答を再編集し、knsmr さんのご質問も含め、ご利用いただきやすいライセンス CC BY（クリエイティブ・コモンズ — 表示 4.0 国際 — CC BY 4.0） で公開いたします。 目次 この文章の背景 目次 knsmr さんのご質問 私の回

[t-wada]

QA＠ITのサービス終了と共に読めなくなっていたプライベートメソッドのテストに関する議論を、関係する皆様に許可をいただいた上でライセンス CC BY で公開しました

[fukken]

"プライベートなメソッドのテストを書きたいということは、実はテスト対象の責務が多すぎることを示唆している" 大体これになる気がする。

[megumin1]

筆者の主張は間違い。特定の言語の貧弱な機能に思考が制限を受けて誤った結論を出している典型的な例。Rust等ではどこでも必要な場所に書けます・書きますよ。"テストを書くべき"と"上位層からの可視性"は直交する概念

[jnst]

事業ドメインによっては糞複雑な処理が必須となるからpublicメソッドで全パターン網羅は辛すぎ。後に仕様変更された時の修正箇所も煩雑になるので個別にprivateメソッドをテストするのは影響範囲を限定する良い手法と思

[su_zu_ki_1010]

なぜテストするのか、を考えると、外から呼び出したときの処理結果が想定した内容かどうかを確認したい。privateなメソッドは外からは呼ばれないのでテストも書かないという話。実際に書ける書けないの話ではない。

[u1tnk]

“リフレクションを使えばプライベートなメソッドのテストは「できる」のですが、そのテストはやがて実装改善の邪魔になりかねません。”これ

[TMCNE]

何かにつけてモジュール分割して中身スカスカのソースファイル量産したがる人嫌い。/モジュール境界なのはその通りなんだけど、モジュールとして分割する程の処理じゃないからプライベートメソッドにしてるわけで…

[magnoliak]

privateのテスト、カバレッジ見すぎると書きたくなる気持ちがでてくるなーとか

[kathew]

永遠の命題って感じでどちらにもそれなりの理由があるので結論が出る話ではない／私は書かない派。"プライベートなメソッドの（中略）実はテスト対象の責務が多すぎることを示唆している"まさにこれが理由で

[shimarisugi]

オブジェクト指向設計ガイドだと結論は「private methodは作るな」「作ってもテストはするな」「ただしテストすることに意味がある場合は除く」だったな。3つ目は設計ミスだがfixできないときの最終手段。概ね記事の通り

[youko03]

“プライベートなメソッドや関数をテストする必要は無いと考えています。プライベートなメソッドは、実装の詳細であるからです。”

[winterfall]

"ほとんどのプライベートメソッドはパブリックメソッド経由でテストできるからです。プライベートメソッドは実装の詳細であり、自動テストのターゲットとなる「外部から見た振る舞い」ではありません。"

[okumuraa1]

]

[nilab]

「プライベートなメソッドや関数をテストする必要は無い」「プライベートなメソッドは、実装の詳細であるから」「パブリックメソッド経由でプライベートメソッドのテスト」「別クラスのパブリックメソッドとする」

[juve534]

privateメソッドにテストを書くことがあるので、それはなにか間違っているんだろうとは思っていた

[Windymelt]

ホワイトボックステストが書きたくなったらそれは設計改善の余地がある症候，ってことかな。実装改善の邪魔になるって書いてあるし。

[nekorails]

何度も読んでる

[tasshi820]

なるほど

[digo]

たまに書きたい時あるのよね。例えばクラス内に閉じた正規表現とか。

[diveintounlimit]

ブラックボックステストはそれでええかも知れんけどねぇ。プライベートもややこしいロジックのメソッドはテスト書いとるよ。パブリックメソッドのテストが死んだらどこで問題起きてるか追いかけるの大変だし。

[iR3]

なるほど

[vanbraam]

だいたい納得; 結局テストは契約の一種. 請負契約なら,アウトプットだけが問題で,それをどうやって成し遂げるかは契約の対象外. そこを縛りたくなるのは権限の委譲 or 責務の分解のどちらかが不十分

[Error401]

プライベートメソッド内のコードがテスト対象なのは明らか。なので「プライベートメソッドを呼び出すテストを書くか書かないか」という問題設定にするのが正しいと思う。

[deep_one]

プライベートなら「モジュール内のテスト」で終わりだろ？／テストを書きたくなるならそれはプライベート指定になってるのがおかしい、は分かる。

[tuto0621]

“短くまとめると、プライベートなメソッドのテストを書く必要は 無い と考えています。”

[GlassGrass]

純粋関数を低レイヤで実装するようなケースだと、保守じゃなく初期開発での正当性担保のためにプライベートでもバシバシテスト書いてる。リファクタリングでテストごと破棄するのもままある

[kmaebashi]

単体テストがホワイトボックステストだとするなら、publicかprivateかでテストの有無が変わるのは明らかにおかしいだろ。ややこしいロジックはprivateに隠蔽すべきだが、そこがテストできないなんて。

[naopr]

経験上、プライベートメソッドのテストをパブリックメソッドのテストで記述するとテストの意図がわかりにくくなってしまう場合が多いのでプライベートメソッドのテストとして書いてる

[RYoMa_0923]

?なんかこのもと記事が書かれた時にはマトモに存在してたか怪しい言語を持ち出してわーわー言うとるのがいるな。

[kagerou_ts]

だいたいリフレクション

[masatomo-m]

バグった時に人が死んだりしないシステムで、という前提なら同意

[ksugimori]

テストコードもクライアントコードのひとつなので疎結合であるべき、と思ってるので書かない派。

[THAL]

あとで読む

[rryu]

元の質問はテストドリブンでやっているとプラベートメソッドを実装する時にテストが書きたくなるがどうすればという感じなので回答がちょっと違うような気もする。

[jaguarsan]

書捨ててコミットしない派

[kazkaz03]

基本的に書かなくて良いと思うけど、「仕方ないけど書かなきゃいけない」ケースがあると言っとかないと、原理主義おじさんが暴れだす

[ngmy]

privateメソッドのテストを書きたくなる時ってたいてい責務の配置をミスっていて、設計を見直すと別のクラスのpublicメソッドに落ち着くことが多いよな。と思って読んだら書いてあった。

[baronhorse]

小さいクラスでやるならこうだよね。複雑すぎるものは外に出したほうがいい

[aprifield]

“ンの一つとして「リファクタリングの支えになる」ことが挙げられますが、リファクタリングとは簡単に言うと「外部から見た振る舞いを変えずに内部の実装をきれいにすること」です。外部から見た振る舞いは、多くの