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

Comments

Stuart Miller’s picture

StatusFileSize
new536 bytes

Patch 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.

dawehner’s picture

Priority: Normal » Major
Status: Active » Reviewed & tested by the community

Given that this might result in a non-running test (not sure) I would consider this as at least major.

star-szr’s picture

Status: Reviewed & tested by the community » Needs review

I mentioned on IRC that if we change the class name instead of the file name we should change the @file as well.

Stuart Miller’s picture

StatusFileSize
new0 bytes

This 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.

Stuart Miller’s picture

Stuart Miller’s picture

StatusFileSize
new315 bytes

Error in uploading previous patch file, this is the correct one.

star-szr’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix

Great, thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed a1c6a9d and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.