From 4937a7430e98499fef89724a7a93aa6e05fd6b64 Mon Sep 17 00:00:00 2001
From: Timo Tijhof <krinklemail@gmail.com>
Date: Tue, 27 Mar 2018 14:23:38 -0700
Subject: [PATCH] resourceloader: Add test for ClientHtml passing down 'target'

This was not previously tested.

Change-Id: I25d9ec28b712b0b699f1369b284faf337b8af5aa
---
 .../ResourceLoaderClientHtmlTest.php          | 38 +++++++++++++++++++
 1 file changed, 38 insertions(+)

diff --git a/tests/phpunit/includes/resourceloader/ResourceLoaderClientHtmlTest.php b/tests/phpunit/includes/resourceloader/ResourceLoaderClientHtmlTest.php
index a75ea56879..4f66bd9744 100644
--- a/tests/phpunit/includes/resourceloader/ResourceLoaderClientHtmlTest.php
+++ b/tests/phpunit/includes/resourceloader/ResourceLoaderClientHtmlTest.php
@@ -217,6 +217,44 @@ class ResourceLoaderClientHtmlTest extends PHPUnit\Framework\TestCase {
 		$this->assertEquals( $expected, $client->getHeadHtml() );
 	}
 
+	/**
+	 * Confirm that 'target' is passed down to the startup module's load url.
+	 *
+	 * @covers ResourceLoaderClientHtml::getHeadHtml
+	 */
+	public function testGetHeadHtmlWithTarget() {
+		$client = new ResourceLoaderClientHtml(
+			self::makeContext(),
+			'example'
+		);
+
+		// phpcs:disable Generic.Files.LineLength
+		$expected = '<script>document.documentElement.className = document.documentElement.className.replace( /(^|\s)client-nojs(\s|$)/, "$1client-js$2" );</script>' . "\n"
+			. '<script async="" src="/w/load.php?debug=false&amp;lang=nl&amp;modules=startup&amp;only=scripts&amp;skin=fallback&amp;target=example"></script>';
+		// phpcs:enable
+
+		$this->assertEquals( $expected, $client->getHeadHtml() );
+	}
+
+	/**
+	 * Confirm that a null 'target' is the same as no target.
+	 *
+	 * @covers ResourceLoaderClientHtml::getHeadHtml
+	 */
+	public function testGetHeadHtmlWithNullTarget() {
+		$client = new ResourceLoaderClientHtml(
+			self::makeContext(),
+			null
+		);
+
+		// phpcs:disable Generic.Files.LineLength
+		$expected = '<script>document.documentElement.className = document.documentElement.className.replace( /(^|\s)client-nojs(\s|$)/, "$1client-js$2" );</script>' . "\n"
+			. '<script async="" src="/w/load.php?debug=false&amp;lang=nl&amp;modules=startup&amp;only=scripts&amp;skin=fallback"></script>';
+		// phpcs:enable
+
+		$this->assertEquals( $expected, $client->getHeadHtml() );
+	}
+
 	/**
 	 * @covers ResourceLoaderClientHtml::getBodyHtml
 	 * @covers ResourceLoaderClientHtml::getLoad
-- 
2.20.1