changed the jinja boolean conversions

Author: Michael Myaskovsky

macros/edr/dbt_artifacts/upload_dbt_invocation.sql

@@ -17,7 +17,7 @@
       'command': flags.WHICH,
       'dbt_version': dbt_version,
       'elementary_version': elementary.get_elementary_package_version(),
-      'full_refresh': edr_evaluate_bool(flags.FULL_REFRESH),
+      'full_refresh': edr_evaluate_bool_condition(flags.FULL_REFRESH),
       'invocation_vars': elementary.get_invocation_vars(),
       'vars': elementary.get_all_vars(),
       'target_name': target.name,

macros/edr/dbt_artifacts/upload_run_results.sql

@@ -36,10 +36,14 @@
     {{ return(dbt_run_results_empty_table_query) }}
 {% endmacro %}
 
+{% macro debug_var(var_name, var_value) %}
+    {% do log("🔍 DEBUG — " ~ var_name ~ ": " ~ var_value, info=True) %}
+{% endmacro %}
 {% macro flatten_run_result(run_result) %}
     {% set run_result_dict = elementary.get_run_result_dict(run_result) %}
     {% set node = elementary.safe_get_with_default(run_result_dict, 'node', {}) %}
     {% set config_dict = elementary.safe_get_with_default(node, 'config', {}) %}
+    {{ debug()}}
     {% set flatten_run_result_dict = {
         'model_execution_id': elementary.get_node_execution_id(node),
         'invocation_id': invocation_id,
@@ -55,7 +59,7 @@
         'execute_completed_at': none,
         'compile_started_at': none,
         'compile_completed_at': none,
-        'full_refresh': edr_evaluate_bool(flags.FULL_REFRESH),
+        'full_refresh': edr_evaluate_bool_condition(flags.FULL_REFRESH),
         'compiled_code': elementary.get_compiled_code(node, as_column_value=true),
         'failures': run_result_dict.get('failures'),
         'query_id': run_result_dict.get('adapter_response', {}).get('query_id'),

macros/utils/data_types/bool_type_values.sql

@@ -1,15 +1,4 @@
 {# Evaluate bool - For jinja bool true/false evaluation towards DBs compatible bool type#}
-{% macro edr_evaluate_bool(return_value) -%}
-    {%- if return_value == true -%} {{ edr_bool_true() }}   
-    {%- else -%} {{ edr_bool_false() }}
-    {%- endif -%}
-{%- endmacro %}
-
-{% macro edr_evaluate_bool_condition(return_value) -%}
-    {%- if return_value == true -%} {{ edr_bool_true_condition() }}   
-    {%- else -%} {{ edr_bool_false_condition() }}
-    {%- endif -%}
-{%- endmacro %}
 
 {# Due to "faking" of bool using bit in T-SQL we have distinguish representation of true/false #}
 {# We have different bools for use in parameter wheres vs. in columns as values#}
@@ -21,35 +10,25 @@
     {{ return(adapter.dispatch('edr_bool_true', 'elementary')()) }}
 {%- endmacro %}
 
-{% macro default__edr_bool_true() %}true{% endmacro %}
-
-{% macro fabric__edr_bool_true() %}1{% endmacro %}
-
-
-{% macro edr_bool_true_condition() -%}
-    {{ return(adapter.dispatch('edr_bool_true_condition', 'elementary')()) }}
-{%- endmacro %}
-
-{% macro default__edr_bool_true_condition() %}true{% endmacro %}
-
-{% macro fabric__edr_bool_true_condition() %}1 = 1{% endmacro %}
-
 {# False bools#} 
 
 {% macro edr_bool_false() -%}
     {{ return(adapter.dispatch('edr_bool_false', 'elementary')()) }}
 {%- endmacro %}
 
-{% macro default__edr_bool_false() %}false{% endmacro %}
-
-{% macro fabric__edr_bool_false() %}0{% endmacro %}
+{% macro edr_evaluate_bool_condition(return_value) %}
+{% if return_value == true %}
+{{ return(adapter.dispatch('edr_bool_true', 'elementary')()) }}
+{% else %}
+{{ return(adapter.dispatch('edr_bool_false', 'elementary')()) }}
+{% endif %}
+{%endmacro%}
 
 
+{% macro default__edr_bool_true() %}{%do return(true)%}{% endmacro %}
 
-{% macro edr_bool_false_condition() -%}
-    {{ return(adapter.dispatch('edr_bool_false_condition', 'elementary')()) }}
-{%- endmacro %}
+{% macro fabric__edr_bool_true() %}1{% endmacro %}
 
-{% macro default__edr_bool_false_condition() %}false{% endmacro %}
+{% macro default__edr_bool_false() %}{%do return(false)%}{% endmacro %}
 
-{% macro fabric__edr_bool_false_condition() %}0 = 1{% endmacro %}
+{% macro fabric__edr_bool_false() %}0{% endmacro %}