テストを遅くしないように気をつけること

テストを遅くしないように気をつけること MatchingAgent x Makuake 合同勉強会 2017/06/22 ( 木)
@kkkw

お前誰よ @kkkw フリーランス 2017 年2 月～ Makuake にjoin サーバーサイド中心に上から下まで

Agenda テストが遅いと何が困るのか Makuake でやったテストの高速化テストコードのリファクタ例

テストが遅いと何が困るのか CI が詰まる PR 発行( テスト走る) →マージ→デプロイの流れ緊急リリースだと致命的品質が落ちるテストを書く、実行するモチベーションが落ちる
開発手法の選択肢が減ることもある

Makuake でやったテストの高速化

やったこと 1. MySQL のチューニング 2. テストコードのリファクタ

Jenkins のjob 実行時間

MySQL のチューニング環境依存のお話なので今日は割愛

テストコードのリファクタ基本的にはプロダクションコードと同じことに気をつける重い処理をやらないデータベースのアクセス外部へのhttp アクセスディスクI/O テストそのものが必要かどうか考える

いくつか紹介なるべくDB へアクセスしない方法を紹介 php でのサンプルスライドの都合上PSR とかは無視確認してないのでエラーとかあるかも

そのFind 本当に必要？本当にDB のsetUP が必要かどうか考える安易に、 nd しない

テストしたいコード Class User { public function get_image_path() { return 'img/user/'.$this->id.'/main.jpg';
} }

よくあるテスト nd で取ってきてる setUp でDB の初期化が必要になる $user = User::find(1); assertSame('img/user/1/main.jpg'
, $user->get_image_path());

こっちのが速い DB から検索するのではなく引数与えてオブジェクト作る $user = User::forge(['id'=>1]); assertSame('img/user/1/main.jpg' , $user->get_image_path());

フレームワークに喧嘩を売らない提供されている機能をテストする必要はない必要があるのは、ちゃんと設定 or 使えているか例えば論理削除のテスト

users テーブル id name deleted_at 1 山田太郎 null 2 山田花子
2017-06-22 12:00:00

よくあるテスト $actual = User::find(1); assertNotEmpty($actual); $actual = User::find(2); assertEmpty($actual);

こっちのが速い例えば、論理削除のクラスを継承していることを確認するテストにする assertSame('SoftDelete', class_parents(User::class));

Mock を使って遅い処理を避ける時間のかかる処理を毎回にコールする必要はない。ただし、Mock は使うべきときだけ使うようにする。

テストしたいコード AB テスト用に取ってきたユーザーを A グループとB グループに分ける class ServiceUser { public
function get_ab_groups(){ $users = User::get_something(); // 時間 SQL User 配列返 $a = $b = []; foreach($users as $user){ if($user->id % 2 === 0) $a[] = $user; else $b[] = $user; } return [ 'a' => $a, 'b' => $b, ]; } }

こうすると早くはできる $users = array_map(function($i){ return User::forge(['id'=> $i]); }, range(1,2)); $mock
= Mockery::mock('alias:' . User::class); $mock->shouldReceive('get_something')->andReturn($users); $actual = (new ServiceUser())->get_ab_groups(); assertSomething($actual);

TestCase クラスは、2 つに分ける大抵のテスティングフレームワークは、 TestCase クラスのsetUp メソッドでデータの初期化を行うので、 TestCase クラスそのものを分けるようにしておく。
ModelUserTest ModelUserNoDbTest ※ディスクの遅い環境は増やしすぎ注意

3 つだとさらによい ModelUserTest データの更新が発生するため毎回初期化する ModelUserReadTest 読み取りのみ。最初だけ初期化する初期化をsetUp ではなく、setUpBeforeClass 的なとこでやる
ModelUserNoDbTest データベースを全く使わない

まとめ遅いテストは悪テストを走らせる環境のチューニングはする遅い処理は書かないでテストできないか検討する最悪Mock に置き換える

ご静聴ありがとうございました

テストを遅くしないように気をつけること

テストを遅くしないように気をつけること

kkkw

More Decks by kkkw

Other Decks in Programming

Featured

Transcript