@@ -672,8 +672,12 class IssueTest < ActiveSupport::TestCase | |||||
672 |
|
672 | |||
673 | def test_required_attribute_names_for_multiple_roles_should_intersect_rules |
|
673 | def test_required_attribute_names_for_multiple_roles_should_intersect_rules | |
674 | WorkflowPermission.delete_all |
|
674 | WorkflowPermission.delete_all | |
675 |
WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, |
|
675 | WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, | |
676 | WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'start_date', :rule => 'required') |
|
676 | :role_id => 1, :field_name => 'due_date', | |
|
677 | :rule => 'required') | |||
|
678 | WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, | |||
|
679 | :role_id => 1, :field_name => 'start_date', | |||
|
680 | :rule => 'required') | |||
677 | user = User.find(2) |
|
681 | user = User.find(2) | |
678 | member = Member.find(1) |
|
682 | member = Member.find(1) | |
679 | issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1) |
|
683 | issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1) | |
@@ -684,14 +688,18 class IssueTest < ActiveSupport::TestCase | |||||
684 | member.save! |
|
688 | member.save! | |
685 | assert_equal [], issue.required_attribute_names(user.reload) |
|
689 | assert_equal [], issue.required_attribute_names(user.reload) | |
686 |
|
690 | |||
687 |
WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, |
|
691 | WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, | |
|
692 | :role_id => 2, :field_name => 'due_date', | |||
|
693 | :rule => 'required') | |||
688 | assert_equal %w(due_date), issue.required_attribute_names(user) |
|
694 | assert_equal %w(due_date), issue.required_attribute_names(user) | |
689 |
|
695 | |||
690 | member.role_ids = [1, 2, 3] |
|
696 | member.role_ids = [1, 2, 3] | |
691 | member.save! |
|
697 | member.save! | |
692 | assert_equal [], issue.required_attribute_names(user.reload) |
|
698 | assert_equal [], issue.required_attribute_names(user.reload) | |
693 |
|
699 | |||
694 |
WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, |
|
700 | WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, | |
|
701 | :role_id => 2, :field_name => 'due_date', | |||
|
702 | :rule => 'readonly') | |||
695 | # required + readonly => required |
|
703 | # required + readonly => required | |
696 | assert_equal %w(due_date), issue.required_attribute_names(user) |
|
704 | assert_equal %w(due_date), issue.required_attribute_names(user) | |
697 | end |
|
705 | end |
General Comments 0
You need to be logged in to leave comments.
Login now