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