意図が伝わりやすいテストの書き方

ユニットテストでリテラルを使うか変数を使うか問題。いろいろ話を聞いて考えた結果をまとめる。

function add(x, y) {
  return x + y
}

テストの意図が伝わりづらい書き方

const val1 = 10
const val2 = 20
const expected = val1 + val2

const actual = add(val1, val2)

expect(actual).toBe(expected)

• テスト対象の中身を知らないと読み解けない
• add(val1, val2)がval1 + val2と等価であることを表現しても意味がない
• テスト対象のロジックを再表現しているだけ

テストの意図が伝わりやすい書き方

expect(add(10, 20)).toBe(30)

// テストツールによっては呼び出しが長くなるので変数使っても良いかも
const actual = result.current.add(10, 20)

• テスト対象の中身を知らなくても読み解ける
• add(10, 20)の結果は、（add関数内でどんな計算がされてるか知ったこっちゃないけど）30が返ってきてほしい
• 英文的にも「Test expect “add(10, 20)” to be “30”」と意味が伝わりやすい