Merge pull request leikind#4 from kickbooster/render_text

Fix 'render text:' deprecation

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## Unreleased
 
 * Add option `filter_control_options` to columns. Initial use is to pass in the options `start_year`, `end_year`, and `max_year_allowed` when using `rails_datetime_helper` - [#12](https://github.com/patricklindsay/wice_grid/pull/18)
+* Fix deprecation (in Rails 5.0.x) and incorrect behaviour (in Rails 5.1.x) of CSV exports
 
 ## 4.0.1 (31 May, 2018)
 

lib/wice/wice_grid_controller.rb

@@ -179,7 +179,7 @@ def send_file_rails2(path, options = {}) #:nodoc:
       @performed_render = false
 
       logger.info "Sending file #{path}" unless logger.nil?
-      File.open(path, 'rb') { |file| render status: options[:status], text: file.read }
+      File.open(path, 'rb') { |file| render status: options[:status], plain: file.read }
     end
 
     DEFAULT_SEND_FILE_OPTIONS_RAILS2 = { #:nodoc:

spec/features/csv_export_request_spec.rb

@@ -7,12 +7,8 @@
   end
 
   it 'should export csv' do
-    # no idea how to test these
-
-    # find(:css, 'button.wg-external-csv-export-button').click
-
-    select 'Urgent', from: 'g1_f_priority_id'
-
-    # find(:css, 'button.div.clickable.export-to-csv-button').click
+    find(:css, 'button.wg-external-csv-export-button').click
+    sleep 1
+    expect(page.body).to include('ID;Title;Priority;Status;Project Name;')
   end
 end