Fix: handle Paren in depends_on validator (#3243)

Author: georgesittas

sqlmesh/core/model/common.py

@@ -93,6 +93,9 @@ def depends_on(cls: t.Type, v: t.Any, values: t.Dict[str, t.Any]) -> t.Optional[
     dialect = values.get("dialect")
     default_catalog = values.get("default_catalog")
 
+    if isinstance(v, exp.Paren):
+        v = v.unnest()
+
     if isinstance(v, (exp.Array, exp.Tuple)):
         return {
             normalize_model_name(

tests/core/test_model.py

@@ -2661,17 +2661,17 @@ def test_missing_schema_warnings():
 
 
 def test_user_provided_depends_on():
-    expressions = d.parse(
-        """
-        MODEL (name db.table, depends_on [table_b]);
-
-        SELECT a FROM table_a
-        """
-    )
+    for l_delim, r_delim in (("(", ")"), ("[", "]")):
+        expressions = d.parse(
+            f"""
+            MODEL (name db.table, depends_on {l_delim}table_b{r_delim});
 
-    model = load_sql_based_model(expressions)
+            SELECT a FROM table_a
+            """
+        )
 
-    assert model.depends_on == {'"table_a"', '"table_b"'}
+        model = load_sql_based_model(expressions)
+        assert model.depends_on == {'"table_a"', '"table_b"'}, f"Delimiters {l_delim}, {r_delim}"
 
 
 def test_check_schema_mapping_when_rendering_at_runtime(assert_exp_eq):