Fix: Set allow_partials models for all dbt model kinds, including full refresh models (#4060)

Author: izeigerman

sqlmesh/dbt/model.py

@@ -556,10 +556,12 @@ def to_sqlmesh(
 
         kind = self.model_kind(context)
         allow_partials = model_kwargs.pop("allow_partials", None)
-        if allow_partials is None and (
-            kind.is_incremental_unmanaged or kind.is_incremental_by_unique_key
+        if (
+            allow_partials is None
+            and kind.is_materialized
+            and not kind.is_incremental_by_time_range
         ):
-            # Set allow_partials to True for dbt incremental models to preserve the original semantics.
+            # Set allow_partials to True for dbt models to preserve the original semantics.
             allow_partials = True
 
         model = create_sql_model(

tests/dbt/test_transformation.py

@@ -1461,7 +1461,7 @@ def test_refs_in_jinja_globals(sushi_test_project: Project, mocker: MockerFixtur
     }
 
 
-def test_dbt_incremental_allow_partials_by_default():
+def test_allow_partials_by_default():
     context = DbtContext()
     context._target = SnowflakeConfig(
         name="target",
@@ -1481,7 +1481,7 @@ def test_dbt_incremental_allow_partials_by_default():
         materialized=Materialization.TABLE.value,
     )
     assert model.allow_partials is None
-    assert not model.to_sqlmesh(context).allow_partials
+    assert model.to_sqlmesh(context).allow_partials
 
     model.materialized = Materialization.INCREMENTAL.value
     assert model.allow_partials is None