どのようなメソッドに対してテストコードを用意すればいいのでしょうか？

Question

PHPUnit を使ったテストを導入してみようと思うのですが独学なため実際の仕事での常識的にテストする対象の決め方をよく知りません。
あらゆるクラスのすべてのメソッドをテストできれば安全ですが実務上でそれが本当に現実的なのかを知りたいです。

例として記事を登録できるCRUDアプリケーションがあるとします。
モデルは次のような構造があるとします。

class Post

メソッド	動作
getContent()	本文を取得
setContent()	本文をセット

コントローラーは次のメソッドを持ちます。

class PostController

メソッド	動作
index()	一覧の表示
show()	記事単体の表示
create()	新規作成フォームの表示
edit()	編集フォームの表示
store()	新規保存処理
update()	更新処理
destroy()	削除処理

データベース接続やレコード登録削除などの動作はフレームワークに用意されているテスト済みのものを使います。（Laravelなどを想定しています）

コントローラーの index() であればビューのレスポンスがオブジェクトで返るので assertInstanceOf() で調べられそうにみえます。
Post のセッターメソッドは特に何も返さないのでセット後にプロパティが変化したことを確かめれば良さそうです。

具体的なテストコードの書き方は後で調べるとして、どのメソッドがどのように動作することを確かめたら「テストした」と言えるのでしょうか？ あるいは複雑なメソッドなど特別なものだけをテストしたら十分なのでしょうか？

判断基準やどの程度細かく調べるのが常識かなどありましたら教えていただきたいと思います。よろしくお願いいたします。

Comments

[m.ts10806]

既に出ている回答で充分と私は思ったのですが、他にどのような回答が欲しいのでしょうか？

Answer 1

何を目的としてユニットテストを書くか次第でどこまで書く必要があるか変わるので、まずはそれを決めましょう。

ユニットテストの基礎
の3. ユニットテストの目的あたりですね。

個人的な感想としては、ユニットテストが採用されるプロジェクトにおいては
全publicメソッドについて、正常系異常系ともに全てテストを書くことでユニットテストの効果が最大限に発揮されると考えています。
（どうせ後でテストするのだから、テストコードに落としておいた方があらゆる点で楽という理由。）

Answer 2

私も仕事でテストコードは書いていますが、全部のメソッドには作っていません。
開発期間も限られているので、「ここは見ておきたい」という重要なメソッドに絞ってテストコードを作る事が殆どです。

Comments

[workr]

回答ありがとうございます。例で出しているような基本的な項目の登録・削除の程度であればテストコードを作るほど重要なメソッドはないと考えていいでしょうか？

[tamina]

そういう意味ではありません。
処理の複雑さなど、バグの出やすいところはコーディングした人が見当が付くはずなので、
自分で判断するのが良いと思います。