fix: add missing sort root dict (#4986)

eakmanrq · web-flow · commit 960bacb6462b · 2025-07-17T19:01:54.000-07:00
diff --git a/sqlmesh/core/model/definition.py b/sqlmesh/core/model/definition.py
@@ -2361,7 +2361,7 @@ def create_python_model(
 
     used_variables = {k: v for k, v in (variables or {}).items() if k in referenced_variables}
     if used_variables:
-        python_env[c.SQLMESH_VARS] = Executable.value(used_variables)
+        python_env[c.SQLMESH_VARS] = Executable.value(used_variables, sort_root_dict=True)
 
     return _create_model(
         PythonModel,
diff --git a/tests/core/test_model.py b/tests/core/test_model.py
@@ -6715,7 +6715,7 @@ def model_with_named_variables(
     )
 
     assert python_model.python_env[c.SQLMESH_VARS] == Executable.value(
-        {"test_var_a": "test_value", "start": "2024-01-01"}
+        {"test_var_a": "test_value", "start": "2024-01-01"}, sort_root_dict=True
     )
 
     context = ExecutionContext(mocker.Mock(), {}, None, None)

Original file line number	Diff line number	Diff line change
`@@ -6715,7 +6715,7 @@ def model_with_named_variables(`
`6715`	`6715`	`)`
`6716`	`6716`
`6717`	`6717`	`assert python_model.python_env[c.SQLMESH_VARS] == Executable.value(`
`6718`		`- {"test_var_a": "test_value", "start": "2024-01-01"}`
	`6718`	`+ {"test_var_a": "test_value", "start": "2024-01-01"}, sort_root_dict=True`
`6719`	`6719`	`)`
`6720`	`6720`
`6721`	`6721`	`context = ExecutionContext(mocker.Mock(), {}, None, None)`