Skip to content

Commit 813800c

Browse files
committed
fix: resolve ruff linting issues in integration tests
Applied ruff auto-fixes to use modern Python 3.10+ syntax: - Converted Union[str, None] to str | None for type annotations - Updated isinstance checks to use modern union syntax - Fixed import organization in test files All linting and type checking now passes for Python 3.10+.
1 parent 5cb0db5 commit 813800c

File tree

4 files changed

+10
-15
lines changed

4 files changed

+10
-15
lines changed

tests/integration/test_direct_api_integration.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,17 +4,15 @@
44
test all Direct API methods against the live Nutrient DWS API.
55
"""
66

7-
from typing import Union
8-
97
import pytest
108

119
from nutrient_dws import NutrientClient
1210

1311
try:
1412
from . import integration_config # type: ignore[attr-defined]
1513

16-
API_KEY: Union[str, None] = integration_config.API_KEY
17-
BASE_URL: Union[str, None] = getattr(integration_config, "BASE_URL", None)
14+
API_KEY: str | None = integration_config.API_KEY
15+
BASE_URL: str | None = getattr(integration_config, "BASE_URL", None)
1816
TIMEOUT: int = getattr(integration_config, "TIMEOUT", 60)
1917
except ImportError:
2018
API_KEY = None

tests/integration/test_live_api.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
"""
55

66
from __future__ import annotations
7-
from typing import Union
87

98
import pytest
109

@@ -13,8 +12,8 @@
1312
try:
1413
from . import integration_config # type: ignore[attr-defined]
1514

16-
API_KEY: Union[str, None] = integration_config.API_KEY
17-
BASE_URL: Union[str, None] = getattr(integration_config, "BASE_URL", None)
15+
API_KEY: str | None = integration_config.API_KEY
16+
BASE_URL: str | None = getattr(integration_config, "BASE_URL", None)
1817
TIMEOUT: int = getattr(integration_config, "TIMEOUT", 60)
1918
except ImportError:
2019
API_KEY = None

tests/integration/test_new_tools_integration.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
"""
66

77
from pathlib import Path
8-
from typing import Union
98

109
import pytest
1110

@@ -14,22 +13,22 @@
1413
try:
1514
from . import integration_config # type: ignore[attr-defined]
1615

17-
API_KEY: Union[str, None] = integration_config.API_KEY
18-
BASE_URL: Union[str, None] = getattr(integration_config, "BASE_URL", None)
16+
API_KEY: str | None = integration_config.API_KEY
17+
BASE_URL: str | None = getattr(integration_config, "BASE_URL", None)
1918
TIMEOUT: int = getattr(integration_config, "TIMEOUT", 60)
2019
except ImportError:
2120
API_KEY = None
2221
BASE_URL = None
2322
TIMEOUT = 60
2423

2524

26-
def assert_is_pdf(file_path_or_bytes: Union[str, bytes]) -> None:
25+
def assert_is_pdf(file_path_or_bytes: str | bytes) -> None:
2726
"""Assert that a file or bytes is a valid PDF.
2827
2928
Args:
3029
file_path_or_bytes: Path to file or bytes content to check.
3130
"""
32-
if isinstance(file_path_or_bytes, (str, bytes)):
31+
if isinstance(file_path_or_bytes, str | bytes):
3332
if isinstance(file_path_or_bytes, str):
3433
with open(file_path_or_bytes, "rb") as f:
3534
content = f.read(8)

tests/integration/test_watermark_image_file_integration.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
import os
44
from pathlib import Path
5-
from typing import Union
65

76
import pytest
87

@@ -11,8 +10,8 @@
1110
try:
1211
from . import integration_config # type: ignore[attr-defined]
1312

14-
API_KEY: Union[str, None] = integration_config.API_KEY
15-
BASE_URL: Union[str, None] = getattr(integration_config, "BASE_URL", None)
13+
API_KEY: str | None = integration_config.API_KEY
14+
BASE_URL: str | None = getattr(integration_config, "BASE_URL", None)
1615
TIMEOUT: int = getattr(integration_config, "TIMEOUT", 60)
1716
except ImportError:
1817
API_KEY = None

0 commit comments

Comments
 (0)