Page MenuHomePhabricator

ParserIntegrationTest failure on release branches after l10n update
Closed, ResolvedPublic

Description

Seen on a no-op patch - https://gerrit.wikimedia.org/r/c/mediawiki/core/+/821332; https://integration.wikimedia.org/ci/job/mediawiki-quibble-composer-mysql-php73-docker/3462/console

23:03:31 1) ParserIntegrationTest::testParse with data set "parserTests.txt: Internal link with kaa linktrail with apostrophes (T29473)" ('[details omitted]')
23:03:31 Failed asserting that two strings are equal.
23:03:31 --- Expected
23:03:31 +++ Actual
23:03:31 @@ @@
23:03:31 -'<p><a href="/index.php?title=Something&amp;action=edit&amp;redlink=1" class="new" title="Something (bet ele jaratılmag&#39;an)">Something'nice</a>\n
23:03:31 +'<p><a href="/index.php?title=Something&amp;action=edit&amp;redlink=1" class="new" title="Something (bet ele jaratılmaǵan)">Something'nice</a>\n
23:03:31  </p>'
23:03:31 
23:03:31 /workspace/src/tests/phpunit/suites/ParserIntegrationTest.php:65
23:03:31 /workspace/src/tests/phpunit/suites/SuiteEventsTrait.php:25
23:03:31 /workspace/src/tests/phpunit/suites/SuiteEventsTrait.php:25
23:03:31 /workspace/src/maintenance/doMaintenance.php:107

Event Timeline

Change 821325 had a related patch set uploaded (by Arlolra; author: Umherirrender):

[mediawiki/core@REL1_35] tests: Update parser test after i18n change

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

cf IRC my own comment... Which also means it's gonna break on REL1_37 "today", and REL1_38 "tomorrow"

Reedy renamed this task from ParserIntegrationTest failure on MW 1.35 to ParserIntegrationTest failure on release branches after l10n update.Aug 9 2022, 12:49 AM
Reedy triaged this task as High priority.

Change 821325 merged by jenkins-bot:

[mediawiki/core@REL1_35] tests: Update parser test after i18n change

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

Change 821339 had a related patch set uploaded (by Reedy; author: Umherirrender):

[mediawiki/core@REL1_37] tests: Update parser test after i18n change

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

Change 821340 had a related patch set uploaded (by Reedy; author: Umherirrender):

[mediawiki/core@REL1_38] tests: Update parser test after i18n change

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

Change 821339 merged by jenkins-bot:

[mediawiki/core@REL1_37] tests: Update parser test after i18n change

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

Change 821340 merged by jenkins-bot:

[mediawiki/core@REL1_38] tests: Update parser test after i18n change

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

Reedy claimed this task.

Change 823263 had a related patch set uploaded (by Arlolra; author: Umherirrender):

[mediawiki/services/parsoid@REL1_37] tests: Update parser test after i18n change

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

Change 823263 merged by jenkins-bot:

[mediawiki/services/parsoid@REL1_37] tests: Update parser test after i18n change

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