Handle deprecations in deduplicate macro (#673)

resolves #668

Author: miles170

CHANGELOG.md

@@ -17,6 +17,7 @@
 
 ## Under the hood
 - Remove deprecated table argument from unpivot ([#671](https://github.com/dbt-labs/dbt-utils/pull/671))
+- Handle deprecations in deduplicate macro ([#673](https://github.com/dbt-labs/dbt-utils/pull/673))
 
 ## Under the hood
 - Fully remove varargs usage in surrogate_key and safe_add ([#674](https://github.com/dbt-labs/dbt-utils/pull/674))

integration_tests/models/sql/schema.yml

@@ -185,11 +185,6 @@ models:
       - dbt_utils.equality:
           compare_model: ref('data_deduplicate_expected')
 
-  - name: test_deduplicate_deprecated
-    tests:
-      - dbt_utils.equality:
-          compare_model: ref('data_deduplicate_expected')
-
   - name: test_not_empty_string_failing
     columns:
       - name: string_trim_whitespace_true

integration_tests/models/sql/test_deduplicate_deprecated.sql

@@ -1,39 +1,4 @@
-{%- macro deduplicate(relation, partition_by, order_by=none, relation_alias=none) -%}
-
-    {%- set error_message_group_by -%}
-Warning: the `group_by` parameter of the `deduplicate` macro is no longer supported and will be deprecated in a future release of dbt-utils.
-Use `partition_by` instead.
-The {{ model.package_name }}.{{ model.name }} model triggered this warning.
-    {%- endset -%}
-
-    {% if kwargs.get('group_by') %}
-    {%- do exceptions.warn(error_message_group_by) -%}
-    {%- endif -%}
-
-    {%- set error_message_order_by -%}
-Warning: `order_by` as an optional parameter of the `deduplicate` macro is no longer supported and will be deprecated in a future release of dbt-utils.
-Supply a non-null value for `order_by` instead.
-The {{ model.package_name }}.{{ model.name }} model triggered this warning.
-    {%- endset -%}
-
-    {% if not order_by %}
-    {%- do exceptions.warn(error_message_order_by) -%}
-    {%- endif -%}
-
-    {%- set error_message_alias -%}
-Warning: the `relation_alias` parameter of the `deduplicate` macro is no longer supported and will be deprecated in a future release of dbt-utils.
-If you were using `relation_alias` to point to a CTE previously then you can now pass the alias directly to `relation` instead.
-The {{ model.package_name }}.{{ model.name }} model triggered this warning.
-    {%- endset -%}
-
-    {% if relation_alias %}
-    {%- do exceptions.warn(error_message_alias) -%}
-    {%- endif -%}
-
-    {% set partition_by = partition_by or kwargs.get('group_by') %}
-    {% set relation = relation_alias or relation %}
-    {% set order_by = order_by or "'1'" %}
-
+{%- macro deduplicate(relation, partition_by, order_by) -%}
     {{ return(adapter.dispatch('deduplicate', 'dbt_utils')(relation, partition_by, order_by)) }}
 {% endmacro %}