意味がないテストをするな。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

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

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