Add HooksHandlerTest case pinning the foreach addModules contract

Asserts that HooksHandler::onParserAfterParse calls both addModuleStyles
and addModules on the ParserOutput for each active component's modules.
Previously only addModuleStyles was called, so components with a scripts
entry (tooltip.fix, popover.fix, carousel.fix) had their init JS dropped.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: malberts

tests/phpunit/Unit/HooksHandlerTest.php

@@ -6,6 +6,8 @@
 use MediaWiki\Extension\BootstrapComponents\ComponentLibrary;
 use MediaWiki\Extension\BootstrapComponents\HooksHandler;
 use MediaWiki\Extension\BootstrapComponents\NestingController;
+use Parser;
+use ParserOutput;
 use PHPUnit\Framework\TestCase;
 
 /**
@@ -72,4 +74,61 @@ public function testOnGalleryGetModes()	{
 	public function testOnOutputPageParserOutput() {
 		$this->assertTrue( true );
 	}
+
+	/**
+	 * For each active component, the foreach in onParserAfterParse must call
+	 * BOTH addModuleStyles (so the module's CSS half loads as a render-blocking
+	 * <link>) AND addModules (so the JS half is queued via mw.loader). Pins the
+	 * contract introduced when sparse per-component module loading was restored;
+	 * previously only addModuleStyles was called, so component init JS never
+	 * reached the page.
+	 */
+	public function testOnParserAfterParseLoadsActiveComponentScriptsAndStyles() {
+		$loadedStyles = [];
+		$loadedModules = [];
+
+		$parserOutput = $this->createMock( ParserOutput::class );
+		$parserOutput->method( 'addModuleStyles' )
+			->willReturnCallback( function ( $m ) use ( &$loadedStyles ) {
+				$loadedStyles = array_merge( $loadedStyles, (array)$m );
+			} );
+		$parserOutput->method( 'addModules' )
+			->willReturnCallback( function ( $m ) use ( &$loadedModules ) {
+				$loadedModules = array_merge( $loadedModules, (array)$m );
+			} );
+
+		$parser = $this->createMock( Parser::class );
+		$parser->method( 'getOutput' )->willReturn( $parserOutput );
+
+		$service = $this->createMock( BootstrapComponentsService::class );
+		$service->method( 'getNameOfActiveSkin' )->willReturn( 'vector' );
+		$service->method( 'getActiveComponents' )->willReturn( [ 'tooltip', 'popover' ] );
+
+		$library = $this->createMock( ComponentLibrary::class );
+		$library->method( 'isRegistered' )->willReturn( true );
+		$library->method( 'getModulesFor' )
+			->willReturnCallback( function ( $name ) {
+				return [ 'ext.bootstrapComponents.' . $name . '.fix' ];
+			} );
+
+		$handler = new HooksHandler(
+			$service,
+			$library,
+			$this->createMock( NestingController::class )
+		);
+
+		$text = '';
+		$handler->onParserAfterParse( $parser, $text, null );
+
+		// Per-component fix modules must reach BOTH addModuleStyles and addModules
+		// (style-only modules treat addModules as a no-op; modules with a scripts
+		// entry get their JS init via that call).
+		$this->assertContains( 'ext.bootstrapComponents.tooltip.fix', $loadedStyles );
+		$this->assertContains( 'ext.bootstrapComponents.popover.fix', $loadedStyles );
+		$this->assertContains( 'ext.bootstrapComponents.tooltip.fix', $loadedModules );
+		$this->assertContains( 'ext.bootstrapComponents.popover.fix', $loadedModules );
+
+		// And the global Bootstrap-library JS module is queued as before.
+		$this->assertContains( 'ext.bootstrap.scripts', $loadedModules );
+	}
 }