Page MenuHomePhabricator

LexemeHistoryActionTest broken by extra HTML
Closed, ResolvedPublic

Description

Seen in three gate-and-submit builds (a fourth one failed due to T421827 instead) for this harmless change.

1) Wikibase\Lexeme\Tests\MediaWiki\Actions\LexemeHistoryActionTest::testFallsBackToParentIfNoLexemeId
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
-'(history-title: Page title)'
+'(history-title: <span lang="en" dir="ltr"><span class="mw-page-title-main">Page title</span></span>)'

/workspace/src/extensions/WikibaseLexeme/tests/phpunit/mediawiki/Actions/LexemeHistoryActionTest.php:46

Probably due to HistoryAction: Ensure correct language and direction of title (T36514) which was merged just now.

Event Timeline

Change #1273846 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/WikibaseLexeme@master] LexemeHistoryActionTest: Update for core changes

https://gerrit.wikimedia.org/r/1273846

Change #1273846 merged by jenkins-bot:

[mediawiki/extensions/WikibaseLexeme@master] LexemeHistoryActionTest: Update for core changes

https://gerrit.wikimedia.org/r/1273846