Revert smarter_csv duplicate header behavior to version 1.9

SplitTime · Mar 14, 2024 · 4beead8 · 4beead8
1 parent 341c11d
commit 4beead8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/etl/extractors/csv_file_strategy.rb b/lib/etl/extractors/csv_file_strategy.rb
@@ -11,11 +11,12 @@ class CsvFileStrategy
       BYTE_ORDER_MARK = String.new("\xEF\xBB\xBF").force_encoding("UTF-8").freeze
       IMPORT_OPTIONS = {
         downcase_header: false,
+        duplicate_header_suffix: nil,
         force_utf8: true,
         remove_empty_values: false,
         row_sep: :auto,
         strip_chars_from_headers: BYTE_ORDER_MARK,
-        strings_as_keys: true
+        strings_as_keys: true,
       }.freeze
 
       attr_reader :errors

diff --git a/spec/lib/etl/extractors/csv_file_strategy_spec.rb b/spec/lib/etl/extractors/csv_file_strategy_spec.rb
@@ -96,7 +96,7 @@
 
         error = subject.errors.first
         expect(error[:title]).to eq("CSV error")
-        expect(error[:detail][:messages]).to include("ERROR: duplicate headers: country,country")
+        expect(error[:detail][:messages]).to include('Duplicate Headers in CSV: {"country"=>2}')
       end
     end
   end