こんな辛いテストはいやだ

Copyright Drecom Co., Ltd. All Rights Reserved. 1
こんな辛い（?）テストはイヤだ！！
三上拓也

自己紹介
⁃ 三上拓也（@mkmn252）
⁃ 所属
⁃ 株式会社ドリコム
⁃ 2015年度新卒入社
⁃ 仕事内容
⁃ ネイティブゲームアプリの
サーバーサイドエンジニア

ざっくり趣味とか

※ 赤枠が行った公演

地獄めぐりしてきました

閑話休題

前置き
⁃ 所属するPJにおいて、Ruby/Railsのバー
ジョンアップを実行した
⁃ Ruby： 2.1.5 -> 2.3.1
⁃ Rails： 4.0.13 -> 4.2.7.1
⁃ テストが充実していて作業に安心できる
⁃ 大抵のコードを網羅している
⁃ バージョンアップ後のチェックがしやすい
⁃ （実機確認もするが）安心できる

ただ、テストの書き方が悪く、
テストしたい部分の前でつまずく or
テスト自体がコケることも・・・

本日の内容：
最近出会った
辛い(？)テストたち

今回紹介する辛いテスト
1. よくあるやつ
2. 文字列比較
3. ディレクトリ指定

⁃ 最初はテスト通る
$ rspec spec/models/skill_spec.rb
.....................................................................................................
Finished in 8.57 seconds (files took 8.64 seconds to load)
101 examples, 0 failures
$ rspec spec/models/skill_spec.rb
FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
⁃ 突然テストが通らなくなる

なぜコケるの？？
# エラー出力
Failure/Error: FactoryGirl.create(:skill, id: i) unless Skill.exists?(id: i)
ActiveRecord::RecordNotUnique:
Mysql2::Error: Duplicate entry '1' for key 'PRIMARY': INSERT INTO `skills` (`id`,
`created_at`, `updated_at`) VALUES (1, '2017-01-26 01:14:33', '2017-01-26 01:14:33') :
master
＿人人人人人人人人人人人人人人人人＿
＞ Primary KeyのDuplicate entry ＜
￣Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y￣

原因
⁃ データが論理削除されてた
before(:all) do
# 参照されるので予めデータを作成する
(1..10).each do |i|
FactoryGirl.create(:skill, id: i) unless Skill.exists?(id: i)
end
End
after(:all)
(1..10).each { |i| Skill.find_by(id: i).try(:destroy!) }
end

対処方法
⁃ FactoryGirlでのid指定createはやめるべ
き
⁃ どうしても必要なら、afterで確実に消し去
る

2. 文字列比較

2. 文字列比較
管理画面などのView
⁃ 出力されるHTML(imgタグ)のテストがある
$ rspec spec/helpers/output_html_tag_helper_spec.rb
F..................F.
Failed examples:
rspec ./spec/helpers/output_html_tag_helper_spec.rb:37 #
OutputHtmlTagHelper.character_image_tag generate img tag of character
rspec ./spec/helpers/output_html_tag_helper_spec.rb:5 # OutputHtmlTagHelper.link_tag make
IMG tag to example.com

2. 文字列比較
原因
⁃ 出力される属性の順番が変わった
Expected：
<img alt=“character” class=“character class_a” src=“http://example.com/character.png” width=“540’ />
Got:
<img width=“540” alt=“character” class=“character class_a” src=“http://example.com/character.png” />
# テストコード
describe “.character_image_tag” do
it “character_image_tag generate img tag of character” do
expect(helper.character_image_tag(20).to eq ¥
’<img alt=“character” class=“character class_a” src=“http://example.com/character.png”
width=“540” />’
end
end
# テスト結果

2. 文字列比較
対処方法
⁃ Nokogiriでゴリゴリ頑張る
⁃ rspec-html-matchers
⁃ テスト捨てる
⁃ → 結果捨てました

同PJの別のレポジトリにて・・・
⁃ あわせてrubyのver.upしようと修正した
(´･ω･`)
＿(__つ/￣￣￣/＿
＼/ /
修正完了したし、
テスト実行するか・・・（ﾎﾟﾁ

$ rspec
.......................
create file_1
create file_2
create file_3
create file_4
create file_5
create file_6
create file_7
create file_8
create file_9
（中略数千行）
Finished in 13 minutes 46 seconds (files took 0.56944 seconds to load)
＿人人人人人人人人人人人人人人＿
＞大量のファイル生成ログ＜
￣Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y￣
※ 別ディレクトリを参照して別ファイルを生成するスクリプトのテスト実行時

(´･ω･`)
＿( つミﾊﾞﾀﾝｯ
＼￣￣￣＼ミ
￣￣￣￣
(´･ω･`)
＿( )
＼￣￣￣＼
￣￣￣￣

原因
⁃ ローカルの別のディレクトリを参照していた
対処方法
⁃ spec/dummyを作ろう
# テストは../other-directoryが存在することを前提に書かれています
raise “../other-directory is not exists!” unless Dir.exist?(“../other-directory”)

まとめ
機能変更 or 追加してないのに、
テストコードの修正が（極力）ないように
テストを書こう

（余談）個人的もっと使われるべきと思うもの
⁃ FactoryGirl.create_list
⁃ 複数データ一括で作りたいなら便利
⁃ Rspec change matcher
⁃ 値の変化チェックにはもってこい

おわり

こんな辛いテストはいやだ

Recommandé

Recommandé

Contenu connexe

Tendances

Tendances (10)

Similaire à こんな辛いテストはいやだ

Similaire à こんな辛いテストはいやだ (20)

Dernier

Dernier (9)

こんな辛いテストはいやだ