X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=tests%2Fphpunit%2Fincludes%2Fchanges%2FRCCacheEntryFactoryTest.php;h=97b4c0887c8f1579909f411178974ba51b5ed373;hb=6f4eda1fd3e4f592c30b4b8a86bac53301eaec9f;hp=4ff1eb8e768f98ca7e133d46307c683c6b06381d;hpb=6ead43f2123fec255acccce6bf68f31b51196a55;p=lhc%2Fweb%2Fwiklou.git

diff --git a/tests/phpunit/includes/changes/RCCacheEntryFactoryTest.php b/tests/phpunit/includes/changes/RCCacheEntryFactoryTest.php
index 4ff1eb8e76..97b4c0887c 100644
--- a/tests/phpunit/includes/changes/RCCacheEntryFactoryTest.php
+++ b/tests/phpunit/includes/changes/RCCacheEntryFactoryTest.php
@@ -141,103 +141,69 @@ class RCCacheEntryFactoryTest extends MediaWikiLangTestCase {
 		$this->assertEquals( 'prev', $cacheEntry->lastlink, 'pref link for delete log or rev' );
 	}
 
+	private function assertValidHTML( $actual ) {
+		// Throws if invalid
+		$doc = PHPUnit_Util_XML::load( $actual, /* isHtml */ true );
+	}
+
 	private function assertUserLinks( $user, $cacheEntry ) {
-		$this->assertTag(
-			[
-				'tag' => 'a',
-				'attributes' => [
-					'class' => 'new mw-userlink'
-				],
-				'content' => $user
-			],
+		$this->assertValidHTML( $cacheEntry->userlink );
+		$this->assertRegExp(
+			'#^<a .*class="new mw-userlink".*><bdi>' . $user . '</bdi></a>#',
 			$cacheEntry->userlink,
 			'verify user link'
 		);
 
-		$this->assertTag(
-			[
-				'tag' => 'span',
-				'attributes' => [
-					'class' => 'mw-usertoollinks'
-				],
-				'child' => [
-					'tag' => 'a',
-					'content' => 'talk',
-				]
-			],
+		$this->assertValidHTML( $cacheEntry->usertalklink );
+		$this->assertRegExp(
+			'#^ <span class="mw-usertoollinks">\(.*<a .+>talk</a>.*\)</span>#',
 			$cacheEntry->usertalklink,
 			'verify user talk link'
 		);
 
-		$this->assertTag(
-			[
-				'tag' => 'span',
-				'attributes' => [
-					'class' => 'mw-usertoollinks'
-				],
-				'child' => [
-					'tag' => 'a',
-					'content' => 'contribs',
-				]
-			],
+		$this->assertValidHTML( $cacheEntry->usertalklink );
+		$this->assertRegExp(
+			'#^ <span class="mw-usertoollinks">\(.*<a .+>contribs</a>.*\)</span>$#',
 			$cacheEntry->usertalklink,
 			'verify user tool links'
 		);
 	}
 
 	private function assertDeleteLogLink( $cacheEntry ) {
-		$this->assertTag(
-			[
-				'tag' => 'a',
-				'attributes' => [
-					'href' => '/wiki/Special:Log/delete',
-					'title' => 'Special:Log/delete'
-				],
-				'content' => 'Deletion log'
-			],
+		$this->assertEquals(
+			'(<a href="/wiki/Special:Log/delete" title="Special:Log/delete">Deletion log</a>)',
 			$cacheEntry->link,
 			'verify deletion log link'
 		);
+
+		$this->assertValidHTML( $cacheEntry->link );
 	}
 
 	private function assertRevDel( $cacheEntry ) {
-		$this->assertTag(
-			[
-				'tag' => 'span',
-				'attributes' => [
-					'class' => 'history-deleted'
-				],
-				'content' => '(username removed)'
-			],
+		$this->assertEquals(
+			' <span class="history-deleted">(username removed)</span>',
 			$cacheEntry->userlink,
 			'verify user link for change with deleted revision and user'
 		);
+		$this->assertValidHTML( $cacheEntry->userlink );
 	}
 
 	private function assertTitleLink( $title, $cacheEntry ) {
-		$this->assertTag(
-			[
-				'tag' => 'a',
-				'attributes' => [
-					'href' => '/wiki/' . $title,
-					'title' => $title
-				],
-				'content' => $title
-			],
+		$this->assertEquals(
+			'<a href="/wiki/' . $title . '" title="' . $title . '">' . $title . '</a>',
 			$cacheEntry->link,
 			'verify title link'
 		);
+		$this->assertValidHTML( $cacheEntry->link );
 	}
 
 	private function assertQueryLink( $content, $params, $link ) {
-		$this->assertTag(
-			[
-				'tag' => 'a',
-				'content' => $content
-			],
+		$this->assertRegExp(
+			"#^<a .+>$content</a>$#",
 			$link,
-			'assert query link element'
+			'verify query link element'
 		);
+		$this->assertValidHTML( $link );
 
 		foreach ( $params as $key => $value ) {
 			$this->assertRegExp( '/' . $key . '=' . $value . '/', $link, "verify $key link params" );