意味がないテストをするな。VERSION==1.0.0 [無断転載禁止]©2ch.net

**デフォルトの名無しさん** · 2017/09/09(土) 15:31:54.82

app.version = '1.0.0'

テストコード
expect(app.version).to.be '1.0.0'

バージョン番号1.1.0に変更すっぞ！

app.version = '1.1.0' に修正
expect(app.version).to.be '1.1.0' に修正

これただのデータの重複、たんなる二重管理ですから＼(^o^)／
メンテナンス工数が二倍になるだけ

誰もそんなことやらねーよって思うかもしれないが

関数だったらやってしまうんだよね。
カバレッジがー（笑）

function getVersion() { return '1.1.0' }

expect(app.getVersion()).to.be '1.1.0'

他にも色々と、意味がないテストがある
意味がないテストしてるやつが多い。
関数の実行結果をテストコードにコピペしてテスト作るやつとかな

**デフォルトの名無しさん** · 2017/09/09(土) 16:54:48.95

どっかユーザの目に入るところにバージョンでるなら意味あるんじゃね？

**デフォルトの名無しさん** · 2017/09/09(土) 17:25:35.09

その前に意味のないコードを入れるな
意味があるならその意味に沿ってちゃんとテストすべき

**デフォルトの名無しさん** · 2017/09/09(土) 20:10:39.54

その前に意味のないスレを立てるな

**デフォルトの名無しさん** · 2017/09/09(土) 20:49:02.98

意味がないテストっていうのは他にも有るな。
例えばテンプレートの適用結果をテストするとかな

HTML生成のためのテンプレートがあって、
テンプレートファイルを手書きして（ここまでは普通）
そのテンプレートに値を入れたHTMLをテストとして手書きする（アホ）

**デフォルトの名無しさん** · 2017/09/09(土) 21:17:51.67

正規表現バリデーションのテストも大概意味がない。
正規表現がものすごく複雑っていうのなら話は別だが、
/^[0-9]+$/ みたいに書いてあるものに対して
数字以外入れたらエラーになることを確かめるテストはいらない

**デフォルトの名無しさん** · 2017/09/09(土) 21:23:57.43

まあ正規表現バリデーションのテストに関して言えば、
正規表現を直接書くのではなく、numonlyなどというメソッドを作るべき。
そのnumonlyのテストを書くのは少しは意味がある。

だけど、このフィールドはnumonlyになっているか？という
確実にテストは必要ない。

SQLのスキーマのテストしてるようなもんだよ。

SQLのcreate table文で、priceの項目をintegerと書いて
そのテストコードで、create table文のpriceがintegerであること
とテストすることに何の意味があるのだろうか

**デフォルトの名無しさん** · 2017/09/09(土) 23:25:06.47

あー、関係ない所でアイデア思いついたわ

一部でテストのことをスペックと呼ぶ文化が有るだろ？
変数の型とかバリデーションとか
コードに書かないでスペックに書けば良いんじゃね？

**デフォルトの名無しさん** · 2017/09/10(日) 02:59:20.70

>>1
>function getVersion() { return '1.1.0' }
>expect(app.getVersion()).to.be '1.1.0'

バージョンアップのたびにテストコードを変更してメンテするのは微妙すぎるだろ
そうじゃなく、そのバージョン用のテストスイートを書いてるのであればその種のテストにも十分意味がある
(例えば1.0系のテストと1.1系のテストを並列でメンテしてる場合)

今は単純文字列を返す実装だから無意味に感じるのだろうが
リファクタして設定ファイルからバージョンを取得するようにしたり
DBからバージョンを取得するようになったりしても
仕様を満たしてるかどうかを確認できる

テストが不要なくらい明白な実装かどうかを
毎回判断しながらテストケースを管理することのほうが意味がない

**デフォルトの名無しさん** · 2017/09/10(日) 03:10:32.77

>>6
バリデーションのテストは
「ある項目に数字以外を入れたらエラーになる」という
仕様を満たしているかどうかのテストであって正規表現のテストをするわけではない
実装が正規表現である必要もない