Fix: ensure paths exist before treating them as relative (metaprogramming) (#4539)

Author: georgesittas

sqlmesh/utils/metaprogramming.py

@@ -37,7 +37,7 @@ def _is_relative_to(path: t.Optional[Path | str], other: t.Optional[Path | str])
     if isinstance(other, str):
         other = Path(other)
 
-    if "site-packages" in str(path):
+    if "site-packages" in str(path) or not path.exists() or not other.exists():
         return False
 
     try:

tests/core/test_model.py

@@ -10129,3 +10129,24 @@ def check_self_schema(evaluator):
 
     context = Context(paths=tmp_path, config=config)
     context.plan(no_prompts=True, auto_apply=True)
+
+
+def test_model_relies_on_os_getenv(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    init_example_project(tmp_path, dialect="duckdb", template=ProjectTemplate.EMPTY)
+
+    (tmp_path / "macros" / "getenv_macro.py").write_text(
+        """
+from os import getenv
+from sqlmesh import macro
+
+@macro()
+def getenv_macro(evaluator):
+    getenv("foo", None)
+    return 1"""
+    )
+    (tmp_path / "models" / "model.sql").write_text(
+        "MODEL (name test); SELECT @getenv_macro() AS foo"
+    )
+
+    monkeypatch.chdir(tmp_path)
+    ctx = Context(paths=tmp_path)