jenkins-bot has submitted this change and it was merged.
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(-)
Approvals:
John Vandenberg: Looks good to me, but someone else must approve
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/tests/textlib_tests.py b/tests/textlib_tests.py
index e4dbc57..e6d67ff 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 10 (Template) case
sensitivity."""
+ super(TestMultiTemplateMatchBuilder, cls).setUpClass()
+ cls._template_not_case_sensitive = (
+ cls.get_site().namespaces.TEMPLATE.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._template_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._template_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._template_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._template_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._template_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: merged
Gerrit-Change-Id: I4eabed2fb7ebeaf1516bb1f80c00a50050efb51c
Gerrit-PatchSet: 3
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>