Skip to content

Commit 980c38c

Browse files
author
nazarfil
committed
test: added untracked test files
1 parent 7a311d1 commit 980c38c

2 files changed

Lines changed: 32 additions & 6 deletions

File tree

openhexa/sdk/utils.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -64,13 +64,9 @@ def __init__(self, token: str | None = None, server_url: str | None = None):
6464
server_url: Server URL. If not provided, will use HEXA_SERVER_URL environment variable.
6565
"""
6666
url = server_url or f"{os.environ['HEXA_SERVER_URL'].rstrip('/')}/graphql/"
67-
token = token or os.environ.get("HEXA_TOKEN")
67+
token = token or os.getenv("HEXA_TOKEN")
6868

69-
env_value = os.environ.get("HEXA_VERIFY_SSL")
70-
if env_value is None:
71-
verify_ssl = True
72-
else:
73-
verify_ssl = env_value.lower() not in ("0", "false")
69+
verify_ssl = os.getenv("HEXA_VERIFY_SSL", "True").lower() not in ("0", "false")
7470

7571
super().__init__(url=url, token=token, verify=verify_ssl)
7672

tests/test_ssl_errors.py

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
"""SSL error handling test module."""
2+
3+
4+
import requests
5+
6+
from openhexa.utils.session import create_requests_session
7+
8+
9+
class TestSSLErrorHandling:
10+
"""Test SSL error handling functionality."""
11+
12+
def test_create_requests_session_verify_default(self):
13+
"""Test create_requests_session with default verify parameter."""
14+
session = create_requests_session()
15+
assert session.verify is True
16+
17+
def test_ssl_error_handling_logic(self):
18+
"""Test that SSL errors are properly converted to GraphQLError."""
19+
ssl_error = requests.exceptions.SSLError("CERTIFICATE_VERIFY_FAILED error")
20+
21+
if "CERTIFICATE_VERIFY_FAILED" in str(ssl_error):
22+
expected_msg = (
23+
"SSL certificate verification failed. "
24+
"If you want to disable SSL verification, set the environment variable: HEXA_VERIFY_SSL=false"
25+
)
26+
assert "SSL certificate verification failed" in expected_msg
27+
assert "HEXA_VERIFY_SSL=false" in expected_msg
28+
29+
other_ssl_error = requests.exceptions.SSLError("Some other SSL error")
30+
assert "Some other SSL error" in str(other_ssl_error)

0 commit comments

Comments
 (0)