Skip to content

Commit f018b76

Browse files
chore(internal): make test_proxy_environment_variables more resilient to env
1 parent 178bb7d commit f018b76

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

tests/test_client.py

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -957,8 +957,14 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
957957
def test_proxy_environment_variables(self, monkeypatch: pytest.MonkeyPatch) -> None:
958958
# Test that the proxy environment variables are set correctly
959959
monkeypatch.setenv("HTTPS_PROXY", "https://example.org")
960-
# Delete in case our environment has this set
960+
# Delete in case our environment has any proxy env vars set
961961
monkeypatch.delenv("HTTP_PROXY", raising=False)
962+
monkeypatch.delenv("ALL_PROXY", raising=False)
963+
monkeypatch.delenv("NO_PROXY", raising=False)
964+
monkeypatch.delenv("http_proxy", raising=False)
965+
monkeypatch.delenv("https_proxy", raising=False)
966+
monkeypatch.delenv("all_proxy", raising=False)
967+
monkeypatch.delenv("no_proxy", raising=False)
962968

963969
client = DefaultHttpxClient()
964970

@@ -1875,8 +1881,14 @@ async def test_get_platform(self) -> None:
18751881
async def test_proxy_environment_variables(self, monkeypatch: pytest.MonkeyPatch) -> None:
18761882
# Test that the proxy environment variables are set correctly
18771883
monkeypatch.setenv("HTTPS_PROXY", "https://example.org")
1878-
# Delete in case our environment has this set
1884+
# Delete in case our environment has any proxy env vars set
18791885
monkeypatch.delenv("HTTP_PROXY", raising=False)
1886+
monkeypatch.delenv("ALL_PROXY", raising=False)
1887+
monkeypatch.delenv("NO_PROXY", raising=False)
1888+
monkeypatch.delenv("http_proxy", raising=False)
1889+
monkeypatch.delenv("https_proxy", raising=False)
1890+
monkeypatch.delenv("all_proxy", raising=False)
1891+
monkeypatch.delenv("no_proxy", raising=False)
18801892

18811893
client = DefaultAsyncHttpxClient()
18821894

0 commit comments

Comments
 (0)