privateなメソッドのテストをしたいです(.net core2.0以上で)

Question

###前提・実現したいこと
こんばんは。いつもお世話になってます。

privateかつstaticなメソッドのテストをしたいのですが、何かいい方法はないでしょうか？

ググっているとMSTestではPrivateTypeを利用した方法が有名みたいです。
しかし、
GitHubのこのイシューにあるように.net core2.0以上ではこの方法が使えません。

スタックオーバーフローのこの投稿の回答では、継承云々をすればいいということも書かれています。
しかしその方法の実装はまだ見つけられていません。

一応、勝手にその方法を想像して、
テストしたいメソッドを持つクラスAを継承したクラスBをTest.cs(仮名)内につくってみたりしました。
しかし、AとBが全く同じものであると言えないのがどうなのかなと思います。
それに、Aはできればsealedにしたいクラスです。
テストに書いたコードが間違っていたら辛いと思います。

PrivateTypeを利用するのと似た方法や、継承云々の方法の実装や、
sealedでも可能にしたり、そんな方法を知って使えるようにしたいです。

###補足情報
環境は、
visualstudio2017 C# .netcore2.0
です。
テスト入門者(MSDNのチュートリアルを少しやったくらい)です。

知恵をお貸しいただければ幸いです。
よろしくお願いします。

###追記(2019年9月8日21:00)
なるべく簡潔に書こうとして失敗して書き漏らしてしまいました内容について、追記します。
今回の質問の対象となっているprivateメソッドは既存のコードの一部です。
ただのサンプルアプリケーションのコードなので、
テストされているか定かではないですし、仕様もあいまいです。

Answer 1

若干ずれた回答になってしまうかもしれませんが、本来 Private なメソッドはテストするべきでないように思います。

Private であるということは Public な他のメソッドから呼んでいるので、その Public なメソッドをテストすることによりカバレッジ取るべきように思います。

また、Private メソッド単体でテストしたくなるほど条件や内容が複雑であれば、他のオブジェクト等への切り出しを検討するかと思います。

Comments

[kaisen]

回答ありがとうございます。

>>本来 Private なメソッドはテストするべきでない
ググってみたら有名な話みたいですね。沢山でてきました。
理解しきれるかわかりませんが少しの間調べてきて、検討してみます。

[kaisen]

TDDの本を斜め読みしたりしたのですが、
テストで成功したpublicメソッドをリファクタリングした結果、
生まれるのがprivateメソッドとなるものととらえました。
確かにprivateメソッドのテストではなく、
元となったpublicメソッドのテストのみで事足りるような気がしてきました。
勉強になりました。

ただ書き漏らしてしまって申し訳ないのですが、今回の質問の元となったprivateメソッドは、テストを通さずに作られたであろうサンプルアプリケーションのコードなんです。
なので、既存のコードでprivateメソッドをテストするときのことなど
よろしければ教えていただきたいです。

[unhappychoice]

その場合も、自分であればその Private メソッドを呼んでいる Public メソッドをテストして、 Private メソッド内の分岐すべてを通します。
そのあとはPublicメソッドのテストが動作を保証してくれるので、適切なオブジェクトにリファクタリングしたりあとは自由に煮るなり焼くなりという形ですかね

[kaisen]

今回のPrivateメソッドの中身、分岐も複雑とは言えなそうです。
呼び出している側のPublicメソッドのテストで十分間に合いそうです。
これでやってみます。もし何かあったら別の質問として投稿することにします。

助かりました、ありがとうございます。

Answer 2

unhappychoice さんに同意しますが、それでもどうしてもテストしなければならない、そして PrivateType が使えない状況なら、リフレクションが役立つかもしれません。

Comments

[Zuishin]

コメント削除

[kaisen]

回答ありがとうございます。

たとえば大規模なテストになってしまう場合、
リフレクションだと少し問題になるとかどこかで読みました。

ただ、手軽な今回のテストでは十分に視野に入れていこうと思います。
また少し規模が大きくなってきたらいろいろご教授いただきたいです。

ありがとうございました。

[Zuishin]

読むときには適当な読み方をせず、理由までちゃんと読んでください。
むしろ理由のほうが大事です。

[kaisen]

確かにその通りです。ありがとうございます。
その記事("どこか"といっていたもの)の魚拓取っておいたので後で読み返そうと思います。
勉強下手なもので、ある程度以上に必要になった時にしか記憶に残らなくて。