Merge "resourceloader: Improve test coverage of ResourceLoader::register()"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Fri, 4 May 2018 01:52:36 +0000 (01:52 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Fri, 4 May 2018 01:52:36 +0000 (01:52 +0000)
tests/phpunit/includes/resourceloader/ResourceLoaderTest.php

index 4e9f539..e811d87 100644 (file)
@@ -99,6 +99,22 @@ class ResourceLoaderTest extends ResourceLoaderTestCase {
                $resourceLoader->register( 'test', new stdClass() );
        }
 
+       /**
+        * @covers ResourceLoader::register
+        */
+       public function testRegisterDuplicate() {
+               $logger = $this->getMockBuilder( Psr\Log\LoggerInterface::class )->getMock();
+               $logger->expects( $this->once() )
+                       ->method( 'warning' );
+               $resourceLoader = new EmptyResourceLoader( null, $logger );
+
+               $module1 = new ResourceLoaderTestModule();
+               $module2 = new ResourceLoaderTestModule();
+               $resourceLoader->register( 'test', $module1 );
+               $resourceLoader->register( 'test', $module2 );
+               $this->assertSame( $module2, $resourceLoader->getModule( 'test' ) );
+       }
+
        /**
         * @covers ResourceLoader::getModuleNames
         */