ResourceLoader::register() works differently since r77011.
authorPlatonides <platonides@users.mediawiki.org>
Thu, 30 Dec 2010 16:16:16 +0000 (16:16 +0000)
committerPlatonides <platonides@users.mediawiki.org>
Thu, 30 Dec 2010 16:16:16 +0000 (16:16 +0000)
This test isn't useful anymore.

tests/phpunit/includes/ResourceLoaderTest.php

index 8de178a..76f0ba8 100644 (file)
@@ -46,20 +46,6 @@ class ResourceLoaderTest extends PHPUnit_Framework_TestCase {
                $resourceLoader->register( $name, $module );
                $this->assertEquals( $module, $resourceLoader->getModule( $name ) );
        }
-
-       /**
-        * Allthough ResourceLoader::register uses type hinting to prevent arbitrary information being passed through as a
-        * ResourceLoaderModule object, null can still get through.
-        *
-        * @depends testCreatingNewResourceLoaderCallsRegistrationHook
-        * @covers ResourceLoader::register
-        * @covers ResourceLoader::getModule
-        * @expectedException MWException
-        */
-       public function testRegisteringNullModuleThrowsAnException( ResourceLoader $resourceLoader ) {
-               $this->markTestIncomplete( "Broken by r77011" );
-               $resourceLoader->register( 'TEST.nullModule', null );
-       }
 }
 
 /* Stubs */
@@ -68,4 +54,4 @@ class ResourceLoaderTestModule extends ResourceLoaderModule { }
 
 /* Hooks */
 global $wgHooks;
-$wgHooks['ResourceLoaderRegisterModules'][] = 'ResourceLoaderTest::resourceLoaderRegisterModules';
\ No newline at end of file
+$wgHooks['ResourceLoaderRegisterModules'][] = 'ResourceLoaderTest::resourceLoaderRegisterModules';