テストコード内では条件分岐を書かないようにする

テストコード内では条件分岐を書かないようにする 2023.01.21 誰でも読める愚直なコードであることの 1 つの目安として、テストコードの中に if 文や三項演算子などの条件分岐が入り込んでいていないことが上げられます。if 文が存在するコードはアンチパターンであるといえます。実際に if 文がテストコードの中に入り込んだ例を見てみましょう。 テストコードは誰でも読める愚直なコードであることが求められます。テストコードにはある種のドキュメントのような、コードの仕様を説明する役割が求められているためです。テストの期待結果が変数になっていて、定義元までジャンプしないと値を確認できないだとか、条件分岐やループが入り込んでいて複雑性が上がっている状態ですと、素直に読みやすいとは言えません。 コードの中では重複排除をするためにさまざまなテクニックを駆使することがありますが、これは単にテストコード

[berlysia]

test.eachを使うときは、そのケースがどういう観点で存在するかも補足するような項目を追加しているかなー。あるいはテーブル自体を分割して観点別にする

[arx0balest]

Arrange-Act-Assert pattern だけでいいよ。みんな凝った書き方しすぎ。

[nihonbuson]

すごく良い試みですね！さらに良くする提案として、テーブル駆動のコードにはテストの意図が無くなっているので、書き加えてみるのはいかがでしょうか？テストメソッド名に「8文字以上32文字以下ならば有効」とか。

[shikiarai]

ものすごく複雑な日本語のテスト仕様書を作ったらちゃんとテストできないってのは簡単に納得されるのにコードになると途端にわからない人、いるよね。

[shingo-sasaki-0529]

さすがにこんなテストコード書く人おらんやろって思ったけど、アプリケーションコードに対する DRY をそのまま適用しようとするとこうなっちゃうのか。

[rck10]

これは大事 / テストは入力情報（パラメータ、テーブル、ファイルなど）を制御化に置いた状態で想定通りの出力を確認することなので、分岐の出番はほとんどない。例外として、経過時間など制御しづらい場合はある。

[yujiorama]

いい意見。それはそれとして、出力値を境界値分析して、出力同値ごとにテストを分けると、1つのテストに書くアサーションがほとんどの場合1つだけになり、見通しが良くなります