Project

General

Profile

Actions

Patch #31113

closed

Update Rails to 5.2.3

Added by Marius BĂLTEANU over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Rails support
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:


Files

0001-Update-Rails-to-5.2.3.patch (9.66 KB) 0001-Update-Rails-to-5.2.3.patch Marius BĂLTEANU, 2019-03-30 15:59

Related issues

Has duplicate Redmine - Feature #31112: Update to Rails 5.2.3Closed

Actions
Blocks Redmine - Patch #31126: Update sqlite3 gem (~> 1.4.0)ClosedGo MAEDA

Actions
Actions #1

Updated by Marius BĂLTEANU over 5 years ago

Patch here.

If the tests pass (results here), I think we can deliver this in 4.0.3.

Actions #2

Updated by Go MAEDA over 5 years ago

Actions #3

Updated by Marius BĂLTEANU over 5 years ago

Some tests fail: https://gitlab.com/marius-balteanu/redmine/-/jobs/185750691

................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................F

Failure:
TimelogControllerTest#test_index_csv [/builds/marius-balteanu/redmine/test/functional/timelog_controller_test.rb:1361]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/timelog_controller_test.rb:1357

.............F

Failure:
TimelogControllerTest#test_index_csv_all_projects [/builds/marius-balteanu/redmine/test/functional/timelog_controller_test.rb:1353]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/timelog_controller_test.rb:1349

.............................................................................................................................................................................................................................................................................................F

Failure:
TimelogReportTest#test_report_all_projects_csv_export [/builds/marius-balteanu/redmine/test/functional/timelog_report_test.rb:236]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/timelog_report_test.rb:227

...F

Failure:
TimelogReportTest#test_csv_cannot_convert_should_be_replaced_big_5 [/builds/marius-balteanu/redmine/test/functional/timelog_report_test.rb:337]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/timelog_report_test.rb:308

.F

Failure:
TimelogReportTest#test_report_csv_export [/builds/marius-balteanu/redmine/test/functional/timelog_report_test.rb:254]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/timelog_report_test.rb:244

......F

Failure:
TimelogReportTest#test_csv_big_5 [/builds/marius-balteanu/redmine/test/functional/timelog_report_test.rb:292]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/timelog_report_test.rb:262

......F

Failure:
TimelogReportTest#test_csv_fr [/builds/marius-balteanu/redmine/test/functional/timelog_report_test.rb:372]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/timelog_report_test.rb:347

....................................................................................................................................................................................................................................................................................................F

Failure:
IssuesControllerTest#test_index_csv_cannot_convert_should_be_replaced_big_5 [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:806]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:794

.....................F

Failure:
IssuesControllerTest#test_index_csv_with_spent_time_column [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:700]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:690

......F

Failure:
IssuesControllerTest#test_index_csv [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:642]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:636

...........................................................................F

Failure:
IssuesControllerTest#test_index_csv_with_all_columns [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:712]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:705

.........................................F

Failure:
IssuesControllerTest#test_index_csv_tw [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:830]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:818

....F

Failure:
IssuesControllerTest#test_index_csv_fr [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:848]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:836

...............................................................F

Failure:
IssuesControllerTest#test_index_csv_big_5 [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:784]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:773

.F

Failure:
IssuesControllerTest#test_index_csv_with_description [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:684]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:672

....................................................................F

Failure:
IssuesControllerTest#test_index_with_relations_column [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:1316]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:1284

...F

Failure:
IssuesControllerTest#test_index_csv_with_project [/builds/marius-balteanu/redmine/test/functional/issues_controller_test.rb:656]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/issues_controller_test.rb:649

.....................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................F

Failure:
UsersControllerTest#test_index_csv [/builds/marius-balteanu/redmine/test/functional/users_controller_test.rb:77]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/users_controller_test.rb:69

F

Failure:
UsersControllerTest#test_index_csv_with_name_filter [/builds/marius-balteanu/redmine/test/functional/users_controller_test.rb:99]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/users_controller_test.rb:93

.F

Failure:
UsersControllerTest#test_index_csv_with_status_filter [/builds/marius-balteanu/redmine/test/functional/users_controller_test.rb:89]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/users_controller_test.rb:81

.F

Failure:
UsersControllerTest#test_index_csv_with_group_filter [/builds/marius-balteanu/redmine/test/functional/users_controller_test.rb:107]:
Expected: "text/csv; header=present" 
  Actual: "text/csv" 

bin/rails test test/functional/users_controller_test.rb:102
Actions #4

Updated by Marius BĂLTEANU over 5 years ago

I think the test fails are generated by this change: https://github.com/rails/rails/pull/35549 and we need to update our tests to expect only “text/csv”.

Jean-Philippe, should we deliver this in 4.0.3?

Actions #5

Updated by Marius BĂLTEANU over 5 years ago

  • File 0001-Update-Rails-to-5.2.3.patch added

Here is the patch.

Actions #6

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (0001-Update-Rails-to-5.2.3.patch)
Actions #8

Updated by Marius BĂLTEANU over 5 years ago

  • Target version changed from Candidate for next minor release to 4.0.4
Actions #9

Updated by Go MAEDA over 5 years ago

Actions #10

Updated by Go MAEDA over 5 years ago

  • Category set to Rails support
  • Status changed from New to Resolved
  • Assignee set to Go MAEDA

Committed the patch. Thanks.

Actions #11

Updated by Go MAEDA over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF