3
1

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 1 year has passed since last update.

「 PHP に tests ディレクトリっていらなくない?」と思ってみた

Posted at

こんにちは。やまゆです。

ということで疑問に思ってみました。

Why?

ソースコードディレクトリとテストディレクトリを分けるのが PHP の一般的な習慣です。

/
  -> src/
    -> Domain/
      -> User/
        -> UserEntity.php
  -> tests/
    -> Unit/
      -> Domain/
        -> User/
          -> UserEntityTest.php

ユニットテストはクラス単位で行うことが多いため、このような構成になることがあるでしょう。

そうすると、同じディレクトリ構成が現れることになります。何が DRY か?状態です。

例えば golang は超シンプルです。

/
  -> user.go
  -> user_test.go

これで go test すると、自動的にテストファイルを判別してくれます。

/
  -> src/
    -> Domain/
      -> User/
        -> UserEntity.php
        -> UserEntityTest.php

つまり、これで良くない?という話です。

(もちろん、クラスをまたいだもう少し大きな IntegrationTest などは別途 integrations 等を用意する必要があると思います)

メリット

  1. ディレクトリに対して DRY を適用できる
    • DRY はみんな大好きだと思います。同じことは繰り返したくないです。
  2. 実装とテストがそばにあるため行き来しやすい
    • 実装する時はテストファイルと同時に開いて編集することが多いですよね
  3. テストがちゃんと書かれているかどうかがわかる
    • テスト書き忘れ、レビュー漏れがあると厄介です

デメリット

  1. ランタイム実装にテスト用のコードが潜り込む危険がある
    • 同じ場所にあるので、テストユーザーを作成するメソッドを間違えて本番環境向けの実装で使ってしまうかもしれません
  2. 対応していないライブラリがあるかもしれない
    • PHP の現在のエコシステム的に、ディレクトリを分離する前提のライブラリがあってもおかしくありません
  3. 同じディレクトリのファイル数が倍増する
    • 設計次第ですが、同じドメイン領域に存在するファイルが倍になります。閲覧性に問題が出るかもしれません

ということで、ちゃんと phpunit/psalm/phpstan/php-cs-fixer あたりの著名なライブラリがこの構成でも動くか検証してみました。

結論: いけそう

もちろん上記で出した以外のデメリットがあると思います。

しかし、なんかこっちの方が良いかもしれないな、というのが今日の感触でした。

3
1
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
3
1

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?