UploadWizard tests fail with jQuery 3, as seen on https://gerrit.wikimedia.org/r/#/c/336610/.
Description
Details
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Krinkle | T127328 Optimise critical rendering path | |||
| Resolved | Krinkle | T124742 Upgrade to jQuery 3 | |||
| Resolved | matthiasmullie | T157201 Stop supporting Firefogg in Upload Wizard now that Firefox is making it impossible | |||
| Resolved | matthiasmullie | T164028 UploadWizard tests fail with jQuery 3 |
Event Timeline
This is probably not a real bug in the code, but just a problem with the tests. We don't wait for promises to be resolved asynchronously. E.g. in snippets like this:
transport.checkStatus().fail( tstub );
...
assert.ok( tstub.calledWith( 'server-error', { error: {
code: 'server-error',
html: mw.message( 'apierror-unknownerror' ).parse()
} } ) );This needs to be something like:
promise = transport.checkStatus().fail( tstub );
...
promise.then( null, function () {
assert.ok( tstub.calledWith( 'server-error', { error: {
code: 'server-error',
html: mw.message( 'apierror-unknownerror' ).parse()
} } ) );
} );Change 350871 had a related patch set uploaded (by Matthias Mullie; owner: Matthias Mullie):
[mediawiki/extensions/UploadWizard@master] Update FormDataTransport tests for jQuery 3
Change 351019 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/UploadWizard@master] Add our EventLogging schemas to test module's dependencies
Change 350871 merged by MarkTraceur:
[mediawiki/extensions/UploadWizard@master] Update FormDataTransport tests for jQuery 3
Change 351019 merged by MarkTraceur:
[mediawiki/extensions/UploadWizard@master] Add our EventLogging schemas to test module's dependencies