gerritbot added a comment.
Change 474543 had a related patch set uploaded (by Xqt; owner: Xqt):
[pywikibot/core@master] [bugfix] Re-enable l10n_tests
https://gerrit.wikimedia.org/r/474543TASK DETAILhttps://phabricator.wikimedia.org/T191632EMAIL
gerritbot added a comment.
Change 462912 merged by jenkins-bot:
[pywikibot/core@master] [tests] Run L10N tests on Travis only once for each patch
https://gerrit.wikimedia.org/r/462912TASK DETAILhttps://phabricator.wikimedia.org/T191632EMAIL
Xqt added a comment.
The test could be done in i18n repository but it seems no tests are done there but the bot verify with +2 together with CR+2TASK DETAILhttps://phabricator.wikimedia.org/T191632EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: XqtCc:
Dvorapa added a comment.
The whole haqndling with disabled_test_modules is a bit weird:
Line 151 disabled_test_modules = [
'tests', # tests of the tests package
'l10n',
]
OK, these will be disabled
Line 155if not i18n.messages_available():
disabled_test_modules.append('l10n')
Xqt added a comment.
I've no glue whether that configuration matrix in .travis.yml inside i18n repository ever runs.TASK DETAILhttps://phabricator.wikimedia.org/T191632EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: XqtCc: gerritbot, Dvorapa, Aklapper, Dalba,
gerritbot added a comment.
Change 462912 had a related patch set uploaded (by Xqt; owner: Xqt):
[pywikibot/core@master] [tests] Run L10N tests only once for each patch
https://gerrit.wikimedia.org/r/462912TASK DETAILhttps://phabricator.wikimedia.org/T191632EMAIL
Dalba added a comment.
Here is an idea (although I don't know how much it can help):
When a commit only affects a script, there is no need to run the all tests, just the related test module for that script, the other parts have already been tested in previous commits. This should work because the
