Chris@0
|
1 # Redmine - project management software
|
Chris@1494
|
2 # Copyright (C) 2006-2014 Jean-Philippe Lang
|
Chris@0
|
3 #
|
Chris@0
|
4 # This program is free software; you can redistribute it and/or
|
Chris@0
|
5 # modify it under the terms of the GNU General Public License
|
Chris@0
|
6 # as published by the Free Software Foundation; either version 2
|
Chris@0
|
7 # of the License, or (at your option) any later version.
|
Chris@909
|
8 #
|
Chris@0
|
9 # This program is distributed in the hope that it will be useful,
|
Chris@0
|
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
|
Chris@0
|
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
Chris@0
|
12 # GNU General Public License for more details.
|
Chris@909
|
13 #
|
Chris@0
|
14 # You should have received a copy of the GNU General Public License
|
Chris@0
|
15 # along with this program; if not, write to the Free Software
|
Chris@0
|
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
Chris@0
|
17
|
Chris@119
|
18 require File.expand_path('../../test_helper', __FILE__)
|
Chris@0
|
19
|
Chris@0
|
20 class IssueRelationTest < ActiveSupport::TestCase
|
Chris@1115
|
21 fixtures :projects,
|
Chris@1115
|
22 :users,
|
Chris@1115
|
23 :roles,
|
Chris@1115
|
24 :members,
|
Chris@1115
|
25 :member_roles,
|
Chris@1115
|
26 :issues,
|
Chris@1115
|
27 :issue_statuses,
|
Chris@1115
|
28 :issue_relations,
|
Chris@1115
|
29 :enabled_modules,
|
Chris@1115
|
30 :enumerations,
|
Chris@1464
|
31 :trackers,
|
Chris@1464
|
32 :projects_trackers
|
Chris@1464
|
33
|
Chris@1464
|
34 include Redmine::I18n
|
Chris@0
|
35
|
Chris@0
|
36 def test_create
|
Chris@0
|
37 from = Issue.find(1)
|
Chris@0
|
38 to = Issue.find(2)
|
Chris@909
|
39
|
Chris@909
|
40 relation = IssueRelation.new :issue_from => from, :issue_to => to,
|
Chris@909
|
41 :relation_type => IssueRelation::TYPE_PRECEDES
|
Chris@0
|
42 assert relation.save
|
Chris@0
|
43 relation.reload
|
Chris@0
|
44 assert_equal IssueRelation::TYPE_PRECEDES, relation.relation_type
|
Chris@0
|
45 assert_equal from, relation.issue_from
|
Chris@0
|
46 assert_equal to, relation.issue_to
|
Chris@0
|
47 end
|
Chris@909
|
48
|
Chris@909
|
49 def test_create_minimum
|
Chris@909
|
50 relation = IssueRelation.new :issue_from => Issue.find(1), :issue_to => Issue.find(2)
|
Chris@909
|
51 assert relation.save
|
Chris@909
|
52 assert_equal IssueRelation::TYPE_RELATES, relation.relation_type
|
Chris@909
|
53 end
|
Chris@909
|
54
|
Chris@0
|
55 def test_follows_relation_should_be_reversed
|
Chris@0
|
56 from = Issue.find(1)
|
Chris@0
|
57 to = Issue.find(2)
|
Chris@909
|
58
|
Chris@909
|
59 relation = IssueRelation.new :issue_from => from, :issue_to => to,
|
Chris@909
|
60 :relation_type => IssueRelation::TYPE_FOLLOWS
|
Chris@0
|
61 assert relation.save
|
Chris@0
|
62 relation.reload
|
Chris@0
|
63 assert_equal IssueRelation::TYPE_PRECEDES, relation.relation_type
|
Chris@0
|
64 assert_equal to, relation.issue_from
|
Chris@0
|
65 assert_equal from, relation.issue_to
|
Chris@0
|
66 end
|
Chris@909
|
67
|
Chris@0
|
68 def test_follows_relation_should_not_be_reversed_if_validation_fails
|
Chris@0
|
69 from = Issue.find(1)
|
Chris@0
|
70 to = Issue.find(2)
|
Chris@909
|
71
|
Chris@909
|
72 relation = IssueRelation.new :issue_from => from, :issue_to => to,
|
Chris@909
|
73 :relation_type => IssueRelation::TYPE_FOLLOWS,
|
Chris@909
|
74 :delay => 'xx'
|
Chris@0
|
75 assert !relation.save
|
Chris@0
|
76 assert_equal IssueRelation::TYPE_FOLLOWS, relation.relation_type
|
Chris@0
|
77 assert_equal from, relation.issue_from
|
Chris@0
|
78 assert_equal to, relation.issue_to
|
Chris@0
|
79 end
|
Chris@909
|
80
|
Chris@0
|
81 def test_relation_type_for
|
Chris@0
|
82 from = Issue.find(1)
|
Chris@0
|
83 to = Issue.find(2)
|
Chris@909
|
84
|
Chris@909
|
85 relation = IssueRelation.new :issue_from => from, :issue_to => to,
|
Chris@909
|
86 :relation_type => IssueRelation::TYPE_PRECEDES
|
Chris@0
|
87 assert_equal IssueRelation::TYPE_PRECEDES, relation.relation_type_for(from)
|
Chris@0
|
88 assert_equal IssueRelation::TYPE_FOLLOWS, relation.relation_type_for(to)
|
Chris@0
|
89 end
|
Chris@909
|
90
|
Chris@119
|
91 def test_set_issue_to_dates_without_issue_to
|
Chris@909
|
92 r = IssueRelation.new(:issue_from => Issue.new(:start_date => Date.today),
|
Chris@909
|
93 :relation_type => IssueRelation::TYPE_PRECEDES,
|
Chris@909
|
94 :delay => 1)
|
Chris@119
|
95 assert_nil r.set_issue_to_dates
|
Chris@119
|
96 end
|
Chris@909
|
97
|
Chris@119
|
98 def test_set_issue_to_dates_without_issues
|
Chris@119
|
99 r = IssueRelation.new(:relation_type => IssueRelation::TYPE_PRECEDES, :delay => 1)
|
Chris@119
|
100 assert_nil r.set_issue_to_dates
|
Chris@119
|
101 end
|
Chris@909
|
102
|
Chris@128
|
103 def test_validates_circular_dependency
|
Chris@128
|
104 IssueRelation.delete_all
|
Chris@1115
|
105 assert IssueRelation.create!(
|
Chris@1115
|
106 :issue_from => Issue.find(1), :issue_to => Issue.find(2),
|
Chris@1115
|
107 :relation_type => IssueRelation::TYPE_PRECEDES
|
Chris@1115
|
108 )
|
Chris@1115
|
109 assert IssueRelation.create!(
|
Chris@1115
|
110 :issue_from => Issue.find(2), :issue_to => Issue.find(3),
|
Chris@1115
|
111 :relation_type => IssueRelation::TYPE_PRECEDES
|
Chris@1115
|
112 )
|
Chris@1115
|
113 r = IssueRelation.new(
|
Chris@1115
|
114 :issue_from => Issue.find(3), :issue_to => Issue.find(1),
|
Chris@1115
|
115 :relation_type => IssueRelation::TYPE_PRECEDES
|
Chris@1115
|
116 )
|
Chris@128
|
117 assert !r.save
|
Chris@1464
|
118 assert_not_equal [], r.errors[:base]
|
Chris@1464
|
119 end
|
Chris@1464
|
120
|
Chris@1464
|
121 def test_validates_circular_dependency_of_subtask
|
Chris@1464
|
122 set_language_if_valid 'en'
|
Chris@1464
|
123 issue1 = Issue.generate!
|
Chris@1464
|
124 issue2 = Issue.generate!
|
Chris@1464
|
125 IssueRelation.create!(
|
Chris@1464
|
126 :issue_from => issue1, :issue_to => issue2,
|
Chris@1464
|
127 :relation_type => IssueRelation::TYPE_PRECEDES
|
Chris@1464
|
128 )
|
Chris@1464
|
129 child = Issue.generate!(:parent_issue_id => issue2.id)
|
Chris@1464
|
130 issue1.reload
|
Chris@1464
|
131 child.reload
|
Chris@1464
|
132
|
Chris@1464
|
133 r = IssueRelation.new(
|
Chris@1464
|
134 :issue_from => child, :issue_to => issue1,
|
Chris@1464
|
135 :relation_type => IssueRelation::TYPE_PRECEDES
|
Chris@1464
|
136 )
|
Chris@1464
|
137 assert !r.save
|
Chris@1464
|
138 assert_include 'This relation would create a circular dependency', r.errors.full_messages
|
Chris@1464
|
139 end
|
Chris@1464
|
140
|
Chris@1464
|
141 def test_subtasks_should_allow_precedes_relation
|
Chris@1464
|
142 parent = Issue.generate!
|
Chris@1464
|
143 child1 = Issue.generate!(:parent_issue_id => parent.id)
|
Chris@1464
|
144 child2 = Issue.generate!(:parent_issue_id => parent.id)
|
Chris@1464
|
145
|
Chris@1464
|
146 r = IssueRelation.new(
|
Chris@1464
|
147 :issue_from => child1, :issue_to => child2,
|
Chris@1464
|
148 :relation_type => IssueRelation::TYPE_PRECEDES
|
Chris@1464
|
149 )
|
Chris@1464
|
150 assert r.valid?
|
Chris@1464
|
151 assert r.save
|
Chris@128
|
152 end
|
Chris@507
|
153
|
Chris@507
|
154 def test_validates_circular_dependency_on_reverse_relations
|
Chris@507
|
155 IssueRelation.delete_all
|
Chris@1115
|
156 assert IssueRelation.create!(
|
Chris@1115
|
157 :issue_from => Issue.find(1), :issue_to => Issue.find(3),
|
Chris@1115
|
158 :relation_type => IssueRelation::TYPE_BLOCKS
|
Chris@1115
|
159 )
|
Chris@1115
|
160 assert IssueRelation.create!(
|
Chris@1115
|
161 :issue_from => Issue.find(1), :issue_to => Issue.find(2),
|
Chris@1115
|
162 :relation_type => IssueRelation::TYPE_BLOCKED
|
Chris@1115
|
163 )
|
Chris@1115
|
164 r = IssueRelation.new(
|
Chris@1115
|
165 :issue_from => Issue.find(2), :issue_to => Issue.find(1),
|
Chris@1115
|
166 :relation_type => IssueRelation::TYPE_BLOCKED
|
Chris@1115
|
167 )
|
Chris@507
|
168 assert !r.save
|
Chris@1464
|
169 assert_not_equal [], r.errors[:base]
|
Chris@1464
|
170 end
|
Chris@1464
|
171
|
Chris@1464
|
172 def test_create_should_make_journal_entry
|
Chris@1464
|
173 from = Issue.find(1)
|
Chris@1464
|
174 to = Issue.find(2)
|
Chris@1464
|
175 from_journals = from.journals.size
|
Chris@1464
|
176 to_journals = to.journals.size
|
Chris@1464
|
177 relation = IssueRelation.new(:issue_from => from, :issue_to => to,
|
Chris@1464
|
178 :relation_type => IssueRelation::TYPE_PRECEDES)
|
Chris@1464
|
179 assert relation.save
|
Chris@1464
|
180 from.reload
|
Chris@1464
|
181 to.reload
|
Chris@1464
|
182 relation.reload
|
Chris@1464
|
183 assert_equal from.journals.size, (from_journals + 1)
|
Chris@1464
|
184 assert_equal to.journals.size, (to_journals + 1)
|
Chris@1464
|
185 assert_equal 'relation', from.journals.last.details.last.property
|
Chris@1517
|
186 assert_equal 'precedes', from.journals.last.details.last.prop_key
|
Chris@1464
|
187 assert_equal '2', from.journals.last.details.last.value
|
Chris@1464
|
188 assert_nil from.journals.last.details.last.old_value
|
Chris@1464
|
189 assert_equal 'relation', to.journals.last.details.last.property
|
Chris@1517
|
190 assert_equal 'follows', to.journals.last.details.last.prop_key
|
Chris@1464
|
191 assert_equal '1', to.journals.last.details.last.value
|
Chris@1464
|
192 assert_nil to.journals.last.details.last.old_value
|
Chris@1464
|
193 end
|
Chris@1464
|
194
|
Chris@1464
|
195 def test_delete_should_make_journal_entry
|
Chris@1464
|
196 relation = IssueRelation.find(1)
|
Chris@1464
|
197 from = relation.issue_from
|
Chris@1464
|
198 to = relation.issue_to
|
Chris@1464
|
199 from_journals = from.journals.size
|
Chris@1464
|
200 to_journals = to.journals.size
|
Chris@1464
|
201 assert relation.destroy
|
Chris@1464
|
202 from.reload
|
Chris@1464
|
203 to.reload
|
Chris@1464
|
204 assert_equal from.journals.size, (from_journals + 1)
|
Chris@1464
|
205 assert_equal to.journals.size, (to_journals + 1)
|
Chris@1464
|
206 assert_equal 'relation', from.journals.last.details.last.property
|
Chris@1517
|
207 assert_equal 'blocks', from.journals.last.details.last.prop_key
|
Chris@1464
|
208 assert_equal '9', from.journals.last.details.last.old_value
|
Chris@1464
|
209 assert_nil from.journals.last.details.last.value
|
Chris@1464
|
210 assert_equal 'relation', to.journals.last.details.last.property
|
Chris@1517
|
211 assert_equal 'blocked', to.journals.last.details.last.prop_key
|
Chris@1464
|
212 assert_equal '10', to.journals.last.details.last.old_value
|
Chris@1464
|
213 assert_nil to.journals.last.details.last.value
|
Chris@507
|
214 end
|
Chris@0
|
215 end
|