Remove redundant requests integration test
We now run the full requests test suite on every pull request. It caught
more issues than this simple test and is actually reliable.
MODIFY
dev-requirements.txt
dev-requirements.txt
@@ -7,7 +7,6 @@ pytest-freezegun==0.4.2
flaky==3.7.0
trustme==0.9.0
cryptography==3.4.7
-requests
backports.zoneinfo==0.2.1;python_version<"3.9"
# https://github.com/twisted/towncrier/pull/356
git+https://github.com/twisted/towncrier.git

MODIFY
noxfile.py
noxfile.py
@@ -170,7 +170,6 @@ def mypy(session: nox.Session) -> None:
"test/with_dummyserver/test_connectionpool.py",
"test/with_dummyserver/test_chunked_transfer.py",
"test/with_dummyserver/test_https.py",
- "test/with_dummyserver/test_integration.py",
"test/with_dummyserver/test_no_ssl.py",
"test/with_dummyserver/test_poolmanager.py",
"test/with_dummyserver/test_proxy_poolmanager.py",

DELETE
test/with_dummyserver/test_integration.py
-import pytest
-import requests
-
-from dummyserver.server import DEFAULT_CA
-from dummyserver.testcase import HTTPDummyServerTestCase, HTTPSDummyServerTestCase
-
-
-class TestHTTPIntegration(HTTPDummyServerTestCase):
- def test_requests_integration(self) -> None:
- with pytest.warns(DeprecationWarning) as records:
- response = requests.get(f"{self.scheme}://{self.host}:{self.port}")
-
- assert 200 == response.status_code
- msg = (
- "The 'strict' parameter is no longer needed on Python 3+. "
- "This will raise an error in urllib3 v3.0.0."
- )
- assert any(
- isinstance(record.message, Warning) and record.message.args[0] == msg
- for record in records
- )
-
-
-class TestHTTPSIntegration(HTTPSDummyServerTestCase):
- def test_requests_integration(self) -> None:
- with pytest.warns(DeprecationWarning) as records:
- response = requests.get(
- f"{self.scheme}://{self.host}:{self.port}", verify=DEFAULT_CA
- )
-
- assert 200 == response.status_code
- msg = (
- "The 'strict' parameter is no longer needed on Python 3+. "
- "This will raise an error in urllib3 v3.0.0."
- )
- assert any(
- isinstance(record.message, Warning) and record.message.args[0] == msg
- for record in records
- )

Remove redundant requests integration test

We now run the full requests test suite on every pull request. It caught
more issues than this simple test and is actually reliable.