Last modified: 2012-11-01 00:26:04 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 T34742, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32742 - mw.loader test suite failing on IE10
mw.loader test suite failing on IE10
Status: RESOLVED WORKSFORME
Product: MediaWiki
Classification: Unclassified
Unit tests (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Krinkle
: upstream
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-30 21:57 UTC by Brion Vibber
Modified: 2012-11-01 00:26 UTC (History)
2 users (show)

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


Attachments

Description Brion Vibber 2011-11-30 21:57:42 UTC
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.
Comment 1 Brion Vibber 2011-11-30 23:50:38 UTC
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.
Comment 3 Brion Vibber 2011-12-01 00:16:32 UTC
Submitted bug report upstream through Microsoft Connect:
https://connect.microsoft.com/ie/feedback/details/710046/window-location-pathname-is-missing-initial-slash
Comment 4 Brion Vibber 2011-12-01 00:21:28 UTC
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.)
Comment 5 Krinkle 2011-12-29 09:52:15 UTC
(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.
Comment 6 Krinkle 2012-11-01 00:25:56 UTC
Special:JavaScriptTest has been implemented in the mean time and I just verified that the mw.loader test suite is passing on IE10.

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


Navigation
Links