fix: full_refresh is expected to be None as a default value but the v… (#2978)

Author: tobymao

sqlmesh/dbt/basemodel.py

@@ -26,7 +26,6 @@
 from sqlmesh.dbt.relation import Policy, RelationType
 from sqlmesh.dbt.test import TestConfig
 from sqlmesh.utils import AttributeDict
-from sqlmesh.utils.conversions import ensure_bool
 from sqlmesh.utils.errors import ConfigError
 from sqlmesh.utils.pydantic import field_validator
 
@@ -144,11 +143,6 @@ def _validate_hooks(cls, v: t.Union[str, t.List[t.Union[SqlStr, str]]]) -> t.Lis
 
         return hooks
 
-    @field_validator("full_refresh", mode="before")
-    @classmethod
-    def _validate_bool(cls, v: str) -> bool:
-        return ensure_bool(v)
-
     @field_validator("grants", mode="before")
     @classmethod
     def _validate_grants(cls, v: t.Dict[str, str]) -> t.Dict[str, t.List[str]]:

tests/dbt/test_transformation.py

@@ -229,6 +229,10 @@ def test_model_kind():
         materialized=Materialization.INCREMENTAL, incremental_strategy="append"
     ).model_kind(context) == IncrementalUnmanagedKind(disable_restatement=False)
 
+    assert ModelConfig(
+        materialized=Materialization.INCREMENTAL, incremental_strategy="append", full_refresh=None
+    ).model_kind(context) == IncrementalUnmanagedKind(disable_restatement=False)
+
     assert ModelConfig(
         materialized=Materialization.INCREMENTAL,
         incremental_strategy="insert_overwrite",