fix default value check for scalar functions

Author: sdebruyn

src/dbt/include/fabric/macros/materializations/functions/scalar.sql

@@ -10,8 +10,9 @@
     {% set args = [] %}
     {% for arg in model.arguments -%}
         {%- set arg_str = '@' ~ arg.name ~ ' ' ~ arg.data_type -%}
-        {%- if arg.default_value is not none -%}
-            {% set arg_str = arg_str ~ ' = ' ~ arg.default_value %}
+        {%- set default_value = arg.get('default_value', none) -%}
+        {%- if default_value is not none -%}
+            {%- set arg_str = arg_str ~ ' = ' ~ arg.default_value -%}
         {%- endif -%}
         {%- do args.append(arg_str) -%}
     {%- endfor %}
@@ -29,4 +30,13 @@
     BEGIN
        RETURN ({{ model.compiled_code }});
     END
+{% endmacro %}
+
+{% macro fabric__scalar_function_volatility_sql() %}
+    {% set volatility = model.config.get('volatility') %}
+    {% if volatility != none %}
+        {# This shouldn't happen unless a new volatility is invented #}
+        {% do unsupported_volatility_warning(volatility) %}
+    {% endif %}
+    {# If no volatility is set, don't add anything and let the data warehouse default it #}
 {% endmacro %}

tests/functional/adapter/test_functions.py

@@ -75,7 +75,7 @@ def test_udfs(self, project, sql_event_catcher):
         assert result.results[0].agate_table.rows[0].values()[0] == 200
 
 
-@pytest.mark.skip(reason="Functions are not fully supported yet.")
+@pytest.mark.skip(reason="Aggregate functions don't exist in T-SQL.")
 class TestBasicSQLUDAFFabric(BasicSQLUDAF):
     pass