Problem/Motivation
The test Drupal\node\Tests\Views\FrontPageTest in core/modules/node/lib/Drupal/node/tests/views doesn't show in test interface as the class name and the file name case don't match.
FrontPageTest => FrontpageTest.php
Steps to reproduce:
1. Install minimal profile
2. Enable simpletest
3. Go to /admin/config/development/testing
4. Try searching for FrontPageTest in Drupal\node\Tests\Views\ namespace
Proposed resolution
Amend filename to match class name
Remaining tasks
Create patch
User interface changes
API changes
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 2194703-Rename-FrontpageTest.php-04.patch | 315 bytes | Stuart Miller |
| #1 | 2194703-rename-FrontPageTest-01.patch | 536 bytes | Stuart Miller |
Comments
Comment #1
Stuart Miller commentedPatch to change class name to match filename as Git doesn't pickup filname case changes. Can be kind of made to but doesn't seem very reliable.
Comment #2
dawehnerGiven that this might result in a non-running test (not sure) I would consider this as at least major.
Comment #3
star-szrI mentioned on IRC that if we change the class name instead of the file name we should change the @file as well.
Comment #4
Stuart Miller commentedThis patch file will change FrontpageTest.php to FrontPageTest.php.
It needs to be applied on an operating system with a case sensitive filesystem.
Another issue I ran into was applying the patch in a Debian virtual machine where my files were in a folder mounted via a NFS share, which is not case sensitive.
Comment #5
Stuart Miller commentedComment #6
Stuart Miller commentedError in uploading previous patch file, this is the correct one.
Comment #7
star-szrGreat, thanks!
Comment #8
alexpottCommitted a1c6a9d and pushed to 8.x. Thanks!