Patch #32921
Rails 6: add header=present to csv tests
Status: | New | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | Rails support | |||
Target version: | 5.0.0 |
Description
fixes
Failure: IssuesControllerTest#test_index_csv_with_all_columns [/redmine/test/functional/issues_controller_test.rb:878]: Expected: "text/csv" Actual: "text/csv; header=present"
Related issues
History
#1
Updated by Pavel Rosický 12 months ago
- File csvheaders2.patch
added
#2
Updated by Marius BALTEANU 12 months ago
- Related to Feature #29914: Migrate to Rails 6 added
#3
Updated by Marius BALTEANU 12 months ago
Pavel, maybe you have a clue because I can't find or understand why media_type
returns "text/csv; header=present" and not only text/csv
as is explained here.
#4
Updated by Pavel Rosický 12 months ago
yeah, because the documentation is obsolete, it was changed here https://github.com/rails/rails/pull/37017
#5
Updated by Marius BALTEANU 12 months ago
- File 0001-Rails-6-add-header-present-to-csv-tests.patch
added
- Target version set to 5.0.0
Pavel Rosický wrote:
yeah, because the documentation is obsolete, it was changed here https://github.com/rails/rails/pull/37017
Thanks, it's clear now. Your patches look good to me and the failing tests because of this are passing now. I'm attaching a merged version of your patches.