The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.The naming convention for AllTests classes and files - module_AllTests - causes problems with the class autoloader. The phpunit command line test runner looks for the module_AllTests class code file in module/AllTests.php, not module_AllTests.php. As such, we should change this naming convention to remove the underscore.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | phpunit-alltests.patch | 2.67 KB | chop |
| #1 | phpunit-alltests.patch | 1.92 KB | chop |
Comments
Comment #1
chop commentedI've put together a patch that changes the file mask for AllTests test suites. It also removes deprecated code initially used to run the test suites without calling the phpunit command line test runner first.
Comment #2
chop commentedStatus change.
Comment #3
chop commentedOK. Great response from all the PHPUnit users out there. :)
I've made a couple more changes and as such have created a new patch. This patch, in addition to what changes we'd already made, adds a bit more explanation of how we're adding the tests to the test suite. Specifically, I've added a couple more comments and separated out the code that deduces the class name from the All Tests suite path. I've also added a standards complaint Drupal code comment block for the Doxygen 'file' command.
Use this patch rather than the previous one.
Comment #4
chop commentedThis patch applied to CVS at r1.3