jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/326469 )
Change subject: Inject skin/ext dependencies ...................................................................... Inject skin/ext dependencies Up until previous commit, we only supported injecting extensions and only on extensions jobs. We now support skins, apply the mecanism on a couple repositories that have been identified to need that: CustomPage extension requires the skin CustomPage. BlueSpiceSkin skin requires the extension BlueSpiceFoundation Add tests to exercise the feature. Bug: T151593 Change-Id: Idc1d5774fe79d4f3542529bd18e09f06fcf6e3d0 --- M tests/test_zuul_mw_dependencies.py M zuul/parameter_functions.py 2 files changed, 28 insertions(+), 0 deletions(-) Approvals: Paladox: Looks good to me, but someone else must approve Hashar: Looks good to me, approved jenkins-bot: Verified diff --git a/tests/test_zuul_mw_dependencies.py b/tests/test_zuul_mw_dependencies.py index bc30074..de6098a 100644 --- a/tests/test_zuul_mw_dependencies.py +++ b/tests/test_zuul_mw_dependencies.py @@ -128,3 +128,25 @@ get_dependencies('skins/Child', mapping), set(['skin/Common']) ) + + def test_inject_skin_on_an_extension(self): + deps = self.fetch_dependencies( + job_name='mwext-testextension-hhvm', + project='mediawiki/extensions/CustomPage') + self.assertDictContainsSubset( + { + 'EXT_NAME': 'CustomPage', + 'SKIN_DEPENDENCIES': 'mediawiki/skins/CustomPage', + }, + deps) + + def test_inject_extension_on_a_skin(self): + deps = self.fetch_dependencies( + job_name='mw-testskin', + project='mediawiki/skins/BlueSpiceSkin') + self.assertDictContainsSubset( + { + 'SKIN_NAME': 'BlueSpiceSkin', + 'EXT_DEPENDENCIES': 'mediawiki/extensions/BlueSpiceFoundation', + }, + deps) diff --git a/zuul/parameter_functions.py b/zuul/parameter_functions.py index 7e5fbb9..cf1cb13 100644 --- a/zuul/parameter_functions.py +++ b/zuul/parameter_functions.py @@ -101,6 +101,11 @@ # 'skins/'. The has is used by the set_mw_dependencies() parameter function # below. dependencies = { + # Skins are listed first to highlight the skin dependencies + 'skins/BlueSpiceSkin': ['BlueSpiceFoundation'], + + # Extensions + # One can add a skin by using: 'skin/XXXXX' 'AbuseFilter': ['AntiSpoof'], 'ApiFeatureUsage': ['Elastica'], 'Arrays': ['Loops', 'ParserFunctions', 'Variables'], @@ -116,6 +121,7 @@ 'ContentTranslation': ['Echo', 'EventLogging', 'GuidedTour', 'UniversalLanguageSelector', 'Wikidata'], 'CookieWarning': ['MobileFrontend'], + 'CustomPage': ['skins/CustomPage'], 'Disambiguator': ['VisualEditor'], 'DonationInterface': ['ContributionTracking'], 'EducationProgram': ['cldr', 'WikiEditor'], -- To view, visit https://gerrit.wikimedia.org/r/326469 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idc1d5774fe79d4f3542529bd18e09f06fcf6e3d0 Gerrit-PatchSet: 2 Gerrit-Project: integration/config Gerrit-Branch: master Gerrit-Owner: Hashar <has...@free.fr> Gerrit-Reviewer: Hashar <has...@free.fr> Gerrit-Reviewer: Legoktm <lego...@member.fsf.org> Gerrit-Reviewer: Paladox <thomasmulhall...@yahoo.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits