継続的にLaravelのUnitTestを書く上で 気をつけていること 2021 年 9 月 28 日 ( 火 )

自己紹介 hyroky.(@handm871) 専門領域： 趣味： バックエンド🍓 ユニットテスト・設計 筋トレ🦾

テストとPHP周りについて ● テスト駆動開発歴：５年 ● PHP歴：５年 ● Laravel歴：３年

テストコードについて思うこと ● サンプルコードが圧倒的に少ない ● プロダクトコードの設計は たくさん ある ○ テストコードの設計は あんまり ない

伝えたい人 テスト駆動開発に ● 興味がある人 ● 初心者

今日話すこと UnitTest について 依存注入できる形で実装する

サンプルコード class FetchTodoService { public function exec(string $user_id, int $todo_id): array { $todo_repos = new TodoRepository(); $result = $todo_repos->fetchTodo($todo_id, $user_id); if ($result === []) { throw new NotFoundException('Todoのデータが存在しません'); } return $result; } }

何がまずいのか？ ● TodoRepositoryに依存している ○ メソッド内で `new` している ○ 代替できないことが問題 ○ DBにテストデータを作る必要がある

Feature Test ● [GET]: /api/todo ● 実データを用いて機能の振る舞いを検証

Unit Test ● TodoListController ○ TodoFetchService ■ TodoRepository

Unit Test TodoListController Jsonを返す FetchTodoService DBから受け取った値を検証する TodoRepository DBへ接続・SQLを実行

Unit Test TodoListController Jsonを返す FetchTodoService DBから受け取った値を検証する TodoRepository DBへ接続・SQLを実行 FetchTodoService はDB接続の責務は不要

変更前 class FetchTodoService { public function exec(string $user_id, int $todo_id): array { $todo_repos = new TodoRepository(); $result = $todo_repos->fetchTodo($todo_id, $user_id); if ($result === []) { throw new NotFoundException('Todoのデータが存在しません'); } return $result; } }

変更後 class FetchTodoService { public function exec( string $user_id, int $todo_id, TodoRepository $todo_repos, ): array { $result = $todo_repos->fetchTodo($todo_id, $user_id); if ($result === []) { throw new LogicException('Todoのデータが存在しません'); } return $result; } } TodoRepository を外部から注入

テストのサンプル class FetchTodoServiceTest extends TestCase { /** @test */ public function データが存在しない場合、空の配列を返すべき (): void { $user_id = 'user_id'; $todo_id = 999999; $todo_repos_mock = Mockery::mock(TodoRepository ::class); $todo_repos_mock ->shouldReceive ('fetchTodo') ->once() ->with($user_id, $todo_id) ->andReturn([]); $service = new TodoService(); parent::expectedException (LogicException ::class); parent::expectExceptionMessage ('Todoのデータが存在しません '); $service->exec($user_id, $todo_id, $todo_repos_mock ); } } 引数を設定 返り値を設定

何がまずいのか？ ● TodoRepositoryに依存している ○ メソッド内で `new` している ○ 代替できないことが問題 ○ DBにテストデータを作る必要がある

テスト駆動開発から見えるもの ● テストを書くことが大変...とおもったら ○ 設計を見直すいい機会

おわり