Skip to content

Commit 3842f8a

Browse files
committed
chore: remove regression tests unrelated to this PR
The TestDatabricksCredentialsPreDeserialize tests covered pre-existing setdefaults (_retry_stop_after_attempts_count, _retry_delay_max) that this PR does not modify, making them out of scope. Co-authored-by: Isaac
1 parent baeba85 commit 3842f8a

1 file changed

Lines changed: 0 additions & 27 deletions

File tree

tests/unit/test_adapter.py

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414
from dbt.adapters.databricks.column import DatabricksColumn
1515
from dbt.adapters.databricks.credentials import (
1616
CATALOG_KEY_IN_SESSION_PROPERTIES,
17-
DatabricksCredentials,
1817
)
1918
from dbt.adapters.databricks.impl import (
2019
DatabricksRelationInfo,
@@ -1352,29 +1351,3 @@ def test_is_uniform_with_invalid_materialization_error(
13521351
DbtConfigError, match="When table_format is 'iceberg', materialized must be"
13531352
):
13541353
adapter.is_uniform(mock_config)
1355-
1356-
1357-
class TestDatabricksCredentialsPreDeserialize:
1358-
"""Tests for DatabricksCredentials.__pre_deserialize__ retry defaults."""
1359-
1360-
def test_pre_deserialize__default_retry_params(self):
1361-
"""Verify retry defaults are set when connection_parameters is empty."""
1362-
data = {"connection_parameters": {}}
1363-
result = DatabricksCredentials.__pre_deserialize__(data)
1364-
assert result["connection_parameters"]["_retry_stop_after_attempts_count"] == 30
1365-
assert result["connection_parameters"]["_retry_delay_max"] == 60
1366-
1367-
def test_pre_deserialize__missing_connection_parameters(self):
1368-
"""Verify retry defaults are set even when connection_parameters key is absent."""
1369-
data: dict[Any, Any] = {}
1370-
result = DatabricksCredentials.__pre_deserialize__(data)
1371-
assert result["connection_parameters"]["_retry_stop_after_attempts_count"] == 30
1372-
assert result["connection_parameters"]["_retry_delay_max"] == 60
1373-
1374-
def test_pre_deserialize__custom_params_preserve_retry_defaults(self):
1375-
"""Verify unrelated connection_parameters don't interfere with retry defaults."""
1376-
data = {"connection_parameters": {"custom_param": "value"}}
1377-
result = DatabricksCredentials.__pre_deserialize__(data)
1378-
assert result["connection_parameters"]["custom_param"] == "value"
1379-
assert result["connection_parameters"]["_retry_stop_after_attempts_count"] == 30
1380-
assert result["connection_parameters"]["_retry_delay_max"] == 60

0 commit comments

Comments
 (0)