From afb2269cb95d1c09291515c5f72c4e425ec999ac Mon Sep 17 00:00:00 2001 From: Hashar Date: Mon, 25 Jun 2018 12:38:07 +0000 Subject: [PATCH] Revert "Use pathinfo() in AutoLoaderStructureTest::testPSR4Completeness" This reverts commit 634c2ec2afd93827e5aaf571396eb76d35948188. Reason for revert: that strips the PSR4 directories from the class. Example: $dir BlueSpiceFoundation/src/" $file BlueSpiceFoundation/src/ConfigDefinition/IntSetting.php $abbrFileName: IntSetting $expectedClassName: BlueSpice\IntSetting $abbrFileName should be relative to $dir and not just the filename. Bug: T198077 Change-Id: Ie934e309fee0392439b4e26d86249f0650e5ea67 --- tests/phpunit/structure/AutoLoaderStructureTest.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/phpunit/structure/AutoLoaderStructureTest.php b/tests/phpunit/structure/AutoLoaderStructureTest.php index 7163916225..2800d021f8 100644 --- a/tests/phpunit/structure/AutoLoaderStructureTest.php +++ b/tests/phpunit/structure/AutoLoaderStructureTest.php @@ -46,7 +46,7 @@ class AutoLoaderStructureTest extends MediaWikiTestCase { // Check that the expected class name (based on the filename) is the // same as the one we found. // Strip directory prefix from front of filename, and .php extension - $abbrFileName = pathinfo( $file, PATHINFO_FILENAME ); + $abbrFileName = substr( substr( $file, strlen( $dir ) ), 0, -4 ); $expectedClassName = $prefix . str_replace( '/', '\\', $abbrFileName ); $this->assertSame( -- 2.20.1