継続的にLaravelのUnitTestを書く上で 気をつけていること

Transcript

1. 継続的にLaravelのUnitTestを書く上で 気をつけていること 2021 年 9 月 28 日 ( 火 )
2. 自己紹介 hyroky.(@handm871) 専門領域： 趣味： バックエンド🍓 ユニットテスト・設計 筋トレ🦾
3. テストとPHP周りについて • テスト駆動開発歴：５年 • PHP歴：５年 • Laravel歴：３年
4. テストコードについて思うこと • サンプルコードが圧倒的に少ない • プロダクトコードの設計は たくさん ある ◦ テストコードの設計は あんまり ない
5. 伝えたい人 テスト駆動開発に • 興味がある人 • 初心者
6. 今日話すこと UnitTest について 依存注入できる形で実装する
7. サンプルコード 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; } }
8. 何がまずいのか？ • TodoRepositoryに依存している ◦ メソッド内で `new` している ◦ 代替できないことが問題 ◦ DBにテストデータを作る必要がある
9. Feature Test • [GET]: /api/todo • 実データを用いて機能の振る舞いを検証
10. Unit Test • TodoListController ◦ TodoFetchService ▪ TodoRepository
11. Unit Test TodoListController Jsonを返す FetchTodoService DBから受け取った値を検証する TodoRepository DBへ接続・SQLを実行
12. Unit Test TodoListController Jsonを返す FetchTodoService DBから受け取った値を検証する TodoRepository DBへ接続・SQLを実行 FetchTodoService はDB接続の責務は不要
13. 変更前 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; } }
14. 変更後 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 を外部から注入
15. テストのサンプル 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 ); } } 引数を設定 返り値を設定
16. 何がまずいのか？ • TodoRepositoryに依存している ◦ メソッド内で `new` している ◦ 代替できないことが問題 ◦ DBにテストデータを作る必要がある
17. テスト駆動開発から見えるもの • テストを書くことが大変...とおもったら ◦ 設計を見直すいい機会
18. おわり