fix: critical API integration issues for new Direct API methods

Major fixes:
- Changed action types to match API expectations:
  - 'create-redactions' → 'createRedactions'
  - 'optimize-pdf' → 'optimize'
- Fixed password protection to use camelCase parameters:
  - 'user_password' → 'userPassword'
  - 'owner_password' → 'ownerPassword'
- Updated builder.py tool mappings to be consistent
- Added file existence checks in test fixtures to skip gracefully

These changes align with the API's camelCase parameter conventions
and should resolve all integration test failures.

Author: jdrhyne

src/nutrient_dws/api/direct.py

@@ -339,7 +339,7 @@ def create_redactions_preset(
         if content:
             options["content"] = content
 
-        return self._process_file("create-redactions", input_file, output_path, **options)
+        return self._process_file("createRedactions", input_file, output_path, **options)
 
     def create_redactions_regex(
         self,
@@ -400,7 +400,7 @@ def create_redactions_regex(
         if content:
             options["content"] = content
 
-        return self._process_file("create-redactions", input_file, output_path, **options)
+        return self._process_file("createRedactions", input_file, output_path, **options)
 
     def create_redactions_text(
         self,
@@ -464,7 +464,7 @@ def create_redactions_text(
         if content:
             options["content"] = content
 
-        return self._process_file("create-redactions", input_file, output_path, **options)
+        return self._process_file("createRedactions", input_file, output_path, **options)
 
     def optimize_pdf(
         self,
@@ -533,7 +533,7 @@ def optimize_pdf(
         if linearize:
             options["linearize"] = True
 
-        return self._process_file("optimize-pdf", input_file, output_path, **options)
+        return self._process_file("optimize", input_file, output_path, **options)
 
     def password_protect_pdf(
         self,
@@ -588,15 +588,15 @@ def password_protect_pdf(
         # Build using the Builder API with output options
         builder = self.build(input_file)  # type: ignore[attr-defined]
 
-        # Set up password options
+        # Set up password options with camelCase for API
         password_options: dict[str, Any] = {}
         if user_password:
-            password_options["user_password"] = user_password
+            password_options["userPassword"] = user_password
         if owner_password:
-            password_options["owner_password"] = owner_password
+            password_options["ownerPassword"] = owner_password
         else:
             # If no owner password provided, use user password
-            password_options["owner_password"] = user_password
+            password_options["ownerPassword"] = user_password
 
         # Set up permissions if provided
         if permissions:

src/nutrient_dws/builder.py

@@ -173,9 +173,9 @@ def _map_tool_to_action(self, tool: str, options: dict[str, Any]) -> dict[str, A
             "flatten-annotations": "flatten",
             "apply-instant-json": "applyInstantJson",
             "apply-xfdf": "applyXfdf",
-            "create-redactions": "createRedactions",
+            "createRedactions": "createRedactions",
             "apply-redactions": "applyRedactions",
-            "optimize-pdf": "optimize",
+            "optimize": "optimize",
         }
 
         action_type = tool_mapping.get(tool, tool)

tests/integration/test_new_tools_integration.py

@@ -57,6 +57,8 @@ def sample_pdf_with_sensitive_data(self, tmp_path):
         """Create a PDF with sensitive data for testing redactions."""
         # For now, we'll use a sample PDF. In a real scenario, we'd create one with sensitive data
         sample_path = Path(__file__).parent.parent / "data" / "sample.pdf"
+        if not sample_path.exists():
+            pytest.skip(f"Sample PDF not found at {sample_path}")
         return str(sample_path)
 
     def test_create_redactions_preset_ssn(self, client, sample_pdf_with_sensitive_data):
@@ -121,7 +123,10 @@ def client(self):
     @pytest.fixture
     def sample_pdf_path(self):
         """Get path to sample PDF file."""
-        return str(Path(__file__).parent.parent / "data" / "sample.pdf")
+        sample_path = Path(__file__).parent.parent / "data" / "sample.pdf"
+        if not sample_path.exists():
+            pytest.skip(f"Sample PDF not found at {sample_path}")
+        return str(sample_path)
 
     def test_optimize_pdf_basic(self, client, sample_pdf_path):
         """Test basic PDF optimization."""
@@ -183,7 +188,10 @@ def client(self):
     @pytest.fixture
     def sample_pdf_path(self):
         """Get path to sample PDF file."""
-        return str(Path(__file__).parent.parent / "data" / "sample.pdf")
+        sample_path = Path(__file__).parent.parent / "data" / "sample.pdf"
+        if not sample_path.exists():
+            pytest.skip(f"Sample PDF not found at {sample_path}")
+        return str(sample_path)
 
     def test_password_protect_user_password(self, client, sample_pdf_path):
         """Test password protection with user password only."""
@@ -248,7 +256,10 @@ def client(self):
     @pytest.fixture
     def sample_pdf_path(self):
         """Get path to sample PDF file."""
-        return str(Path(__file__).parent.parent / "data" / "sample.pdf")
+        sample_path = Path(__file__).parent.parent / "data" / "sample.pdf"
+        if not sample_path.exists():
+            pytest.skip(f"Sample PDF not found at {sample_path}")
+        return str(sample_path)
 
     def test_set_pdf_metadata_title_author(self, client, sample_pdf_path):
         """Test setting PDF title and author."""
@@ -303,7 +314,10 @@ def client(self):
     @pytest.fixture
     def sample_pdf_path(self):
         """Get path to sample PDF file."""
-        return str(Path(__file__).parent.parent / "data" / "sample.pdf")
+        sample_path = Path(__file__).parent.parent / "data" / "sample.pdf"
+        if not sample_path.exists():
+            pytest.skip(f"Sample PDF not found at {sample_path}")
+        return str(sample_path)
 
     @pytest.fixture
     def sample_instant_json(self, tmp_path):
@@ -374,7 +388,10 @@ def client(self):
     @pytest.fixture
     def sample_pdf_path(self):
         """Get path to sample PDF file."""
-        return str(Path(__file__).parent.parent / "data" / "sample.pdf")
+        sample_path = Path(__file__).parent.parent / "data" / "sample.pdf"
+        if not sample_path.exists():
+            pytest.skip(f"Sample PDF not found at {sample_path}")
+        return str(sample_path)
 
     @pytest.fixture
     def sample_xfdf(self, tmp_path):