テストをするときネストをするな - Avoid Nesting when you're Testing

Kent C. Dodds 氏による Avoid Nesting when you're Testing という記事を翻訳させていただきました。 以下、本文。 これからご紹介するのは、React コンポーネントのテストに適用される一般的なテスト原則です。例に React を使用していますが、この概念正しく理解するのに役立つことを願っています。

[jay-es]

beforeEach などをコード再利用の手段として使用するのは NG。コードをたどってミュータブルな変数の値を時系列で追跡しなければいけなくなる。セットアップ用の関数を作り、テストごとに呼び出す

[kazokmr]

強い言葉で伝えてるように感じたけど原文はあくまで個人の意見って感じだった。テストコードをプロダクトコードと同じ思考で書いてるのをよく見るけど、敢えて冗長に書いた方が良いよね

[igrep]

行きすぎた抽象化の問題はそうだとして、beforeEachが悪いと言うよりbeforeEachに伴ってmutableな変数を使わざるを得なくなるってのが問題を誘発してるんじゃないかと思えるなぁ。RSpecのlet!みたいなことができればいいのか？

[secseek]

テストって本体のコードと全然違う発想が必要になりますよね。同じような処理でもだらだら書き連ねる方が正しいとか

[carolina04]

テストが単純な場合はコードを複製した方が簡潔になることもあります。この小さなテストセットでは関数の抽象化はあまりメリットがなく、開発者がファイル内でこれらの関数が定義されている場所を探すコストがかかり

[ustam]

森羅万象においてネストが正しかった試しは無いからな。

[versatile]

ネストと、テストが韻を踏んでる。それを面白がって書いた文章ではなかろうか

[mak_in]

できるだけ冗長でも宣言的に分かりやすく、で作るんだけど、地味に静的解析で引っかかるのが面倒。あれをテスト用にカスタマイズするのダルいから、テスト用の静的解析がメジャーになってデフォルト設定にして欲しい