テストコードの期待値はDRYを捨ててベタ書きする ～テストコードの重要な役割とは？～ - Qiita

はじめに みなさん、DRY原則はご存知でしょうか？ DRY = Don't repeat yourselfの略で「繰り返しを避けること」という意味ですよね。 良いコードを書くための重要かつ基本的な原則なので、みなさんよくご存知だと思います。 ですが、DRY原則はテストコードを書く場合は必ずしも最善にはならない場合があります。 他の人が書いたテストコードを見ていると、テストコードにDRY原則を適用したために、かえって悪いコードになっているケースをときどき見かけます。 この記事ではなぜテストコードをDRYにすると良くないのか、ということを説明します。 追記：タイトルを変更しました @t_wada さんのコメントを受けて、タイトルを見直しました。 「テストコードはDRYを捨ててベタ書きする」 => 「テストコードの期待値はDRYを捨ててベタ書きする」 【注意】この記事は画一的なテストコードの書き

[qtamaki]

DRYってつまり、テストする側もされる側も同じ人間が同じロジックを実装する事になり両方間違えるよね。もしくは間違えたのがどちらかわからない問題発生

[YAK]

仕様変更してレッドにならないテストコードはあまり良くないと思っていて、それを「壊れやすい」って表現するのだけあまり同意できなかった

[Xray]

わかる。が、期待値は本質的にDRYにならないと思った。

[mat_aki]

テストコードの期待値はDRYを捨ててベタ書きする ～テストコードの重要な役割とは？～ by @jnchito on @Qiita

[petitviolet]

ベタ書きにすればいいってもんじゃないよね。抽象化はやめようくらいかと。

[shimada227]

“shared”

[dorapon2000]

“この時点でテストコードの重要な役割の一つである「対象のコードの仕様がひと目でわかること（仕様書になっていること）」を満たせなくなります。”

[carolina04]

“この時点でテストコードの重要な役割の一つである「対象のコードの仕様がひと目でわかること（仕様書になっていること）」を満たせなくなります。 改善策：期待値はベタ書きする というわけで、こういったケ”

[ginpei]

面倒だからと期待値をコードで表現すると、失敗するべき試験で失敗しなくなっちゃうのでべた書きしましょう、と。コメント欄にあるが必ずしもDRYが悪なわけではない。

[lequinharay]

定期的に出てくるやつ

[machupicchubeta]

いいはなし

[tyru]

わかる

[zyzy]

テスト書くために電卓使って同じ計算したことある算数苦手人間。電卓がバグってたら知らん。

[teracy_junk]

Androidでだけど、期待値はべた書き（parameterized含め）にしてる

[endo_5501]

わかる

[etakaha]

期待値を直書きする

[ono_matope]

これも良い記事だ。

[uva]

勉強になります

[vanbraam]

これいつも悩む;テストコードは共通化も(rspecでいうとshared_example)も危険.可読性は情報のlocalityが高い程良くなる傾向があるが,共通化すると情報のlocalityが落ちる

[NetPenguin]

よくいわれる話だが、プロダクトコードには一般解、テストコードには特殊解。あと「ベタ書きすれば良いってもんじゃないよ」とかなら、そのテスト範囲に特化したDSLをつくろうね。

[dekasasaki]

あー、これはよくわかる。

[kgrock]

テストコードの期待値はDRYを捨ててベタ書きする ～テストコードの重要な役割とは？～

[tbpg]

コメント欄のパラメタライズドテストいいなー。すき。

[wnoguchi0727]

良いまとめ

[marmotY]

ただ繰り返しを避けることと抽象化の区別が付いていない事例。それだけ。

[akatakun]

「対象のコードの仕様がひと目でわかること（仕様書になっていること）」・「確実にバグを検出できること」とDRY原則は両立しにくい

[masaru_b_cl]

震源ここだったらしい

[w1234567]

自分は期待値は文字列ベタ書きで1メソッド1回しか比較しないってスタイルに落ち着いた、下手にややこしいことされると保守しづらいねん

[Dai_Kamijo]

テストコードはDRYを捨ててベタ書きする ～テストコードの重要な役割とは？～ by @jnchito on @Qiita ｜コメント欄含みで興味深い — 上條　大 (@Dai_Kamijo) June 6, 2016 from Twitter https://twitter.com/Dai_Kamijo June 06, 2016 at 12:16PM via IFTTT

[onefootinthegrave]

テストコードは愚直にかいたほーがいーよね