l10n_test check the validity of templates on reals wiki and verifys wether the template really exists. It is not necessary to run these test for every instance of the test matrix. Running it once for each commited patch would be enough. Or to reduce the test time further it would be sufficient to run one test when i18n submodule is updated.
|Open||None||T152961 Improve tests suite|
|Resolved||Xqt||T186323 Travis and Appveyor: Major improvement to test time|
|Declined||Xqt||T191632 l10n_tests should run only once for every committed patch|
The whole handling with disabled_test_modules is a bit weird:
disabled_test_modules = [ 'tests', # tests of the tests package 'l10n', ]
OK, these will be disabled
if not i18n.messages_available(): disabled_test_modules.append('l10n')
Isn't it already in the list?
if 'PYWIKIBOT_TEST_MODULES' in os.environ: disabled_test_modules = set(test_modules) - set(_enabled_test_modules)
Now we overwrite it even before we use it?
Removing task assignee due to inactivity, as this open task has been assigned for more than two years (see emails sent to assignee on May26 and Jun17, and T270544). Please assign this task to yourself again if you still realistically [plan to] work on this task - it would be very welcome!
(See https://www.mediawiki.org/wiki/Bug_management/Assignee_cleanup for tips how to best manage your individual work in Phabricator.)