新しい職場で提案したら歓迎されたので投稿しておく。

テストコード開発方針

漫然とテストコードを書いていると、以下のような問題が発生することがある。

このような問題を避けるため、以下の方針を定める。

ビジネス上の価値に比例したテスト

コードの価値をビジネスへの影響や回避方法の有無により以下のようにランク付けする。

複数のランクに該当する場合、より多く該当するランクに分類する。

テスト対象がメジャーに該当する場合、より大量の／時間のかかるのテストが許容される。マイナーに該当する場合、最低限の正常系のテストが許容される。トリビアルの場合、テストは書かない。

インテグレーションテストの対象は以下のものに限る：

なお、ViewにはJavaScriptを含む。以下に具体例を挙げる：

ただし、インテグレーションテストは低速なので、各対象につきテストは１回に抑える。値を変えた網羅的なテストはユニットテストで行うこと。

Controllerのテストを書く場合、その対象は以下のものに限る:

Controllerが上記以外の処理をしている場合は、その処理を以下の手順で別のクラスに展開する：

Controllerが生成しViewで利用するインスタンスのテストは、必要ならインテグレーションテストにする（assignsとassert_templateはDeprecatedです。

privateメソッドのテストは以下の理由で書くべきでない：

テスト対象外のオブジェクトが必要な場合はスタブやモックを利用する。

ただし、ControllerがModelのscopeをチェインして利用する場合など、複数の解法がある場合は実オブジェクトを使ってよい。

以上の方針よりも開発チーム内でのコミュニケーションが優先される。