Last modified: 2014-01-03 16:19:38 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T61212, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 59212 - MathLaTeXMLTest::testIntegration broken
MathLaTeXMLTest::testIntegration broken
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Continuous integration (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-01-02 15:20 UTC by Sam Reed (reedy)
Modified: 2014-01-03 16:19 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sam Reed (reedy) 2014-01-02 15:20:50 UTC
Upon trying to merge a revision that didn't touch the actual running code or unit tests, it was noticed that the tests apparently currently in place are broken. This should be fixed

15:15:09 1) MathLaTeXMLTest::testIntegration
15:15:09 Rendering of a+b in plain Text mode
15:15:09 Failed asserting that two strings are equal.
15:15:09 --- Expected
15:15:09 +++ Actual
15:15:09 @@ @@
15:15:09 -'<span class="tex" dir="ltr" id="a_b"><math xmlns="http://www.w3.org/1998/Math/MathML" id="p1.1.m1" class="ltx_Math" alttext="a+b" xml:id="p1.1.m1.1" display="inline" xref="p1.1.m1.1.cmml">   <semantics xml:id="p1.1.m1.1a" xref="p1.1.m1.1.cmml">     <mrow xml:id="p1.1.m1.1.4" xref="p1.1.m1.1.4.cmml">       <mi xml:id="p1.1.m1.1.1" xref="p1.1.m1.1.1.cmml">a</mi>       <mo xml:id="p1.1.m1.1.2" xref="p1.1.m1.1.2.cmml">+</mo>       <mi xml:id="p1.1.m1.1.3" xref="p1.1.m1.1.3.cmml">b</mi>     </mrow>     <annotation-xml xml:id="p1.1.m1.1.cmml" encoding="MathML-Content" xref="p1.1.m1.1">       <apply xml:id="p1.1.m1.1.4.cmml" xref="p1.1.m1.1.4">         <plus xml:id="p1.1.m1.1.2.cmml" xref="p1.1.m1.1.2"/>         <ci xml:id="p1.1.m1.1.1.cmml" xref="p1.1.m1.1.1">a</ci>         <ci xml:id="p1.1.m1.1.3.cmml" xref="p1.1.m1.1.3">b</ci>       </apply>     </annotation-xml>     <annotation xml:id="p1.1.m1.1b" encoding="application/x-tex" xref="p1.1.m1.1.cmml">a+b</annotation>   </semantics> </math></span>'
15:15:09 +'<strong class='error'>Failed to parse(LaTeXML timeout from &#039;http://latexml.mathweb.org/convert&#039;): a+b</strong>
15:15:09 +'
15:15:09 
15:15:09 /srv/ssd/jenkins-slave/workspace/mwext-Math-testextensions-master/extensions/Math/tests/MathLaTeXMLTest.php:134
15:15:09 /srv/ssd/jenkins-slave/workspace/mwext-Math-testextensions-master/tests/phpunit/MediaWikiTestCase.php:123
15:15:09 /srv/ssd/jenkins-slave/workspace/mwext-Math-testextensions-master/tests/phpunit/MediaWikiPHPUnitCommand.php:80
15:15:09 /srv/ssd/jenkins-slave/workspace/mwext-Math-testextensions-master/tests/phpunit/MediaWikiPHPUnitCommand.php:64
15:15:09
Comment 1 Sam Reed (reedy) 2014-01-02 15:21:11 UTC
https://integration.wikimedia.org/ci/job/mwext-Math-testextensions-master/453/console is possibly easier to read! ;)
Comment 2 physikerwelt 2014-01-02 15:23:20 UTC
https://gerrit.wikimedia.org/r/#/c/85801/
Comment 3 Gerrit Notification Bot 2014-01-02 18:06:25 UTC
Change 104994 had a related patch set uploaded by Physikerwelt:
Skip LaTeXML integration tests

https://gerrit.wikimedia.org/r/104994
Comment 4 physikerwelt 2014-01-03 09:32:43 UTC
Seems to be related to Jenkins, since the test work locally.
This test worked fine for month. But at some point in time Jenkins did no longer allow oubound connections to the webservice. Locally the test passes even today.
(Again a crucial point, that was hidden becaus the change to Math 2.0 was too big.)
Comment 5 Antoine "hashar" Musso (WMF) 2014-01-03 16:16:28 UTC
The real fix is to use a http proxy which is made possible by injecting $wgHTTPProxy in MediaWiki configuration.  That part is done via bug 59253 / https://gerrit.wikimedia.org/r/105198
Comment 6 Gerrit Notification Bot 2014-01-03 16:19:03 UTC
Change 104994 abandoned by Physikerwelt:
Skip LaTeXML integration tests

Reason:
No longer needed:
See
Bug: 59253

https://gerrit.wikimedia.org/r/104994
Comment 7 Antoine "hashar" Musso (WMF) 2014-01-03 16:19:38 UTC
Test change https://gerrit.wikimedia.org/r/#/c/105188/

The Math integration works so I guess $wgHTTPProxy is set properly \O/

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links