Page MenuHomePhabricator

ProofreadPage CI test lstIncorrectParserTest.txt: LST: Transclude double redirect (bug 42477) is failing
Closed, ResolvedPublic

Description

bugzilla bug 42477 = T44477: lst: transcluding a broken redirect does not show a clear error message

See https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ProofreadPage/+/950006.

18:03:06 1) ParserIntegrationTest::testParse with data set "lstIncorrectParserTest.txt: LST: Transclude double redirect (bug 42477)" ('legacy')
18:03:06 Failed asserting that two strings are equal.
18:03:06 --- Expected
18:03:06 +++ Actual
18:03:06 @@ @@
18:03:06 -''
18:03:06 +'<p>sec 1\n
18:03:06 +</p>'
18:03:06 
18:03:06 /workspace/src/tests/phpunit/suites/ParserIntegrationTest.php:68
18:03:06 /workspace/src/tests/phpunit/suites/SuiteEventsTrait.php:69
18:03:06 /workspace/src/tests/phpunit/suites/SuiteEventsTrait.php:69
18:03:06 phpvfscomposer:///workspace/src/vendor/phpunit/phpunit/phpunit:97
18:03:06 === Logs generated by test case
18:03:06 [wfDebug] [debug] ParserTestRunner::runLegacyTest: running LST: Transclude double redirect (bug 42477) {"private":false}
18:03:06 [wfDebug] [debug] ParserFactory: using default preprocessor {"private":false}

Event Timeline

This looks like a issue with MediaWiki-extensions-LabeledSectionTransclusion, though given that nothing has been merged in that extension since quite a while, I would assume that something has changed upstream in the last 1 hr (last CI test ran at Aug 17 11:26PM and your patch ran on Aug 17 12:13PM)?

Change 950135 had a related patch set uploaded (by Majavah; author: Majavah):

[mediawiki/extensions/LabeledSectionTransclusion@master] tests: Duplicate [[Sections]] definition to test that uses it

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

Change 950135 merged by jenkins-bot:

[mediawiki/extensions/LabeledSectionTransclusion@master] Disable broken test

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

taavi claimed this task.
taavi subscribed.

Fixed!