test(volume): verify option and error boundaries

Signed-off-by: Aakash Saravanan <aakash.saravanan@databricks.com>

Author: aakash-saravanan-db

CHANGELOG.md

@@ -1,7 +1,7 @@
 # Release History
 
 # Unreleased
-- Support SQL PUT, GET, and REMOVE file transfers with a native kernel exposing `Statement.set_staging_allowed_local_paths`; older wheels reject transfers and continue to support ordinary queries. Streaming PUT remains unsupported in kernel mode.
+- Support SQL PUT, GET, and REMOVE file transfers with a native kernel built from `KERNEL_REV`. Published kernel 1.4.0 does not support these transfers; the release version and dependency floor are pending [kernel #443](https://github.com/databricks/databricks-sql-kernel/issues/443). Older wheels reject transfers and continue to support ordinary queries. Streaming PUT remains unsupported in kernel mode.
 - Fix inline parameters (`use_inline_params=True`) so a sequence directly after `IN` renders as a list again: `x IN %(ids)s` renders as `x IN (1,2,3)` instead of `x IN ARRAY(1,2,3)`, which Spark rejects. Inside the list, tuples render as rows for a multi-column `IN`, and a `set` or `frozenset` is accepted again. Sequences anywhere else still render as `ARRAY(...)` ([#629](https://github.com/databricks/databricks-sql-python/issues/629))
 
 # 4.9.0 (2026-11-04)

CONNECTION_PARAMETERS.md

@@ -171,7 +171,7 @@ to change without notice.
 | `use_inline_params`           | `bool` \| `"silent"`              |   ✅   |   ✅   | `False`       | Render parameters inline (legacy) vs. native bound params (DBR 14.1+). Shared cursor code performs the rendering before dispatch, so both backends honor it identically. |
 | `ignore_transactions`         | `bool`                            |   ✅   |   ✅   | `True`        | When `True`: `commit()` is a no-op, `rollback()` raises `NotSupportedError`, and setting `autocommit` is a no-op.          |
 | `fetch_autocommit_from_server`| `bool`                            |   ✅   |   ✅   | `False`       | Query the server (`SET AUTOCOMMIT`) for autocommit state instead of returning the cached value.                           |
-| `staging_allowed_local_path`  | `str` \| `List[str]`              |   ✅   |   ✅   | `None`        | Local path(s) permitted for Unity Catalog Volume `PUT`/`GET`. Kernel transfers require a wheel exposing `Statement.set_staging_allowed_local_paths` (kernel [#439](https://github.com/databricks/databricks-sql-kernel/pull/439)); older wheels reject transfer commands. Kernel mode does not support streaming PUT.          |
+| `staging_allowed_local_path`  | `str` \| `List[str]`              |   ✅   | Source build | `None`        | Local path(s) permitted for Unity Catalog Volume `PUT`/`GET`. Requires a native kernel built from `KERNEL_REV` with [kernel #439](https://github.com/databricks/databricks-sql-kernel/pull/439). Published 1.4.0 lacks this capability; the release version and dependency floor are pending [kernel #443](https://github.com/databricks/databricks-sql-kernel/issues/443). Older wheels reject transfers. Kernel mode does not support streaming PUT. |
 
 ## Telemetry
 

tests/unit/test_kernel_client.py

@@ -816,7 +816,19 @@ def test_execute_command_forwards_row_limit(row_limit):
     ],
 )
 @pytest.mark.parametrize("async_op", [False, True])
-def test_execute_command_forwards_allowed_local_paths(allowed, expected, async_op):
+@pytest.mark.parametrize(
+    "operation, uses_paths",
+    [
+        ("PUT '/tmp/files/input' INTO '/Volumes/c/s/v/input'", True),
+        ("GET '/Volumes/c/s/v/input' TO '/tmp/files/output'", True),
+        ("/* download */ GET '/Volumes/c/s/v/input' TO '/tmp/files/output'", True),
+        ("REMOVE '/Volumes/c/s/v/input'", False),
+        ("SELECT 1", False),
+    ],
+)
+def test_execute_command_forwards_allowed_local_paths(
+    allowed, expected, async_op, operation, uses_paths
+):
     c = _make_client()
     c._kernel_session = MagicMock()
     cursor = MagicMock()
@@ -831,7 +843,7 @@ def test_execute_command_forwards_allowed_local_paths(allowed, expected, async_o
     stmt.execute.return_value = executed
     stmt.submit.return_value = executed
     c.execute_command(
-        operation="PUT '/tmp/files/input' INTO '/Volumes/c/s/v/input'",
+        operation=operation,
         session_id=MagicMock(),
         max_rows=1,
         max_bytes=1,
@@ -842,7 +854,9 @@ def test_execute_command_forwards_allowed_local_paths(allowed, expected, async_o
         async_op=async_op,
         enforce_embedded_schema_correctness=False,
     )
-    stmt.set_staging_allowed_local_paths.assert_called_once_with(expected)
+    stmt.set_staging_allowed_local_paths.assert_called_once_with(
+        expected if uses_paths else []
+    )
     (stmt.submit if async_op else stmt.execute).assert_called_once()
 
 
@@ -2657,20 +2671,27 @@ def test_only_reserved_headers_omits_kwarg(self, monkeypatch):
 
 
 @pytest.mark.parametrize(
-    "code, http_status, expected",
+    "code, http_status, error_code, expected",
     [
-        ("PermissionDenied", None, ProgrammingError),
-        ("InvalidArgument", None, ProgrammingError),
-        ("PermissionDenied", 403, OperationalError),
-        ("NotFound", None, FileNotFoundError),
-        ("NotFound", 404, OperationalError),
-        ("NetworkError", None, OperationalError),
+        ("PermissionDenied", None, "FILE_TRANSFER_ERROR", ProgrammingError),
+        ("InvalidArgument", None, "FILE_TRANSFER_ERROR", ProgrammingError),
+        ("InvalidArgument", 400, "FILE_TRANSFER_ERROR", OperationalError),
+        ("PermissionDenied", 403, "FILE_TRANSFER_ERROR", OperationalError),
+        ("NotFound", None, "FILE_TRANSFER_ERROR", FileNotFoundError),
+        ("NotFound", 404, "FILE_TRANSFER_ERROR", OperationalError),
+        ("NetworkError", None, "FILE_TRANSFER_ERROR", OperationalError),
+        ("Internal", None, "FILE_TRANSFER_ERROR", OperationalError),
+        ("Internal", None, None, DatabaseError),
     ],
 )
-def test_file_transfer_error_matches_thrift_exception(code, http_status, expected):
+def test_file_transfer_error_matches_thrift_exception(
+    code, http_status, error_code, expected
+):
     error = _FakeKernelError(code=code, message="File transfer failed")
-    error.error_code = "FILE_TRANSFER_ERROR"
+    error.error_code = error_code
     error.http_status = http_status
     result = kernel_client._reraise_kernel_error(error)
     assert type(result) is expected
-    assert result.error_code == "FILE_TRANSFER_ERROR"
+    assert result.error_code == error_code
+    assert result.code == code
+    assert result.http_status == http_status