Handle deprecations in deduplicate macro

resolves #668

Author: miles170

CHANGELOG.md

@@ -18,6 +18,9 @@
 ## Under the hood
 - Remove deprecated table argument from unpivot ([#671](https://github.com/dbt-labs/dbt-utils/pull/671))
 
+## Under the hood
+- Handle deprecations in deduplicate macro ([#673](https://github.com/dbt-labs/dbt-utils/pull/673))
+
 ## Fixes
 - Better handling of whitespaces in the star macro ([#651](https://github.com/dbt-labs/dbt-utils/pull/651))
 - Fix to correct behavior in `mutually_exclusive_ranges` test in certain situations when `zero_length_range_allowed: true` and multiple ranges in a partition have the same value for `lower_bound_column`. ([[#659](https://github.com/dbt-labs/dbt-utils/issues/659)], [#660](https://github.com/dbt-labs/dbt-utils/pull/660))
@@ -27,7 +30,7 @@
 - [@epapineau](https://github.com/epapineau) (#634)
 - [@courentin](https://github.com/courentin) (#651)
 - [@sfc-gh-ancoleman](https://github.com/sfc-gh-ancoleman) (#660)
-- [@miles170](https://github.com/miles170) (#671)
+- [@miles170](https://github.com/miles170)
 - [@emilyriederer](https://github.com/emilyriederer) 
 
 # dbt-utils v0.8.6

integration_tests/models/sql/schema.yml

@@ -188,11 +188,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
@@ -218,4 +213,3 @@ models:
       - assert_equal:
           actual: actual
           expected: expected
-

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 %}