From 8677a9ae10f80d06923d3d7283a2011bda2ab1f1 Mon Sep 17 00:00:00 2001
From: Michelle Ark <michelle.ark@dbtlabs.com>
Date: Wed, 25 Sep 2024 17:35:09 +0100
Subject: [PATCH] simplify constructing unrendered_config from jinja

---
 core/dbt/clients/jinja_static.py | 53 +++-----------------------------
 1 file changed, 5 insertions(+), 48 deletions(-)

diff --git a/core/dbt/clients/jinja_static.py b/core/dbt/clients/jinja_static.py
index 3cf277977ed..3d32b3603c9 100644
--- a/core/dbt/clients/jinja_static.py
+++ b/core/dbt/clients/jinja_static.py
@@ -200,7 +200,7 @@ def statically_parse_unrendered_config(string: str) -> Optional[Dict[str, Any]]:
 
     For example, given:
     "{{ config(materialized=env_var('DBT_TEST_STATE_MODIFIED')) }}\nselect 1 as id"
-    returns: {"materialized" : "env_var('DBT_TEST_STATE_MODIFIED')"}
+    returns: {'materialized': "Keyword(key='materialized', value=Call(node=Name(name='env_var', ctx='load'), args=[Const(value='DBT_TEST_STATE_MODIFIED')], kwargs=[], dyn_args=None, dyn_kwargs=None))"}
 
     No config call:
     "select 1 as id"
@@ -241,50 +241,7 @@ def statically_parse_unrendered_config(string: str) -> Optional[Dict[str, Any]]:
 
 
 def construct_static_kwarg_value(kwarg):
-    static_kwarg_value = _construct_static_kwarg_value(kwarg)
-    if isinstance(static_kwarg_value, str):
-        return static_kwarg_value.strip("'")
-    else:
-        return static_kwarg_value
-
-
-def _construct_static_kwarg_value(kwarg) -> str:
-    kwarg_type = type(kwarg.value).__name__
-    if kwarg_type == "Const":
-        return (
-            f"'{kwarg.value.value}'" if isinstance(kwarg.value.value, str) else kwarg.value.value
-        )
-
-    elif kwarg_type == "Call":
-        unrendered_args = []
-        for arg in kwarg.value.args:
-            arg_type = type(arg).__name__
-            if arg_type == "Const":
-                unrendered_args.append(
-                    f"'{arg.value}'" if isinstance(arg.value, str) else arg.value
-                )
-
-        unrendered_kwargs = {}
-        for call_kwarg in kwarg.value.kwargs:
-            kwarg_value = _construct_static_kwarg_value(call_kwarg)
-
-            unrendered_kwargs[call_kwarg.key] = kwarg_value
-
-        formatted_unrendered_kwargs = [
-            f"{key}={value}" for key, value in unrendered_kwargs.items()
-        ]
-
-        formatted_all_args = ", ".join(
-            str(x) for x in (unrendered_args + formatted_unrendered_kwargs)
-        )
-
-        return f"{kwarg.value.node.name}({formatted_all_args})"
-
-    elif kwarg_type == "CondExpr":
-        # Instead of trying to re-assemble complex kwarg value, simply stringify the value
-        # This is still useful to be able to detect changes in unrendered configs, even if it is
-        # not an exact representation of the user input.
-        return str(kwarg.value)
-
-    else:
-        return ""
+    # Instead of trying to re-assemble complex kwarg value, simply stringify the value
+    # This is still useful to be able to detect changes in unrendered configs, even if it is
+    # not an exact representation of the user input.
+    return str(kwarg)