Project

General

Profile

Actions

Patch #31865

closed

Add missing fixtures to ImportsControllerTest

Added by Yuichi HARADA over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

ImportsControllerTest#test_post_run_with_notifications has failed.

$ RAILS_ENV=test bundle exec rake db:migrate:reset

$ RAILS_ENV=test bundle exec rake test TEST=test/functional/imports_controller_test.rb
Run options: --seed 25105

# Running:

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

Failure:
ImportsControllerTest#test_post_run_with_notifications [test/functional/imports_controller_test.rb:275]:
Expected: 6
  Actual: 3

bin/rails test test/functional/imports_controller_test.rb:240

..

Finished in 2.226044s, 6.7384 runs/s, 30.5475 assertions/s.
15 runs, 68 assertions, 1 failures, 0 errors, 0 skips

I think that because the fixtures is not enough.

diff --git a/test/functional/imports_controller_test.rb b/test/functional/imports_controller_test.rb
index 871a08346..ed9024306 100644
--- a/test/functional/imports_controller_test.rb
+++ b/test/functional/imports_controller_test.rb
@@ -21,7 +21,7 @@ require File.expand_path('../../test_helper', __FILE__)

 class ImportsControllerTest < Redmine::ControllerTest
   fixtures :projects, :enabled_modules,
-           :users, :email_addresses,
+           :users, :email_addresses, :user_preferences,
            :roles, :members, :member_roles,
            :issues, :issue_statuses,
            :trackers, :projects_trackers,
Actions #1

Updated by Go MAEDA over 5 years ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA
  • Target version set to 4.1.0

Committed the fix. Thanks.

Actions

Also available in: Atom PDF