ruff fix

Author: msch-nutrient

tests/integration/conftest.py

@@ -15,4 +15,4 @@ def pytest_collection_modifyitems(config, items):
     """Automatically mark integration tests."""
     for item in items:
         if "integration" in str(item.fspath):
-            item.add_marker(pytest.mark.integration)
+            item.add_marker(pytest.mark.integration)

tests/integration/test_live_api.py

@@ -3,12 +3,9 @@
 These tests require a valid API key configured in integration_config.py.
 """
 
-import os
 import pytest
-from pathlib import Path
 
 from nutrient_dws import NutrientClient
-from nutrient_dws.exceptions import AuthenticationError
 
 try:
     from . import integration_config
@@ -64,19 +61,19 @@ def test_basic_api_connectivity(self, client, sample_pdf_path):
     def test_convert_operation(self, client, sample_pdf_path, tmp_path):
         """Test a basic convert operation (example - adjust based on available tools)."""
         output_path = tmp_path / "output.pdf"
-        
+
         # This is an example - adjust based on actual available tools
         # result = client.convert_to_pdf(input_file=sample_pdf_path, output_path=str(output_path))
-        
+
         # assert output_path.exists()
         # assert output_path.stat().st_size > 0
 
     def test_builder_api_basic(self, client, sample_pdf_path):
         """Test basic builder API functionality."""
         builder = client.build(input_file=sample_pdf_path)
-        
+
         # Test that we can add steps without errors
         # This will need to be updated based on actual available tools
         # builder.add_step("example-tool", {})
-        
-        assert builder is not None
+
+        assert builder is not None

tests/unit/test_builder.py

@@ -1,6 +1,5 @@
 """Unit tests for Builder API."""
 
-import pytest
 
 from nutrient_dws.builder import BuildAPIWrapper
 
@@ -18,7 +17,7 @@ def test_builder_add_step():
     """Test adding steps to builder."""
     builder = BuildAPIWrapper(None, "test.pdf")
     result = builder.add_step("convert-to-pdf", options={"format": "docx"})
-    
+
     assert result is builder  # Should return self for chaining
     assert len(builder._actions) == 1
     assert builder._actions[0]["type"] == "convert-to-pdf"
@@ -33,7 +32,7 @@ def test_builder_chaining():
         .add_step("rotate-pages", options={"degrees": 90})
         .add_step("watermark-pdf", options={"text": "DRAFT"})
     )
-    
+
     assert result is builder
     assert len(builder._actions) == 3
     # Actions are transformed by _map_tool_to_action, so check structure exists
@@ -47,7 +46,7 @@ def test_builder_set_output_options():
         metadata={"title": "Test Doc"},
         optimize=True
     )
-    
+
     assert result is builder
     assert builder._output_options["metadata"]["title"] == "Test Doc"
     assert builder._output_options["optimize"] is True
@@ -57,7 +56,7 @@ def test_builder_execute_requires_client():
     """Test that execute requires a client."""
     builder = BuildAPIWrapper(None, "test.pdf")
     builder.add_step("convert-to-pdf")
-    
+
     # Without a proper client, this would fail when trying to access client methods
     # We can't test the actual failure without mocking, so just ensure the method exists
-    assert hasattr(builder, "execute")
+    assert hasattr(builder, "execute")

tests/unit/test_file_handler.py

@@ -1,14 +1,9 @@
 """Unit tests for file handling utilities."""
 
 import io
-from pathlib import Path
-
-import pytest
 
 from nutrient_dws.file_handler import (
     prepare_file_input,
-    save_file_output,
-    stream_file_content,
 )
 
 
@@ -58,4 +53,4 @@ class DummyFile:
         pass
 
     size = get_file_size(DummyFile())
-    assert size is None
+    assert size is None

tests/unit/test_http_client.py

@@ -1,8 +1,6 @@
 """Unit tests for HTTPClient."""
 
-import pytest
 
-from nutrient_dws.exceptions import APIError, AuthenticationError, NutrientTimeoutError
 from nutrient_dws.http_client import HTTPClient
 
 
@@ -36,4 +34,4 @@ def test_http_client_context_manager():
     """Test HTTP client can be used as context manager."""
     with HTTPClient(api_key="test-key") as client:
         assert client is not None
-        assert hasattr(client, "_session")
+        assert hasattr(client, "_session")