diff options
| author | Toshi MARUYAMA <marutosijp2@yahoo.co.jp> | 2012-10-13 11:17:43 +0000 |
|---|---|---|
| committer | Toshi MARUYAMA <marutosijp2@yahoo.co.jp> | 2012-10-13 11:17:43 +0000 |
| commit | 8487a3e0e63f4b7bced21a16827f7e6e6f4b9430 (patch) | |
| tree | 86e6225c8ec79dfda878cc5704853df04afd9928 /test/unit | |
| parent | a1b82c7650b58a35964c8f16c8fc7d92192f3eb2 (diff) | |
| download | redmine-8487a3e0e63f4b7bced21a16827f7e6e6f4b9430.tar.gz redmine-8487a3e0e63f4b7bced21a16827f7e6e6f4b9430.zip | |
code layout clean up test_required_attribute_names_for_multiple_roles_should_intersect_rules of unit issue test
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10648 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'test/unit')
| -rw-r--r-- | test/unit/issue_test.rb | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/test/unit/issue_test.rb b/test/unit/issue_test.rb index 9e8c2044a..84a33a0c7 100644 --- a/test/unit/issue_test.rb +++ b/test/unit/issue_test.rb @@ -672,8 +672,12 @@ class IssueTest < ActiveSupport::TestCase def test_required_attribute_names_for_multiple_roles_should_intersect_rules WorkflowPermission.delete_all - WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'due_date', :rule => 'required') - WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'start_date', :rule => 'required') + WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, + :role_id => 1, :field_name => 'due_date', + :rule => 'required') + WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, + :role_id => 1, :field_name => 'start_date', + :rule => 'required') user = User.find(2) member = Member.find(1) issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1) @@ -684,14 +688,18 @@ class IssueTest < ActiveSupport::TestCase member.save! assert_equal [], issue.required_attribute_names(user.reload) - WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 2, :field_name => 'due_date', :rule => 'required') + WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, + :role_id => 2, :field_name => 'due_date', + :rule => 'required') assert_equal %w(due_date), issue.required_attribute_names(user) member.role_ids = [1, 2, 3] member.save! assert_equal [], issue.required_attribute_names(user.reload) - WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 2, :field_name => 'due_date', :rule => 'readonly') + WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, + :role_id => 2, :field_name => 'due_date', + :rule => 'readonly') # required + readonly => required assert_equal %w(due_date), issue.required_attribute_names(user) end |