Page MenuHomePhabricator

mw.loader test suite failing on IE10
Closed, DeclinedPublic

Description

This test times out on IE 10 Platform Preview 4 running on Windows 8 Developer Preview (it seems fine in the IE 10 that's still stock with Win8 DP). Need to watch out for it at least, make sure it's not a sticky regression in the platform.


Version: unspecified
Severity: normal

Details

Reference
bz32742

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:06 AM
bzimport set Reference to bz32742.

This is failing for me by trying to load a broken URL like this:

http://192.168.38.113trunk/tests/qunit/data/qunitOkCall.js?1322725095008

we're missing a '/' there on the location's path...

... this seems to come directly from window.location.pathname. :( I'll check docs and see if that's a theoretically valid thing to do, in which case we should work around it by prepending a / if needed.

Looks like a regression; same bug is listed as having been fixed in the IE 10 that comes with Win8 DP:
https://connect.microsoft.com/IE/feedback/details/679708/the-javascript-property-document-location-pathname-does-not-start-with-a-slash#tabs

(Note that to view any of these links you need to sign in with your Microsoft Windows Live account and jump through some hoops.)

(In reply to comment #1)

This is failing for me by trying to load a broken URL like this:

http://192.168.38.113trunk/tests/qunit/data/qunitOkCall.js?1322725095008

we're missing a '/' there on the location's path...

... this seems to come directly from window.location.pathname. :( I'll check
docs and see if that's a theoretically valid thing to do, in which case we
should work around it by prepending a / if needed.

This is partly due to the ugly way used to determine the path to ./test/qunit/data.
Since the tests execute from static html, there is no wgScriptPath etc.

This should be redundant soon when we're testing from a SpecialPage instead with full MediaWiki context.

Special:JavaScriptTest has been implemented in the mean time and I just verified that the mw.loader test suite is passing on IE10.