[MediaWiki-commits] [Gerrit] pywikibot/core[master]: Replace assertRaises with assertRaisesRegex in http_tests.py
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/331601 )
Change subject: Replace assertRaises with assertRaisesRegex in http_tests.py
..
Replace assertRaises with assertRaisesRegex in http_tests.py
Bug: T154281
Change-Id: I87707d58bab07886e8b35cb1030563df841c9844
---
M tests/http_tests.py
1 file changed, 22 insertions(+), 18 deletions(-)
Approvals:
John Vandenberg: Looks good to me, approved
jenkins-bot: Verified
diff --git a/tests/http_tests.py b/tests/http_tests.py
index ca90f76..c1ce2f0 100644
--- a/tests/http_tests.py
+++ b/tests/http_tests.py
@@ -133,13 +133,14 @@
"""HTTPS certificate test."""
+CERT_VERIFY_FAILED_RE = 'certificate verify failed'
hostname = 'testssl-expire-r2i2.disig.sk'
def test_https_cert_error(self):
"""Test if http.fetch respects disable_ssl_certificate_validation."""
-self.assertRaises(pywikibot.FatalServerError,
- http.fetch,
-
uri='https://testssl-expire-r2i2.disig.sk/index.en.html')
+self.assertRaisesRegex(pywikibot.FatalServerError,
self.CERT_VERIFY_FAILED_RE,
+ http.fetch,
+
uri='https://testssl-expire-r2i2.disig.sk/index.en.html')
http.session.close() # clear the connection
with warnings.catch_warnings(record=True) as warning_log:
@@ -152,9 +153,9 @@
http.session.close() # clear the connection
# Verify that it now fails again
-self.assertRaises(pywikibot.FatalServerError,
- http.fetch,
-
uri='https://testssl-expire-r2i2.disig.sk/index.en.html')
+self.assertRaisesRegex(pywikibot.FatalServerError,
self.CERT_VERIFY_FAILED_RE,
+ http.fetch,
+
uri='https://testssl-expire-r2i2.disig.sk/index.en.html')
http.session.close() # clear the connection
# Verify that the warning occurred
@@ -181,23 +182,25 @@
def test_http_504(self):
"""Test that a HTTP 504 raises the correct exception."""
-self.assertRaises(pywikibot.Server504Error,
- http.fetch,
- uri=self.get_httpbin_url('/status/504'))
+self.assertRaisesRegex(pywikibot.Server504Error, 'Server
([^\:]+|[^\:]+:[0-9]+) timed out',
+ http.fetch,
+ uri=self.get_httpbin_url('/status/504'))
def test_server_not_found(self):
"""Test server not found exception."""
-self.assertRaises(requests.exceptions.ConnectionError,
- http.fetch,
- uri='http://ru-sib.wikipedia.org/w/api.php',
- default_error_handling=True)
+self.assertRaisesRegex(requests.exceptions.ConnectionError,
+ 'Max retries exceeded with url: /w/api.php',
+ http.fetch,
+ uri='http://ru-sib.wikipedia.org/w/api.php',
+ default_error_handling=True)
def test_invalid_scheme(self):
"""Test invalid scheme."""
# A InvalidSchema is raised within requests
-self.assertRaises(requests.exceptions.InvalidSchema,
- http.fetch,
- uri='invalid://url')
+self.assertRaisesRegex(requests.exceptions.InvalidSchema,
+ 'No connection adapters were found for
\'invalid://url\'',
+ http.fetch,
+ uri='invalid://url')
def test_follow_redirects(self):
"""Test follow 301 redirects correctly."""
@@ -421,6 +424,7 @@
"""Test that HttpRequest correct handles the charsets given."""
+CODEC_CANT_DECODE_RE = 'codec can\'t decode byte'
net = False
STR = u'äöü'
@@ -507,9 +511,9 @@
req = CharsetTestCase._create_request('utf16',
CharsetTestCase.LATIN1_BYTES)
self.assertEqual('utf16', req.charset)
-self.assertRaises(UnicodeDecodeError, lambda: req.encoding)
+self.assertRaisesRegex(UnicodeDecodeError, self.CODEC_CANT_DECODE_RE,
lambda: req.encoding)
self.assertEqual(req.raw, CharsetTestCase.LATIN1_BYTES)
-self.assertRaises(UnicodeDecodeError, lambda: req.content)
+self.assertRaisesRegex(UnicodeDecodeError, self.CODEC_CANT_DECODE_RE,
lambda: req.content)
class BinaryTestCase(TestCase):
--
To view, visit https://gerrit.wikimedia.org/r/331601
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I87707d58bab07886e8b35cb1030563df841c9844
Gerrit-PatchSet: 9
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
[MediaWiki-commits] [Gerrit] pywikibot/core[master]: Replace assertRaises with assertRaisesRegex in http_tests.py
Divadsn has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/331601 )
Change subject: Replace assertRaises with assertRaisesRegex in http_tests.py
..
Replace assertRaises with assertRaisesRegex in http_tests.py
Bug: T154281
Change-Id: I87707d58bab07886e8b35cb1030563df841c9844
---
M tests/http_tests.py
1 file changed, 10 insertions(+), 8 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core
refs/changes/01/331601/1
diff --git a/tests/http_tests.py b/tests/http_tests.py
index ca90f76..ebbfe56 100644
--- a/tests/http_tests.py
+++ b/tests/http_tests.py
@@ -133,18 +133,19 @@
"""HTTPS certificate test."""
+CERT_VERIFY_FAILED_RE = 'SSL
routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed'
hostname = 'testssl-expire-r2i2.disig.sk'
def test_https_cert_error(self):
"""Test if http.fetch respects disable_ssl_certificate_validation."""
-self.assertRaises(pywikibot.FatalServerError,
+self.assertRaisesRegex(pywikibot.FatalServerError,
self.CERT_VERIFY_FAILED_RE,
http.fetch,
uri='https://testssl-expire-r2i2.disig.sk/index.en.html')
http.session.close() # clear the connection
with warnings.catch_warnings(record=True) as warning_log:
response = http.fetch(
-uri='https://testssl-expire-r2i2.disig.sk/index.en.html',
+
uri='https://testssl-expire-r2i2.disig.sk/assertRaisesRegex(index.en.html',
disable_ssl_certificate_validation=True)
r = response.content
self.assertIsInstance(r, unicode)
@@ -152,7 +153,7 @@
http.session.close() # clear the connection
# Verify that it now fails again
-self.assertRaises(pywikibot.FatalServerError,
+self.assertRaisesRegex(pywikibot.FatalServerError,
self.CERT_VERIFY_FAILED_RE,
http.fetch,
uri='https://testssl-expire-r2i2.disig.sk/index.en.html')
http.session.close() # clear the connection
@@ -181,13 +182,13 @@
def test_http_504(self):
"""Test that a HTTP 504 raises the correct exception."""
-self.assertRaises(pywikibot.Server504Error,
+self.assertRaisesRegex(pywikibot.Server504Error, 'Server httpbin.org
timed out',
http.fetch,
uri=self.get_httpbin_url('/status/504'))
def test_server_not_found(self):
"""Test server not found exception."""
-self.assertRaises(requests.exceptions.ConnectionError,
+self.assertRaisesRegex(requests.exceptions.ConnectionError, 'Max
retries exceeded with url: /w/api.php',
http.fetch,
uri='http://ru-sib.wikipedia.org/w/api.php',
default_error_handling=True)
@@ -195,7 +196,7 @@
def test_invalid_scheme(self):
"""Test invalid scheme."""
# A InvalidSchema is raised within requests
-self.assertRaises(requests.exceptions.InvalidSchema,
+self.assertRaisesRegex(requests.exceptions.InvalidSchema, 'No
connection adapters were found for \'invalid://url\'',
http.fetch,
uri='invalid://url')
@@ -421,6 +422,7 @@
"""Test that HttpRequest correct handles the charsets given."""
+CODEC_CANT_DECODE_RE = 'codec can\'t decode byte'
net = False
STR = u'äöü'
@@ -507,9 +509,9 @@
req = CharsetTestCase._create_request('utf16',
CharsetTestCase.LATIN1_BYTES)
self.assertEqual('utf16', req.charset)
-self.assertRaises(UnicodeDecodeError, lambda: req.encoding)
+self.assertRaisesRegex(UnicodeDecodeError, self.CODEC_CANT_DECODE_RE,
lambda: req.encoding)
self.assertEqual(req.raw, CharsetTestCase.LATIN1_BYTES)
-self.assertRaises(UnicodeDecodeError, lambda: req.content)
+self.assertRaisesRegex(UnicodeDecodeError, self.CODEC_CANT_DECODE_RE,
lambda: req.content)
class BinaryTestCase(TestCase):
--
To view, visit https://gerrit.wikimedia.org/r/331601
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I87707d58bab07886e8b35cb1030563df841c9844
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Divadsn
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
