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
Gerrit-Owner: Divadsn <divad.nnamtd...@gmail.com>
Gerrit-Reviewer: Divadsn <divad.nnamtd...@gmail.com>
Gerrit-Reviewer: John Vandenberg <jay...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to