Skip to content

Commit 13c0271

Browse files
committed
chore: tidy up tests
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
1 parent 6f58cf8 commit 13c0271

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

tests/test_client.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -952,7 +952,7 @@ def test_client_secure_channel_uses_custom_cert(mock_diode_authentication, tmp_p
952952
):
953953
mock_load_certs.return_value = cert_content
954954

955-
client = DiodeClient(
955+
_ = DiodeClient(
956956
target="grpcs://localhost:8081",
957957
app_name="my-producer",
958958
app_version="0.0.1",
@@ -980,7 +980,7 @@ def test_client_without_cert_file_uses_default_certs(mock_diode_authentication):
980980
):
981981
mock_load_certs.return_value = b"default cert content"
982982

983-
client = DiodeClient(
983+
_ = DiodeClient(
984984
target="grpcs://localhost:8081",
985985
app_name="my-producer",
986986
app_version="0.0.1",
@@ -1003,8 +1003,8 @@ def test_client_without_cert_file_uses_default_certs(mock_diode_authentication):
10031003
def test_should_verify_tls_with_different_schemes():
10041004
"""Test _should_verify_tls with different URL schemes."""
10051005
from netboxlabs.diode.sdk.client import (
1006-
_should_verify_tls,
10071006
_DIODE_SKIP_TLS_VERIFY_ENVVAR_NAME,
1007+
_should_verify_tls,
10081008
)
10091009

10101010
# Clear environment variable to avoid interference
@@ -1020,8 +1020,8 @@ def test_should_verify_tls_with_different_schemes():
10201020
def test_should_verify_tls_with_skip_env_var():
10211021
"""Test _should_verify_tls with DIODE_SKIP_TLS_VERIFY environment variable."""
10221022
from netboxlabs.diode.sdk.client import (
1023-
_should_verify_tls,
10241023
_DIODE_SKIP_TLS_VERIFY_ENVVAR_NAME,
1024+
_should_verify_tls,
10251025
)
10261026

10271027
original_env = os.environ.get(_DIODE_SKIP_TLS_VERIFY_ENVVAR_NAME)

0 commit comments

Comments
 (0)