John Vandenberg has uploaded a new change for review. https://gerrit.wikimedia.org/r/291577
Change subject: Adjust recent added tests for case-sensitive site ...................................................................... Adjust recent added tests for case-sensitive site 2230213 caused new errors on the Travis wiktionary jobs. Also fix non-pep8 method names. Bug: T134676 Change-Id: I4eabed2fb7ebeaf1516bb1f80c00a50050efb51c --- M tests/textlib_tests.py 1 file changed, 25 insertions(+), 9 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core refs/changes/77/291577/1 diff --git a/tests/textlib_tests.py b/tests/textlib_tests.py index e4dbc57..06d72ae 100644 --- a/tests/textlib_tests.py +++ b/tests/textlib_tests.py @@ -1353,44 +1353,60 @@ dry = True - def test_noMatch(self): + @classmethod + def setUpClass(cls): + """Cache namespace 0 case sensitivity.""" + super(TestMultiTemplateMatchBuilder, cls).setUpClass() + cls._ns0_not_case_sensitive = ( + cls.get_site().namespaces[0].case != 'case-sensitive') + + def test_no_match(self): """Test text without any desired templates.""" string = 'The quick brown fox' builder = _MultiTemplateMatchBuilder(self.site) self.assertIsNone(re.search(builder.pattern('quick'), string)) - def test_Match(self): + def test_match(self): """Test text with one match without parameters.""" string = 'The {{quick}} brown fox' builder = _MultiTemplateMatchBuilder(self.site) self.assertIsNotNone(re.search(builder.pattern('quick'), string)) - self.assertIsNotNone(re.search(builder.pattern('Quick'), string)) + self.assertEqual(bool(re.search(builder.pattern('Quick'), string)), + self._ns0_not_case_sensitive) def test_match_with_params(self): """Test text with one match with parameters.""" - string = 'The {{Quick|brown}} fox' + string = 'The {{quick|brown}} fox' builder = _MultiTemplateMatchBuilder(self.site) self.assertIsNotNone(re.search(builder.pattern('quick'), string)) - self.assertIsNotNone(re.search(builder.pattern('Quick'), string)) + self.assertEqual(bool(re.search(builder.pattern('Quick'), string)), + self._ns0_not_case_sensitive) def test_match_msg(self): """Test text with {{msg:..}}.""" string = 'The {{msg:quick}} brown fox' builder = _MultiTemplateMatchBuilder(self.site) self.assertIsNotNone(re.search(builder.pattern('quick'), string)) - self.assertIsNotNone(re.search(builder.pattern('Quick'), string)) + self.assertEqual(bool(re.search(builder.pattern('Quick'), string)), + self._ns0_not_case_sensitive) def test_match_template_prefix(self): """Test pages with {{template:..}}.""" string = 'The {{%s:%s}} brown fox' template = 'template' builder = _MultiTemplateMatchBuilder(self.site) + if self._ns0_not_case_sensitive: + quick_list = ('quick', 'Quick') + else: + quick_list = ('quick', ) + for t in (template.upper(), template.lower(), template.title()): - for q in ('quick', 'Quick'): + for q in quick_list: self.assertIsNotNone(re.search(builder.pattern('quick'), string % (t, q))) - self.assertIsNotNone(re.search(builder.pattern('Quick'), - string % (t, q))) + self.assertEqual(bool(re.search(builder.pattern('Quick'), + string % (t, q))), + self._ns0_not_case_sensitive) class TestGetLanguageLinks(SiteAttributeTestCase): -- To view, visit https://gerrit.wikimedia.org/r/291577 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4eabed2fb7ebeaf1516bb1f80c00a50050efb51c Gerrit-PatchSet: 1 Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-Owner: John Vandenberg <jay...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits