- 16 名前:デフォルトの名無しさん mailto:sage [2017/09/10(日) 10:56:13.13 ID:G4ZVCKWZ.net]
- >>10
> バリデーションのテストは > 「ある項目に数字以外を入れたらエラーになる」という > 仕様を満たしているかどうかのテストであって正規表現のテストをするわけではない > 実装が正規表現である必要もない はい、そのとおりですね。 処理をしているのが正規表現であるかそれ以外のコードであるかは関係ない。 だからここでやるべきことはnumberOnlyという関数を作ること そしてそのnumberOnlyが仕様を満たしているかどうか? のテストです。 これは入念にテストする必要があるでしょう。 もしてテストしなければいけない(?)項目はもう一つ残っていますね? numberOnlyのバリデーションが設定されている項目はpriceだけではなく sizeやcountなどもいくつもあるでしょう。 それらの数値項目が10個あったとき、10個全てに numberOnlyでやったのと同じ入念なテストは必要ないのですよ。 まあやるとしたとしても「ある項目のバリデーションがnumberOnlyであること」 ぐらいでしょうが、これもまさに項目のバリデーションにnumberOnlyと 書いたんだから当然numberOnlyに決まってるだろという データの二重管理にすぎない問題になるわけですよ。 自分で定義したものが(書いたものが)、自分で定義した(書いた)とおりに なっているかなんて意味がないテストです
|

|