Clean up code quality: remove unused imports, dead code, compile inline regex

- Remove unused imports in test_batch_edge_cases.py (_RecordCreate, _RecordUpdate, _QuerySql, _TableList)
- Remove unused BatchResult import in test_batch_serialization.py
- Remove dead _mock_batch_response function in test_batch_scenarios.py
- Compile boundary regex to module-level _BOUNDARY_RE constant in _batch.py

Author: Saurabh Badenkal

src/PowerPlatform/Dataverse/data/_batch.py

@@ -594,8 +594,11 @@ def _raise_top_level_batch_error(response: Any) -> None:
     )
 
 
+_BOUNDARY_RE = re.compile(r'boundary="?([^";,\s]+)"?', re.IGNORECASE)
+
+
 def _extract_boundary(content_type: str) -> Optional[str]:
-    m = re.search(r'boundary="?([^";,\s]+)"?', content_type, re.IGNORECASE)
+    m = _BOUNDARY_RE.search(content_type)
     return m.group(1) if m else None
 
 

tests/unit/data/test_batch_edge_cases.py

@@ -16,12 +16,8 @@
     _BatchClient,
     _ChangeSet,
     _ChangeSetBatchItem,
-    _RecordCreate,
     _RecordDelete,
     _RecordGet,
-    _RecordUpdate,
-    _QuerySql,
-    _TableList,
     _extract_boundary,
     _raise_top_level_batch_error,
     _split_multipart,

tests/unit/data/test_batch_serialization.py

@@ -29,7 +29,7 @@
 from PowerPlatform.Dataverse.core.errors import HttpError
 from PowerPlatform.Dataverse.models.upsert import UpsertItem
 from PowerPlatform.Dataverse.data._raw_request import _RawRequest
-from PowerPlatform.Dataverse.models.batch import BatchItemResponse, BatchResult
+from PowerPlatform.Dataverse.models.batch import BatchItemResponse
 
 
 def _make_od():

tests/unit/test_batch_scenarios.py

@@ -28,50 +28,6 @@ def _make_od():
     return od
 
 
-def _mock_batch_response(batch_boundary, parts):
-    """Build a mock HTTP response from a list of (status, headers_dict, body_str) tuples."""
-    body_parts = []
-    for status_line, headers, body in parts:
-        lines = [
-            f"--{batch_boundary}",
-            "Content-Type: application/http",
-            "Content-Transfer-Encoding: binary",
-        ]
-        for k, v in (headers or {}).items():
-            lines.append(f"{k}: {v}")
-        lines.append("")
-        lines.append(status_line)
-        for k, v in (headers or {}).items():
-            if k.lower() != "content-id":
-                pass  # handled below
-        # Add response headers that contain OData-EntityId, Content-Type, etc.
-        resp_lines = [status_line]
-        if body:
-            resp_lines.append("Content-Type: application/json; odata.metadata=minimal")
-        resp_lines.append("OData-Version: 4.0")
-        # Add OData-EntityId if present in headers
-        for k, v in (headers or {}).items():
-            resp_lines.append(f"{k}: {v}")
-        resp_lines.append("")
-        if body:
-            resp_lines.append(body)
-        resp_text = "\r\n".join(resp_lines)
-        part = (
-            f"--{batch_boundary}\r\n"
-            "Content-Type: application/http\r\n"
-            "Content-Transfer-Encoding: binary\r\n"
-            "\r\n" + resp_text + "\r\n"
-        )
-        body_parts.append(part)
-    body_parts.append(f"--{batch_boundary}--\r\n")
-    full_body = "".join(body_parts)
-
-    mock_resp = MagicMock()
-    mock_resp.headers = {"Content-Type": f'multipart/mixed; boundary="{batch_boundary}"'}
-    mock_resp.text = full_body
-    return mock_resp
-
-
 # ---------------------------------------------------------------------------
 # Scenario 1: Response ordering matches operation order
 # ---------------------------------------------------------------------------