Following up on drew s PR review

README.md

@@ -364,7 +364,10 @@ Arguments:
 
     - table: Table name, required
     - cast_to: The data type to cast the unpivoted values to, default is varchar
-    - exclude: A list of columns to exclude from the unpivot.
+    - exclude: A list of columns to exclude from the unpivot operation but keep in the resulting table.
+    - remove: A list of columns to remove from the resulting table
+    - field_name: column name in the resulting table for field
+    - value_name: column name in the resulting table for value
 
 ---
 ### Web

integration_tests/data/sql/data_unpivot_expected.csv

@@ -1,7 +1,10 @@
 customer_id,created_at,prop,val
-123,2017-01-01,status,active
-123,2017-01-01,segment,tier 1
-234,2017-02-01,status,active
-234,2017-02-01,segment,tier 3
-567,2017-03-01,status,churned
-567,2017-03-01,segment,tier 2
+123,"2017-01-01","segment","tier 1"
+123,"2017-01-01","status","active"
+123,"2017-01-01","name","name 1"
+234,"2017-02-01","segment","tier 3"
+234,"2017-02-01","status","active"
+234,"2017-02-01","name","name 3"
+567,"2017-03-01","status","churned"
+567,"2017-03-01","name","name 2"
+567,"2017-03-01","segment","tier 2"

integration_tests/models/sql/schema.yml

@@ -85,6 +85,11 @@ models:
       - dbt_utils.equality:
           compare_model: ref('data_unpivot_original_api_expected')
 
+  - name: test_unpivot
+    tests:
+      - dbt_utils.equality:
+          compare_model: ref('data_unpivot_expected')
+
   - name: test_star
     tests:
       - dbt_utils.equality:

integration_tests/models/sql/test_unpivot_original_api.sql

@@ -18,7 +18,7 @@ select
     customer_id,
     created_at,
     case
-        when '{{ target.name }}' = 'snowflake' then lower(field_name)
+        when '{{ target.name }}' = 'snowflake' then lower(FIELD_NAME)
         else field_name
     end as field_name,
     value

macros/sql/unpivot.sql

@@ -39,8 +39,8 @@ Arguments:
         {{ exclude_col }},
       {%- endfor %}
 
-      cast('{{ col.column }}' as {{ dbt_utils.type_string() }}) as "{{ field_name }}",
-      cast({{ col.column }} as {{ cast_to }}) as "{{ value_name }}"
+      cast('{{ col.column }}' as {{ dbt_utils.type_string() }}) as {{ field_name }},
+      cast({{ col.column }} as {{ cast_to }}) as {{ value_name }}
 
     from {{ table }}