jenkins-bot submitted this change.

View Change

Approvals: Mpaa: Looks good to me, approved jenkins-bot: Verified
[bugfix] Remove HttpRequestURI tests

site parameter of http.request is mandatory and may not be None.
Remove test for outdated and removed behaviour with deprecation.

Change-Id: I8ceeae15b88addd8afd2c9ed5eef914af8c6ad56
---
M tests/http_tests.py
1 file changed, 0 insertions(+), 30 deletions(-)

diff --git a/tests/http_tests.py b/tests/http_tests.py
index 6b48725..e3a631e 100644
--- a/tests/http_tests.py
+++ b/tests/http_tests.py
@@ -49,36 +49,6 @@
self.assertIsInstance(r.raw, bytes)


-class HttpRequestURI(DeprecationTestCase):
-
- """Tests using http.request without a site."""
-
- sites = {
- 'www-wp': {
- 'hostname': 'www.wikipedia.org',
- },
- 'www-wq': {
- 'hostname': 'www.wikiquote.org',
- },
- }
-
- def test_http(self):
- """Test http.request using http://www.wikipedia.org/."""
- r = http.request(site=None, uri='http://www.wikipedia.org/')
- self.assertIsInstance(r, str)
- self.assertIn('<html lang="mul"', r)
- self.assertOneDeprecationParts(
- 'Invoking http.request without argument site', 'http.fetch()')
-
- def test_https(self):
- """Test http.request using https://www.wikiquote.org/."""
- r = http.request(site=None, uri='https://www.wikiquote.org/')
- self.assertIsInstance(r, str)
- self.assertIn('<html lang="mul"', r)
- self.assertOneDeprecationParts(
- 'Invoking http.request without argument site', 'http.fetch()')
-
-
class TestGetAuthenticationConfig(TestCase):

"""Test http.get_authentication."""

To view, visit change 648582. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I8ceeae15b88addd8afd2c9ed5eef914af8c6ad56
Gerrit-Change-Number: 648582
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Mpaa <mpaa.wiki@gmail.com>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged