Enhancement: Declare test code in separate directory #5
Conversation
localheinz
force-pushed
the
feature/separate-directory
branch
2 times, most recently
from
a521960 to
917a413
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5 +/- ##
========================================
Coverage ? 100.00%
Complexity ? 8
========================================
Files ? 3
Lines ? 16
Branches ? 0
========================================
Hits ? 16
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Sentry. |
localheinz
force-pushed
the
feature/separate-directory
branch
3 times, most recently
from
a9eab03 to
4f5af4c
Compare
| @@ -1,5 +1,6 @@ | |||
| /.github/ export-ignore | |||
| /.phive/ export-ignore | |||
| /test/ export-ignore | |||
There was a problem hiding this comment.
When you work on a package and declare test code in a separate directory, adding exclusions for test code in .gitattributes is less work.
| "autoload-dev": { | ||
| "psr-4": { | ||
| "Localheinz\\OrganizingTestCodeInPhp\\Test\\": "test/" | ||
| } | ||
| }, |
There was a problem hiding this comment.
When you declare test code in a separate directory, managing exclusions for the composer autoloader in composer.json is unnecessary. Instead, you configure and document namespaces for test code via an autoloader configuration in the autoload-dev section.
| "directories": [ | ||
| "src" | ||
| ] |
There was a problem hiding this comment.
When you declare test code in a separate directory, managing exclusions for infection/infection in infection.json is unnecessary.
phpunit.xml
Outdated
| <include> | ||
| <directory suffix=".php">src/</directory> | ||
| </include> |
There was a problem hiding this comment.
When you declare test code in a separate directory, managing exclusions for collecting code coverage with phpunit/phpunit in phpunit.xml is unnecessary.
localheinz
force-pushed
the
feature/separate-directory
branch
2 times, most recently
from
ef027e9 to
e657c81
Compare
localheinz
force-pushed
the
feature/separate-directory
branch
2 times, most recently
from
e4cc495 to
648de2f
Compare
localheinz
force-pushed
the
feature/separate-directory
branch
from
648de2f to
d4ae561
Compare
This pull request
test/