fix: bigquery actually requires "MATERIALIZED VIEW" in DCL for

actual materialized views

Author: newtonapple

sqlmesh/core/engine_adapter/bigquery.py

@@ -1377,8 +1377,13 @@ def _get_grant_expression(self, table: exp.Table) -> exp.Expression:
 
     @staticmethod
     def _grant_object_kind(table_type: DataObjectType) -> str:
-        if table_type == DataObjectType.VIEW or table_type == DataObjectType.MATERIALIZED_VIEW:
+        if table_type == DataObjectType.VIEW:
             return "VIEW"
+        if table_type == DataObjectType.MATERIALIZED_VIEW:
+            # We actually need to use "MATERIALIZED VIEW" here even though it's not listed
+            # as a supported resource_type in the BigQuery DCL doc:
+            # https://cloud.google.com/bigquery/docs/reference/standard-sql/data-control-language
+            return "MATERIALIZED VIEW"
         return "TABLE"
 
     def _dcl_grants_config_expr(

tests/core/engine_adapter/test_bigquery.py

@@ -1336,7 +1336,7 @@ def test_sync_grants_config_with_overlaps(
     [
         (DataObjectType.TABLE, "TABLE"),
         (DataObjectType.VIEW, "VIEW"),
-        (DataObjectType.MATERIALIZED_VIEW, "TABLE"),
+        (DataObjectType.MATERIALIZED_VIEW, "MATERIALIZED VIEW"),
     ],
 )
 def test_sync_grants_config_object_kind(