@@ -99,18 +99,36 class IssueRelationTest < ActiveSupport::TestCase | |||||
99 |
|
99 | |||
100 | def test_validates_circular_dependency |
|
100 | def test_validates_circular_dependency | |
101 | IssueRelation.delete_all |
|
101 | IssueRelation.delete_all | |
102 | assert IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => IssueRelation::TYPE_PRECEDES) |
|
102 | assert IssueRelation.create!( | |
103 | assert IssueRelation.create!(:issue_from => Issue.find(2), :issue_to => Issue.find(3), :relation_type => IssueRelation::TYPE_PRECEDES) |
|
103 | :issue_from => Issue.find(1), :issue_to => Issue.find(2), | |
104 |
|
|
104 | :relation_type => IssueRelation::TYPE_PRECEDES | |
|
105 | ) | |||
|
106 | assert IssueRelation.create!( | |||
|
107 | :issue_from => Issue.find(2), :issue_to => Issue.find(3), | |||
|
108 | :relation_type => IssueRelation::TYPE_PRECEDES | |||
|
109 | ) | |||
|
110 | r = IssueRelation.new( | |||
|
111 | :issue_from => Issue.find(3), :issue_to => Issue.find(1), | |||
|
112 | :relation_type => IssueRelation::TYPE_PRECEDES | |||
|
113 | ) | |||
105 | assert !r.save |
|
114 | assert !r.save | |
106 | assert_not_nil r.errors[:base] |
|
115 | assert_not_nil r.errors[:base] | |
107 | end |
|
116 | end | |
108 |
|
117 | |||
109 | def test_validates_circular_dependency_on_reverse_relations |
|
118 | def test_validates_circular_dependency_on_reverse_relations | |
110 | IssueRelation.delete_all |
|
119 | IssueRelation.delete_all | |
111 | assert IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(3), :relation_type => IssueRelation::TYPE_BLOCKS) |
|
120 | assert IssueRelation.create!( | |
112 | assert IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => IssueRelation::TYPE_BLOCKED) |
|
121 | :issue_from => Issue.find(1), :issue_to => Issue.find(3), | |
113 | r = IssueRelation.new(:issue_from => Issue.find(2), :issue_to => Issue.find(1), :relation_type => IssueRelation::TYPE_BLOCKED) |
|
122 | :relation_type => IssueRelation::TYPE_BLOCKS | |
|
123 | ) | |||
|
124 | assert IssueRelation.create!( | |||
|
125 | :issue_from => Issue.find(1), :issue_to => Issue.find(2), | |||
|
126 | :relation_type => IssueRelation::TYPE_BLOCKED | |||
|
127 | ) | |||
|
128 | r = IssueRelation.new( | |||
|
129 | :issue_from => Issue.find(2), :issue_to => Issue.find(1), | |||
|
130 | :relation_type => IssueRelation::TYPE_BLOCKED | |||
|
131 | ) | |||
114 | assert !r.save |
|
132 | assert !r.save | |
115 | assert_not_nil r.errors[:base] |
|
133 | assert_not_nil r.errors[:base] | |
116 | end |
|
134 | end |
General Comments 0
You need to be logged in to leave comments.
Login now