ResourceLoader namespace

Move ResourceLoader classes to their own namespace. Strip the
"ResourceLoader" prefix from all except ResourceLoader and
ResourceLoaderContext.

Move the tests by analogy.

I used a namespace alias "RL" in some callers since RL\Module is less
ambiguous at the call site than just "Module".

I did not address DependencyStore which continues to have a non-standard
location and namespace.

Change-Id: I92998ae6a82e0b935c13e02a183e7c324fa410a3

1 files changed, 0 insertions, 969 deletions

diff --git a/tests/phpunit/includes/resourceloader/ResourceLoaderStartUpModuleTest.php b/tests/phpunit/includes/resourceloader/ResourceLoaderStartUpModuleTest.php
deleted file mode 100644
index bf9ce370d9a3..000000000000
--- a/tests/phpunit/includes/resourceloader/ResourceLoaderStartUpModuleTest.php
+++ /dev/null
@@ -1,969 +0,0 @@
-<?php
-
-class ResourceLoaderStartUpModuleTest extends ResourceLoaderTestCase {
-
-	protected static function expandPlaceholders( $text ) {
-		return strtr( $text, [
-			'{blankVer}' => self::BLANK_VERSION
-		] );
-	}
-
-	public function provideGetModuleRegistrations() {
-		return [
-			[ [
-				'msg' => 'Empty registry',
-				'modules' => [],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([]);'
-			] ],
-			[ [
-				'msg' => 'Basic registry',
-				'modules' => [
-					'test.blank' => [ 'class' => ResourceLoaderTestModule::class ],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        ""
-    ]
-]);',
-			] ],
-			[ [
-				'msg' => 'Optimise the dependency tree (basic case)',
-				'modules' => [
-					'a' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'b', 'c', 'd' ],
-					],
-					'b' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'c' ],
-					],
-					'c' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [],
-					],
-					'd' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [],
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "a",
-        "",
-        [
-            1,
-            3
-        ]
-    ],
-    [
-        "b",
-        "",
-        [
-            2
-        ]
-    ],
-    [
-        "c",
-        ""
-    ],
-    [
-        "d",
-        ""
-    ]
-]);',
-			] ],
-			[ [
-				'msg' => 'Optimise the dependency tree (tolerate unknown deps)',
-				'modules' => [
-					'a' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'b', 'c', 'x' ]
-					],
-					'b' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'c', 'x' ]
-					],
-					'c' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => []
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "a",
-        "",
-        [
-            1,
-            "x"
-        ]
-    ],
-    [
-        "b",
-        "",
-        [
-            2,
-            "x"
-        ]
-    ],
-    [
-        "c",
-        ""
-    ]
-]);',
-			] ],
-			[ [
-				// Regression test for T223402.
-				'msg' => 'Optimise the dependency tree (indirect circular dependency)',
-				'modules' => [
-					'top' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'middle1', 'util' ],
-					],
-					'middle1' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'middle2', 'util' ],
-					],
-					'middle2' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'bottom' ],
-					],
-					'bottom' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'top' ],
-					],
-					'util' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [],
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "top",
-        "",
-        [
-            1,
-            4
-        ]
-    ],
-    [
-        "middle1",
-        "",
-        [
-            2,
-            4
-        ]
-    ],
-    [
-        "middle2",
-        "",
-        [
-            3
-        ]
-    ],
-    [
-        "bottom",
-        "",
-        [
-            0
-        ]
-    ],
-    [
-        "util",
-        ""
-    ]
-]);',
-			] ],
-			[ [
-				// Regression test for T223402.
-				'msg' => 'Optimise the dependency tree (direct circular dependency)',
-				'modules' => [
-					'top' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [ 'util', 'top' ],
-					],
-					'util' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [],
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "top",
-        "",
-        [
-            1,
-            0
-        ]
-    ],
-    [
-        "util",
-        ""
-    ]
-]);',
-			] ],
-			[ [
-				'msg' => 'Group signature',
-				'modules' => [
-					'test.blank' => [ 'class' => ResourceLoaderTestModule::class ],
-					'test.group.foo' => [
-						'class' => ResourceLoaderTestModule::class,
-						'group' => 'x-foo',
-					],
-					'test.group.bar' => [
-						'class' => ResourceLoaderTestModule::class,
-						'group' => 'x-bar',
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        ""
-    ],
-    [
-        "test.group.foo",
-        "",
-        [],
-        2
-    ],
-    [
-        "test.group.bar",
-        "",
-        [],
-        3
-    ]
-]);'
-			] ],
-			[ [
-				'msg' => 'Different target (non-test should not be registered)',
-				'modules' => [
-					'test.blank' => [ 'class' => ResourceLoaderTestModule::class ],
-					'test.target.foo' => [
-						'class' => ResourceLoaderTestModule::class,
-						'targets' => [ 'x-foo' ],
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        ""
-    ]
-]);'
-			] ],
-			[ [
-				'msg' => 'Different skin (irrelevant skin modules should not be registered)',
-				'modules' => [
-					'test.blank' => [ 'class' => ResourceLoaderTestModule::class ],
-					'test.skin.fallback' => [
-						'class' => ResourceLoaderTestModule::class,
-						'skins' => [ 'fallback' ],
-					],
-					'test.skin.foo' => [
-						'class' => ResourceLoaderTestModule::class,
-						'skins' => [ 'foo' ],
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        ""
-    ],
-    [
-        "test.skin.fallback",
-        ""
-    ]
-]);'
-			] ],
-			[ [
-				'msg' => 'Safemode disabled (default; register all modules)',
-				'modules' => [
-					// Default origin: ORIGIN_CORE_SITEWIDE
-					'test.blank' => [ 'class' => ResourceLoaderTestModule::class ],
-					'test.core-generated' => [
-						'class' => ResourceLoaderTestModule::class,
-						'origin' => ResourceLoaderModule::ORIGIN_CORE_INDIVIDUAL
-					],
-					'test.sitewide' => [
-						'class' => ResourceLoaderTestModule::class,
-						'origin' => ResourceLoaderModule::ORIGIN_USER_SITEWIDE
-					],
-					'test.user' => [
-						'class' => ResourceLoaderTestModule::class,
-						'origin' => ResourceLoaderModule::ORIGIN_USER_INDIVIDUAL
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        ""
-    ],
-    [
-        "test.core-generated",
-        ""
-    ],
-    [
-        "test.sitewide",
-        ""
-    ],
-    [
-        "test.user",
-        ""
-    ]
-]);'
-			] ],
-			[ [
-				'msg' => 'Safemode enabled (filter modules with user/site origin)',
-				'extraQuery' => [ 'safemode' => '1' ],
-				'modules' => [
-					// Default origin: ORIGIN_CORE_SITEWIDE
-					'test.blank' => [ 'class' => ResourceLoaderTestModule::class ],
-					'test.core-generated' => [
-						'class' => ResourceLoaderTestModule::class,
-						'origin' => ResourceLoaderModule::ORIGIN_CORE_INDIVIDUAL
-					],
-					'test.sitewide' => [
-						'class' => ResourceLoaderTestModule::class,
-						'origin' => ResourceLoaderModule::ORIGIN_USER_SITEWIDE
-					],
-					'test.user' => [
-						'class' => ResourceLoaderTestModule::class,
-						'origin' => ResourceLoaderModule::ORIGIN_USER_INDIVIDUAL
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        ""
-    ],
-    [
-        "test.core-generated",
-        ""
-    ]
-]);'
-			] ],
-			[ [
-				'msg' => 'Foreign source',
-				'sources' => [
-					'example' => [
-						'loadScript' => 'http://example.org/w/load.php',
-						'apiScript' => 'http://example.org/w/api.php',
-					],
-				],
-				'modules' => [
-					'test.blank' => [
-						'class' => ResourceLoaderTestModule::class,
-						'source' => 'example'
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php",
-    "example": "http://example.org/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        "",
-        [],
-        null,
-        "example"
-    ]
-]);'
-			] ],
-			[ [
-				'msg' => 'Conditional dependency function',
-				'modules' => [
-					'test.x.core' => [ 'class' => ResourceLoaderTestModule::class ],
-					'test.x.polyfill' => [
-						'class' => ResourceLoaderTestModule::class,
-						'skipFunction' => 'return true;'
-					],
-					'test.y.polyfill' => [
-						'class' => ResourceLoaderTestModule::class,
-						'skipFunction' =>
-							'return !!(' .
-							'    window.JSON &&' .
-							'    JSON.parse &&' .
-							'    JSON.stringify' .
-							');'
-					],
-					'test.z.foo' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [
-							'test.x.core',
-							'test.x.polyfill',
-							'test.y.polyfill',
-						],
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.x.core",
-        ""
-    ],
-    [
-        "test.x.polyfill",
-        "",
-        [],
-        null,
-        null,
-        "return true;"
-    ],
-    [
-        "test.y.polyfill",
-        "",
-        [],
-        null,
-        null,
-        "return !!(    window.JSON \u0026\u0026    JSON.parse \u0026\u0026    JSON.stringify);"
-    ],
-    [
-        "test.z.foo",
-        "",
-        [
-            0,
-            1,
-            2
-        ]
-    ]
-]);',
-			] ],
-			[ [
-				'msg' => 'ES6-only module',
-				'modules' => [
-					'test.es6' => [
-						'class' => ResourceLoaderTestModule::class,
-						'es6' => true
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.es6",
-        "!"
-    ]
-]);',
-			] ],
-			[ [
-				'msg' => 'noscript group omitted (T291735)',
-				'modules' => [
-					'test.not-noscript' => [
-						'class' => ResourceLoaderTestModule::class,
-					],
-					'test.noscript' => [
-						'class' => ResourceLoaderTestModule::class,
-						'group' => 'noscript',
-					],
-					'test.also-noscript' => [
-						'class' => ResourceLoaderTestModule::class,
-						'group' => 'noscript',
-					],
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.not-noscript",
-        ""
-    ]
-]);',
-			] ],
-			[ [
-				// This may seem like an edge case, but a plain MediaWiki core install
-				// with a few extensions installed is likely far more complex than this
-				// even, not to mention an install like Wikipedia.
-				// TODO: Make this even more realistic.
-				'msg' => 'Advanced (everything combined)',
-				'sources' => [
-					'example' => [
-						'loadScript' => 'http://example.org/w/load.php',
-						'apiScript' => 'http://example.org/w/api.php',
-					],
-				],
-				'modules' => [
-					'test.blank' => [ 'class' => ResourceLoaderTestModule::class ],
-					'test.x.core' => [ 'class' => ResourceLoaderTestModule::class ],
-					'test.x.util' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [
-							'test.x.core',
-						],
-					],
-					'test.x.foo' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [
-							'test.x.core',
-						],
-					],
-					'test.x.bar' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [
-							'test.x.core',
-							'test.x.util',
-						],
-					],
-					'test.x.quux' => [
-						'class' => ResourceLoaderTestModule::class,
-						'dependencies' => [
-							'test.x.foo',
-							'test.x.bar',
-							'test.x.util',
-							'test.x.unknown',
-						],
-					],
-					'test.group.foo.1' => [
-						'class' => ResourceLoaderTestModule::class,
-						'group' => 'x-foo',
-					],
-					'test.group.foo.2' => [
-						'class' => ResourceLoaderTestModule::class,
-						'group' => 'x-foo',
-					],
-					'test.group.bar.1' => [
-						'class' => ResourceLoaderTestModule::class,
-						'group' => 'x-bar',
-					],
-					'test.group.bar.2' => [
-						'class' => ResourceLoaderTestModule::class,
-						'group' => 'x-bar',
-						'source' => 'example',
-					],
-					'test.target.foo' => [
-						'class' => ResourceLoaderTestModule::class,
-						'targets' => [ 'x-foo' ],
-					],
-					'test.target.bar' => [
-						'class' => ResourceLoaderTestModule::class,
-						'source' => 'example',
-						'targets' => [ 'x-foo' ],
-					],
-					'test.es6' => [
-						'class' => ResourceLoaderTestModule::class,
-						'es6' => true
-					]
-				],
-				'out' => '
-mw.loader.addSource({
-    "local": "/w/load.php",
-    "example": "http://example.org/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        ""
-    ],
-    [
-        "test.x.core",
-        ""
-    ],
-    [
-        "test.x.util",
-        "",
-        [
-            1
-        ]
-    ],
-    [
-        "test.x.foo",
-        "",
-        [
-            1
-        ]
-    ],
-    [
-        "test.x.bar",
-        "",
-        [
-            2
-        ]
-    ],
-    [
-        "test.x.quux",
-        "",
-        [
-            3,
-            4,
-            "test.x.unknown"
-        ]
-    ],
-    [
-        "test.group.foo.1",
-        "",
-        [],
-        2
-    ],
-    [
-        "test.group.foo.2",
-        "",
-        [],
-        2
-    ],
-    [
-        "test.group.bar.1",
-        "",
-        [],
-        3
-    ],
-    [
-        "test.group.bar.2",
-        "",
-        [],
-        3,
-        "example"
-    ],
-    [
-        "test.es6",
-        "!"
-    ]
-]);'
-			] ],
-		];
-	}
-
-	/**
-	 * @dataProvider provideGetModuleRegistrations
-	 * @covers ResourceLoaderStartUpModule
-	 * @covers ResourceLoader::makeLoaderRegisterScript
-	 */
-	public function testGetModuleRegistrations( $case ) {
-		$extraQuery = $case['extraQuery'] ?? [];
-		$context = $this->getResourceLoaderContext( $extraQuery );
-		$rl = $context->getResourceLoader();
-		if ( isset( $case['sources'] ) ) {
-			$rl->addSource( $case['sources'] );
-		}
-		$rl->register( $case['modules'] );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl->getConfig() );
-		$out = ltrim( $case['out'], "\n" );
-
-		// Disable log from getModuleRegistrations via MWExceptionHandler
-		// for case where getVersionHash() is expected to throw.
-		$this->setLogger( 'exception', new Psr\Log\NullLogger() );
-
-		$this->assertEquals(
-			self::expandPlaceholders( $out ),
-			$module->getModuleRegistrations( $context ),
-			$case['msg']
-		);
-	}
-
-	/**
-	 * These test cases test behaviour that are specific to production mode.
-	 *
-	 * @see provideGetModuleRegistrations
-	 */
-	public function provideGetModuleRegistrationsProduction() {
-		yield 'Version falls back gracefully if getModuleContent throws' => [ [
-			'modules' => [
-				'test.fail' => [
-					'factory' => function () {
-						$mock = $this->getMockBuilder( ResourceLoaderTestModule::class )
-							->onlyMethods( [ 'getModuleContent' ] )->getMock();
-						$mock->method( 'getModuleContent' )->will(
-							$this->throwException( new Exception )
-						);
-						return $mock;
-					}
-				]
-			],
-			'out' => 'mw.loader.addSource({"local":"/w/load.php"});' . "\n"
-				. 'mw.loader.register([["test.fail",""]]);' . "\n"
-				. 'mw.loader.state({"test.fail":"error"});',
-		] ];
-		yield 'Version falls back gracefully if getDefinitionSummary throws' => [ [
-			'modules' => [
-				'test.fail' => [
-					'factory' => function () {
-						$mock = $this->getMockBuilder( ResourceLoaderTestModule::class )
-							->onlyMethods( [
-								'enableModuleContentVersion',
-								'getDefinitionSummary'
-							] )
-							->getMock();
-						$mock->method( 'enableModuleContentVersion' )->willReturn( false );
-						$mock->method( 'getDefinitionSummary' )->will(
-							$this->throwException( new Exception )
-						);
-						return $mock;
-					}
-				]
-			],
-			'out' => 'mw.loader.addSource({"local":"/w/load.php"});' . "\n"
-				. 'mw.loader.register([["test.fail",""]]);' . "\n"
-				. 'mw.loader.state({"test.fail":"error"});',
-		] ];
-	}
-
-	/**
-	 * @dataProvider provideGetModuleRegistrationsProduction
-	 * @covers ResourceLoaderStartUpModule
-	 * @covers ResourceLoader
-	 */
-	public function testGetModuleRegistrationsProduction( array $case ) {
-		$context = $this->getResourceLoaderContext( [ 'debug' => 'false' ] );
-		$rl = $context->getResourceLoader();
-		$rl->register( $case['modules'] );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl->getConfig() );
-		$out = ltrim( $case['out'], "\n" );
-
-		// Tolerate exception logs for cases that expect getVersionHash() to throw.
-		$this->setLogger( 'exception', new Psr\Log\NullLogger() );
-
-		$this->assertEquals(
-			self::expandPlaceholders( $out ),
-			$module->getModuleRegistrations( $context )
-		);
-	}
-
-	public static function provideRegistrations() {
-		return [
-			[ [
-				'test.blank' => [ 'class' => ResourceLoaderTestModule::class ],
-				'test.min' => [
-					'class' => ResourceLoaderTestModule::class,
-					'skipFunction' =>
-						'return !!(' .
-						'    window.JSON &&' .
-						'    JSON.parse &&' .
-						'    JSON.stringify' .
-						');',
-					'dependencies' => [
-						'test.blank',
-					],
-				],
-			] ]
-		];
-	}
-
-	/**
-	 * @covers ResourceLoaderStartUpModule::getModuleRegistrations
-	 * @dataProvider provideRegistrations
-	 */
-	public function testRegistrationsMinified( $modules ) {
-		$context = $this->getResourceLoaderContext( [
-			'debug' => 'false',
-		] );
-		$rl = $context->getResourceLoader();
-		$rl->register( $modules );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl->getConfig() );
-		$out = 'mw.loader.addSource({"local":"/w/load.php"});' . "\n"
-		. 'mw.loader.register(['
-		. '["test.blank","{blankVer}"],'
-		. '["test.min","{blankVer}",[0],null,null,'
-		. '"return!!(window.JSON\u0026\u0026JSON.parse\u0026\u0026JSON.stringify);"'
-		. ']]);';
-
-		$this->assertEquals(
-			self::expandPlaceholders( $out ),
-			$module->getModuleRegistrations( $context ),
-			'Minified output'
-		);
-	}
-
-	/**
-	 * @covers ResourceLoaderStartUpModule::getModuleRegistrations
-	 * @dataProvider provideRegistrations
-	 */
-	public function testRegistrationsUnminified( $modules ) {
-		$context = $this->getResourceLoaderContext( [
-			'debug' => 'true',
-		] );
-		$rl = $context->getResourceLoader();
-		$rl->register( $modules );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl->getConfig() );
-		$out =
-'mw.loader.addSource({
-    "local": "/w/load.php"
-});
-mw.loader.register([
-    [
-        "test.blank",
-        ""
-    ],
-    [
-        "test.min",
-        "",
-        [
-            0
-        ],
-        null,
-        null,
-        "return !!(    window.JSON \u0026\u0026    JSON.parse \u0026\u0026    JSON.stringify);"
-    ]
-]);';
-
-		$this->assertEquals(
-			self::expandPlaceholders( $out ),
-			$module->getModuleRegistrations( $context ),
-			'Unminified output'
-		);
-	}
-
-	/**
-	 * @covers ResourceLoaderStartupModule
-	 */
-	public function testGetVersionHash_varyConfig() {
-		$context = $this->getResourceLoaderContext();
-
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $context->getResourceLoader()->getConfig() );
-		$version1 = $module->getVersionHash( $context );
-
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $context->getResourceLoader()->getConfig() );
-		$version2 = $module->getVersionHash( $context );
-
-		$this->assertEquals(
-			$version1,
-			$version2,
-			'Deterministic version hash'
-		);
-	}
-
-	/**
-	 * @covers ResourceLoaderStartupModule
-	 */
-	public function testGetVersionHash_varyModule() {
-		$context1 = $this->getResourceLoaderContext( [
-			'debug' => 'false',
-		] );
-		$rl1 = $context1->getResourceLoader();
-		$rl1->register( [
-			'test.a' => [ 'class' => ResourceLoaderTestModule::class ],
-			'test.b' => [ 'class' => ResourceLoaderTestModule::class ],
-		] );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl1->getConfig() );
-		$version1 = $module->getVersionHash( $context1 );
-
-		$context2 = $this->getResourceLoaderContext();
-		$rl2 = $context2->getResourceLoader();
-		$rl2->register( [
-			'test.b' => [ 'class' => ResourceLoaderTestModule::class ],
-			'test.c' => [ 'class' => ResourceLoaderTestModule::class ],
-		] );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl2->getConfig() );
-		$version2 = $module->getVersionHash( $context2 );
-
-		$context3 = $this->getResourceLoaderContext();
-		$rl3 = $context3->getResourceLoader();
-		$rl3->register( [
-			'test.a' => [ 'class' => ResourceLoaderTestModule::class ],
-			'test.b' => [
-				'class' => ResourceLoaderTestModule::class,
-				'script' => 'different',
-			],
-		] );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl3->getConfig() );
-		$version3 = $module->getVersionHash( $context3 );
-
-		// Module name *is* significant (T201686)
-		$this->assertNotEquals(
-			$version1,
-			$version2,
-			'Module name is significant'
-		);
-
-		$this->assertNotEquals(
-			$version1,
-			$version3,
-			'Hash change of any module impacts startup hash'
-		);
-	}
-
-	/**
-	 * @covers ResourceLoaderStartUpModule
-	 */
-	public function testGetVersionHash_varyDeps() {
-		$context = $this->getResourceLoaderContext( [ 'debug' => 'false' ] );
-		$rl = $context->getResourceLoader();
-		$rl->register( [
-			'test.a' => [
-				'class' => ResourceLoaderTestModule::class,
-				'dependencies' => [ 'x', 'y' ],
-			],
-		] );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl->getConfig() );
-		$version1 = $module->getVersionHash( $context );
-
-		$context = $this->getResourceLoaderContext();
-		$rl = $context->getResourceLoader();
-		$rl->register( [
-			'test.a' => [
-				'class' => ResourceLoaderTestModule::class,
-				'dependencies' => [ 'x', 'z' ],
-			],
-		] );
-		$module = new ResourceLoaderStartUpModule();
-		$module->setConfig( $rl->getConfig() );
-		$version2 = $module->getVersionHash( $context );
-
-		// Dependencies *are* significant (T201686)
-		$this->assertNotEquals(
-			$version1,
-			$version2,
-			'Dependencies are significant'
-		);
-	}
-
-}