PHPカンファレンス福岡2023
レギュラートーク(15分)

こんなテストケース(PHPUnit)はいやだ

shimabox しまぶ shimabox
4

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

  • テストケース名が test_executeメソッド() としか書かれていない
  • ひとつのテストケースの中で、複数のシナリオが確認されている
  • なんでもかんでもデータプロバイダー
  • テストケースの中にif文がやたら書かれている
  • ひとつのテストケースを確認するのに、大量のデータセットが必要
  • etc

こういったテストコードがレビューで飛んできたとき、自分はちょっといやだなぁと感じてしまいます。

本トークでは、なぜ上記のようなテストコードをいやだなぁと感じてしまうのか、何が問題になるのか、それに対してどのように書き換えればいいのかをコード例を交えながらお話しできればと思います。