123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194 |
- <?php
- /**
- * @coversNothing
- */
- class AutoLoaderStructureTest extends MediaWikiTestCase {
- /**
- * Assert that there were no classes loaded that are not registered with the AutoLoader.
- *
- * For example foo.php having class Foo and class Bar but only registering Foo.
- * This is important because we should not be relying on Foo being used before Bar.
- */
- public function testAutoLoadConfig() {
- $results = self::checkAutoLoadConf();
- $this->assertEquals(
- $results['expected'],
- $results['actual']
- );
- }
- public function providePSR4Completeness() {
- foreach ( AutoLoader::$psr4Namespaces as $prefix => $dir ) {
- foreach ( $this->recurseFiles( $dir ) as $file ) {
- yield [ $prefix, $dir, $file ];
- }
- }
- }
- private function recurseFiles( $dir ) {
- return ( new File_Iterator_Facade() )->getFilesAsArray( $dir, [ '.php' ] );
- }
- /**
- * @dataProvider providePSR4Completeness
- */
- public function testPSR4Completeness( $prefix, $dir, $file ) {
- global $wgAutoloadLocalClasses, $wgAutoloadClasses;
- $contents = file_get_contents( $file );
- list( $classesInFile, $aliasesInFile ) = self::parseFile( $contents );
- $classes = array_keys( $classesInFile );
- if ( $classes ) {
- $this->assertCount( 1, $classes,
- "Only one class per file in PSR-4 autoloaded classes ($file)" );
- // 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 = substr( substr( $file, strlen( $dir ) ), 0, -4 );
- $expectedClassName = $prefix . str_replace( '/', '\\', $abbrFileName );
- $this->assertSame(
- $expectedClassName,
- $classes[0],
- "Class not autoloaded properly"
- );
- } else {
- // Dummy assertion so this test isn't marked in risky
- // if the file has no classes nor aliases in it
- $this->assertCount( 0, $classes );
- }
- if ( $aliasesInFile ) {
- $otherClasses = $wgAutoloadLocalClasses + $wgAutoloadClasses;
- foreach ( $aliasesInFile as $alias => $class ) {
- $this->assertArrayHasKey( $alias, $otherClasses,
- 'Alias must be in the classmap autoloader'
- );
- }
- }
- }
- private static function parseFile( $contents ) {
- // We could use token_get_all() here, but this is faster
- // Note: Keep in sync with ClassCollector
- $matches = [];
- preg_match_all( '/
- ^ [\t ]* (?:
- (?:final\s+)? (?:abstract\s+)? (?:class|interface|trait) \s+
- (?P<class> [a-zA-Z0-9_]+)
- |
- class_alias \s* \( \s*
- ([\'"]) (?P<original> [^\'"]+) \g{-2} \s* , \s*
- ([\'"]) (?P<alias> [^\'"]+ ) \g{-2} \s*
- \) \s* ;
- |
- class_alias \s* \( \s*
- (?P<originalStatic> [a-zA-Z0-9_]+)::class \s* , \s*
- ([\'"]) (?P<aliasString> [^\'"]+ ) \g{-2} \s*
- \) \s* ;
- )
- /imx', $contents, $matches, PREG_SET_ORDER );
- $namespaceMatch = [];
- preg_match( '/
- ^ [\t ]*
- namespace \s+
- ([a-zA-Z0-9_]+(\\\\[a-zA-Z0-9_]+)*)
- \s* ;
- /imx', $contents, $namespaceMatch );
- $fileNamespace = $namespaceMatch ? $namespaceMatch[1] . '\\' : '';
- $classesInFile = [];
- $aliasesInFile = [];
- foreach ( $matches as $match ) {
- if ( !empty( $match['class'] ) ) {
- // 'class Foo {}'
- $class = $fileNamespace . $match['class'];
- $classesInFile[$class] = true;
- } else {
- if ( !empty( $match['original'] ) ) {
- // 'class_alias( "Foo", "Bar" );'
- $aliasesInFile[$match['alias']] = $match['original'];
- } else {
- // 'class_alias( Foo::class, "Bar" );'
- $aliasesInFile[$match['aliasString']] = $fileNamespace . $match['originalStatic'];
- }
- }
- }
- return [ $classesInFile, $aliasesInFile ];
- }
- protected static function checkAutoLoadConf() {
- global $wgAutoloadLocalClasses, $wgAutoloadClasses, $IP;
- // wgAutoloadLocalClasses has precedence, just like in includes/AutoLoader.php
- $expected = $wgAutoloadLocalClasses + $wgAutoloadClasses;
- $actual = [];
- $files = array_unique( $expected );
- foreach ( $files as $class => $file ) {
- // Only prefix $IP if it doesn't have it already.
- // Generally local classes don't have it, and those from extensions and test suites do.
- if ( substr( $file, 0, 1 ) != '/' && substr( $file, 1, 1 ) != ':' ) {
- $filePath = "$IP/$file";
- } else {
- $filePath = $file;
- }
- if ( !file_exists( $filePath ) ) {
- $actual[$class] = "[file '$filePath' does not exist]";
- continue;
- }
- Wikimedia\suppressWarnings();
- $contents = file_get_contents( $filePath );
- Wikimedia\restoreWarnings();
- if ( $contents === false ) {
- $actual[$class] = "[couldn't read file '$filePath']";
- continue;
- }
- list( $classesInFile, $aliasesInFile ) = self::parseFile( $contents );
- foreach ( $classesInFile as $className => $ignore ) {
- $actual[$className] = $file;
- }
- // Only accept aliases for classes in the same file, because for correct
- // behavior, all aliases for a class must be set up when the class is loaded
- // (see <https://bugs.php.net/bug.php?id=61422>).
- foreach ( $aliasesInFile as $alias => $class ) {
- if ( isset( $classesInFile[$class] ) ) {
- $actual[$alias] = $file;
- } else {
- $actual[$alias] = "[original class not in $file]";
- }
- }
- }
- return [
- 'expected' => $expected,
- 'actual' => $actual,
- ];
- }
- public function testAutoloadOrder() {
- $path = realpath( __DIR__ . '/../../..' );
- $oldAutoload = file_get_contents( $path . '/autoload.php' );
- $generator = new AutoloadGenerator( $path, 'local' );
- $generator->setExcludePaths( array_values( AutoLoader::getAutoloadNamespaces() ) );
- $generator->initMediaWikiDefault();
- $newAutoload = $generator->getAutoload( 'maintenance/generateLocalAutoload.php' );
- $this->assertEquals( $oldAutoload, $newAutoload, 'autoload.php does not match' .
- ' output of generateLocalAutoload.php script.' );
- }
- }
|