Actions
Defect #32525
closed
CSV related tests fail with Rails 5.2.4
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Affected version:
Description
22 CSV related tests fail with Rails 5.2.4.
Failure: TimelogControllerTest#test_index_csv_all_projects [/Users/maeda/redmines/redmine-trunk/test/functional/timelog_controller_test.rb:1386]: Expected: "text/csv" Actual: "text/csv; header=present" bin/rails test test/functional/timelog_controller_test.rb:1382 . . .
diff --git a/Gemfile b/Gemfile
index ec61c6950..25da95c7e 100644
--- a/Gemfile
+++ b/Gemfile
@@ -2,7 +2,7 @@ source 'https://rubygems.org'
gem "bundler", ">= 1.5.0"
-gem "rails", "5.2.3"
+gem "rails", "5.2.4"
gem "rouge", "~> 3.13.0"
gem "request_store", "1.0.5"
gem "mini_mime", "~> 1.0.1"
Files
Related issues
Updated by Go MAEDA about 5 years ago
- Related to Feature #32526: Update Rails to 5.2.4.1 added
Updated by Go MAEDA about 5 years ago
We changed the tests to expect the content type as "text/csv" instead of "text/csv; header=present" in r18028 in order to follow the change https://github.com/rails/rails/pull/35549, but the change in Rails was reverted by https://github.com/rails/rails/pull/35706/files.
Updated by Go MAEDA about 5 years ago
- File 32525.patch 32525.patch added
The code after applying the attached patch supports both Rails 5.2.3 and Rails 5.2.4.
Updated by Go MAEDA about 5 years ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed as a part of #32526.
Actions