There are three test files with exactly the same name. Whilst this does not break anything it can make debugging test failures unnecessarily complicated.
The files are:
- core/modules/system/lib/Drupal/system/Tests/System/ThemeTest.php
- core/modules/system/lib/Drupal/system/Tests/Theme/ThemeTest.php
- core/modules/taxonomy/lib/Drupal/taxonomy/Tests/ThemeTest.php
The output of simpletest has the following format - whilst the function has enough info to find the test the when the filename is the same things aren't as clear as they should be.
| Message | Group | Filename | Line | Function | Status | |
|---|---|---|---|---|---|---|
| Enabled modules: theme_test | Other | ThemeTest.php | 26 | Drupal\system\Tests\Theme\ThemeTest->setUp() | ||
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 1717914-1.drupal8.test-names.patch | 42.01 KB | alexpott |
Comments
Comment #1
alexpottThe attached patch renames the classes/files
Comment #2
tim.plunkettI'd like to mark this "won't fix", but let's have some discussion first.
There is no reason to have "Drupal\taxonomy\Tests\TaxonomyThemeTest", it's redundant to have "taxonomy" in there twice.
And I don't think ThemeInterfaceTest or ThemeAPITest are any more descriptive, other than to just be different from each other.
Comment #3
xjm#1: 1717914-1.drupal8.test-names.patch queued for re-testing.
Comment #16
quietone commentedThere has been no further discussion here in 10 years and the testing environment has moved to PHPUnit.
Although #2 recommended won't fix, I am marking this outdated.
Thanks!