Lokal Profil has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/268408

Change subject: Add edit failure tests for url data type
......................................................................

Add edit failure tests for url data type

Adds two tests for the url data type and makes the WbMonolingualText
tests more explicit.

Change-Id: Ib857db4ac926a199913cc4b596e14603c58c09da
---
M tests/edit_failure_tests.py
1 file changed, 25 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core 
refs/changes/08/268408/1

diff --git a/tests/edit_failure_tests.py b/tests/edit_failure_tests.py
index 395a516..1977ca3 100644
--- a/tests/edit_failure_tests.py
+++ b/tests/edit_failure_tests.py
@@ -142,15 +142,38 @@
         item = pywikibot.ItemPage(repo, 'Q68')
         claim = self._make_WbMonolingualText_claim(repo, text='Test this!',
                                                    language='foo')
-        self.assertAPIError('modification-failed', None, item.addClaim, claim)
+        self.assertAPIError('modification-failed', 'Illegal value: foo',
+                            item.addClaim, claim)
 
     def test_WbMonolingualText_invalid_text(self):
         """Attempt adding a monolingual text with an invalid non-string 
text."""
         repo = self.get_repo()
         item = pywikibot.ItemPage(repo, 'Q68')
         claim = self._make_WbMonolingualText_claim(repo, text=123456, 
language='en')
-        self.assertAPIError('invalid-snak', None, item.addClaim, claim)
+        self.assertAPIError('invalid-snak',
+                            'Invalid snak (Can only construct a '
+                            'MonolingualTextValue with a string value.)',
+                            item.addClaim, claim)
 
+    def test_url_malformed_url(self):
+        """Attempt adding a malformed URL to a url claim."""
+        repo = self.get_repo()
+        item = pywikibot.ItemPage(repo, 'Q68')
+        claim = pywikibot.page.Claim(repo, 'P506', datatype='url')
+        claim.setTarget('Not a URL at all')
+        self.assertAPIError('modification-failed',
+                            'Malformed URL: Not a URL at all',
+                            item.addClaim, claim)
+
+    def test_url_invalid_protocol(self):
+        """Attempt adding invalid latex to a math claim."""
+        repo = self.get_repo()
+        item = pywikibot.ItemPage(repo, 'Q68')
+        claim = pywikibot.page.Claim(repo, 'P506', datatype='url')
+        claim.setTarget('wtf://wikiba.se')
+        self.assertAPIError('modification-failed',
+                            'Unsupported URL scheme: wtf',
+                            item.addClaim, claim)
 
 if __name__ == '__main__':
     try:

-- 
To view, visit https://gerrit.wikimedia.org/r/268408
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib857db4ac926a199913cc4b596e14603c58c09da
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Lokal Profil <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to