fix: have python depends_on actually override (#1628)

Author: eakmanrq

sqlmesh/core/model/definition.py

@@ -1726,7 +1726,7 @@ def create_python_model(
     depends_on = (
         _parse_depends_on(entrypoint, python_env) - {name}
         if depends_on is None and python_env is not None
-        else None
+        else depends_on
     )
     return _create_model(
         PythonModel,

tests/core/test_model.py

@@ -1303,6 +1303,24 @@ def my_model(context, **kwargs):
     )
 
 
+def test_python_model_depends_on() -> None:
+    @model(
+        name="model_with_depends_on", kind="full", columns={'"COL"': "int"}, depends_on=["foo.bar"]
+    )
+    def my_model(context, **kwargs):
+        context.table("foo")
+        context.table(model_name=CONST + ".baz")
+
+    m = model.get_registry()["model_with_depends_on"].model(
+        module_path=Path("."),
+        path=Path("."),
+    )
+
+    # We are not expecting the context.table() calls to be reflected in the model's depends_on since we
+    # explicitly specified the depends_on argument.
+    assert m.depends_on == {"foo.bar"}
+
+
 def test_python_models_returning_sql(assert_exp_eq) -> None:
     config = Config(model_defaults=ModelDefaultsConfig(dialect="snowflake"))
     context = Context(config=config)