fixture テストのサンプルを追加する

[m-tmatma]

PR の目的

fixture テストのサンプルを追加して、使い方を共有する

カテゴリ

• その他 (単体テストのサンプル追加)

PR の背景

fixture テストを sakura editor で実装していないので、使い方を示す。

PR のメリット

fixture テストの実装方法の参考になる。

PR のデメリット (トレードオフとかあれば)

なし

PR の影響範囲

• 単体テストが増える
• [WIP] vs2017ソリューションにテストを組み込みたい #796 (vs2017ソリューションにテストを組み込みたい #793) のプロジェクトにこのPRで追加したファイルを追加する必要がある

関連チケット

#943 (comment)

参考資料

• http://opencv.jp/googletestdocs/primer.html#primer-test-fixtures
• https://github.com/google/googletest/blob/ea9c49b9cd410cca4df39b376f7da7cddf2dcf6d/googletest/docs/primer.md#basic-concepts
• https://github.com/google/googletest/blob/ea9c49b9cd410cca4df39b376f7da7cddf2dcf6d/googletest/docs/primer.md#test-fixtures-using-the-same-data-configuration-for-multiple-tests

[m-tmatma]

実行するとこんな出力になります。

[----------] 2 tests from FixtureSample
[ RUN      ] FixtureSample.Test1
FixtureSample::SetUp
FixtureSample::Test1
FixtureSample::TearDown
[       OK ] FixtureSample.Test1 (0 ms)
[ RUN      ] FixtureSample.Test2
FixtureSample::SetUp
FixtureSample::Test2
FixtureSample::TearDown
[       OK ] FixtureSample.Test2 (0 ms)
[----------] 2 tests from FixtureSample (84 ms total)

[m-tmatma]

流用した説明の不備を修正

[berryzplus]

対応ありがとうございます。

思うところあって、ぼくの判断は保留します。

実行するとこんな出力になります。

#944 (comment)
開発中には、こういう出力が欲しいケースがままあります。

ある程度形が出来上がって、CIにかける段階でこういう出力が欲しいかどうかは好みです。
Test Driven Development を模索するなら「欲しい」側に倒れそうですが、単純なprintfだと出力を抑制する方法がないのを気にしています。

[m-tmatma]

この出力は SetUp 等の説明をするために入れたもので普通はいらないです。
使い方を示すサンプルなので。
こういうことができます、ということを示すことが目的です。

[m-tmatma]

printf を使わずにやる方法もあるみたいですが、
それはまた別のサンプルで。

[m-tmatma]

レビュープリーズ

[m-tmatma]

思うところあって、ぼくの判断は保留します。

このコメントはいただけないです。
これなら、こういう理由で反対というほうがまだいいです。

保留して、どうするのかが欲しいです。

[berryzplus]

保留して、どうするのかが欲しいです。

他レビュアーの参入待ちです。

強く反対はしない（＝入れる意味はないと思う）けど、入れたい人がいれば勝手にやってくださし、と宣言したつもりでした。

---

久しぶりに見ましたけど、入れる意義（＝マージしたら何が嬉しいか）がわからんかったです。どっちかというと「要りますか？＝（ＮＧ寄り）」の印象です。

• fixture テストとは？の説明がない。 なんだと思ってます？
• 「googletest fixture テスト」をググれば、サンプルは簡単に発見できます。海千山千のサンプルとPRのサンプルの違いは？

[m-tmatma]

でら最初からそう書けばいい。

[berryzplus]

fixtureテストとは何か、使うと何が嬉しいかを説明した上でのPRであったなら、価値のあるものだったと思っています。

ぼくは初めからどういうものか知っていたので出力に工夫がないのを気にしましたが、別な観点で評価することも出来ると思っています。評価される機会を奪いたくなかったのでああいう書き方をしてました。

「これがfixtureテストだ。見習えアフォども！」な方向に進めばマージの可能性はあったんじゃないかな。サンプル書くってのは本質的にそういう側面を持つ行為だと思っとります:smile:

[m-tmatma]

だったら最初にそれを書かないと意味がないです