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

Reply via email to