こんなPHPUnitのレビューはいやだと言われぬために by しまぶ

テストコードのプルリクが来て意気揚々と見てみると、こんなコードが書かれていたりしませんか？

• 何を表明するテストなのか分かりづらい
• なんでもかんでもデータプロバイダーだ
• 条件分岐が入っている
• やたらDRYだ
• 準備、実行、確認が無秩序に書かれている
• etc

こういったテストがレビューで飛んできたとき、見るのがしんどそう...と、自分はまず別タブに移動してしまいます。
レビューはすぐにしたいんです。それなのになぜネガティブな感じから始まるのでしょう？もちろん理由はあるのです。

そこで本トークでは、上記テストコードに対する問題点と解決策についてコード例を交えつつ説明し、
こんなPHPUnitのレビューはいやだと言われぬために僕らが今日からできることをお話したいと思います。
レビュアーが嫌がるテストコードを知ることで、レビュアーに優しいテストコードを書いていきましょう！