Last modified: 2014-09-26 22:09:05 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 T72250, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 70250 - Allow tests to specify what extensions and or what order things are loaded in
Allow tests to specify what extensions and or what order things are loaded in
Status: NEW
Product: Wikimedia
Classification: Unclassified
Continuous integration (Other open bugs)
wmf-deployment
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-01 11:29 UTC by Addshore
Modified: 2014-09-26 22:09 UTC (History)
5 users (show)

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


Attachments

Description Addshore 2014-09-01 11:29:43 UTC
The remaining issue is that the Wikidata.php entry file is loaded before we can define some vars in php, thus the repo / client is not loaded hence Lib is not loaded and we get the exception about undefined constant.
The wikidata.php file is loaded by the 50_mw_ext_loader.php file.

As far as I can tell the best ways to deal with this would be to:

Allow tests to specify what they want to be added to localsettings, this would probably require far to much repetition in regard to extension tests, although would mean we can test extensions that have badly named entry points.

Allow tests to specify specific things to add to localsettings but if this is not defined then fallback to the autoloading of entry points (seems like quite a good plan)

Allow tests to skip the autoloading of extensions from 50_mw_ext_loader.php

Allow tests to skip any defined conf.d file by adding it to some sort of ignore array prior to them all loading

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


Navigation
Links