This PR can be summarized in the following changelog entry:
N/A
Relevant technical choices:
Looks like PR #142 inadvertently disabled the unit tests as the test files are no longer PSR-4 compliant and the name suffix check done by PHPUnit may or may not be case-sensitive. Mea culpa.
These minor changes should re-enable them.
Note: The acf.php dummy file is explicitly excluded from being autoloaded as the Yoast_ACF_Analysis_Dependency_ACF class does a class_exists() without setting the $autoload parameter to false.
As this may actually be advantageous in a situation were WP plugins are being installed via Composer, I've elected not to change that function call, but to exclude the test dummy file from the autoload class map instead.
Test instructions
This PR can be tested by following these steps:
Check the run logs for the Travis builds to see that the unit tests are being run again and/or try running the unit tests locally.
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
Looks like PR #142 inadvertently disabled the unit tests as the test files are no longer PSR-4 compliant and the name suffix check done by PHPUnit may or may not be case-sensitive. Mea culpa.
These minor changes should re-enable them.
Note: The
acf.php
dummy file is explicitly excluded from being autoloaded as theYoast_ACF_Analysis_Dependency_ACF
class does aclass_exists()
without setting the$autoload
parameter tofalse
. As this may actually be advantageous in a situation were WP plugins are being installed via Composer, I've elected not to change that function call, but to exclude the test dummy file from the autoload class map instead.Test instructions
This PR can be tested by following these steps: