Chris@0
|
1 # Redmine - project management software
|
Chris@1115
|
2 # Copyright (C) 2006-2012 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 VersionTest < ActiveSupport::TestCase
|
Chris@1115
|
21 fixtures :projects, :users, :issues, :issue_statuses, :trackers, :enumerations, :versions, :projects_trackers
|
Chris@0
|
22
|
Chris@0
|
23 def setup
|
Chris@0
|
24 end
|
Chris@909
|
25
|
Chris@0
|
26 def test_create
|
Chris@0
|
27 v = Version.new(:project => Project.find(1), :name => '1.1', :effective_date => '2011-03-25')
|
Chris@0
|
28 assert v.save
|
Chris@0
|
29 assert_equal 'open', v.status
|
Chris@909
|
30 assert_equal 'none', v.sharing
|
Chris@0
|
31 end
|
Chris@909
|
32
|
Chris@0
|
33 def test_invalid_effective_date_validation
|
Chris@0
|
34 v = Version.new(:project => Project.find(1), :name => '1.1', :effective_date => '99999-01-01')
|
Chris@1115
|
35 assert !v.valid?
|
Chris@1115
|
36 v.effective_date = '2012-11-33'
|
Chris@1115
|
37 assert !v.valid?
|
Chris@1115
|
38 v.effective_date = '2012-31-11'
|
Chris@1115
|
39 assert !v.valid?
|
Chris@1115
|
40 v.effective_date = 'ABC'
|
Chris@1115
|
41 assert !v.valid?
|
Chris@1115
|
42 assert_include I18n.translate('activerecord.errors.messages.not_a_date'),
|
Chris@1115
|
43 v.errors[:effective_date]
|
Chris@0
|
44 end
|
Chris@909
|
45
|
Chris@0
|
46 def test_progress_should_be_0_with_no_assigned_issues
|
Chris@0
|
47 project = Project.find(1)
|
Chris@0
|
48 v = Version.create!(:project => project, :name => 'Progress')
|
Chris@0
|
49 assert_equal 0, v.completed_pourcent
|
Chris@0
|
50 assert_equal 0, v.closed_pourcent
|
Chris@0
|
51 end
|
Chris@909
|
52
|
Chris@0
|
53 def test_progress_should_be_0_with_unbegun_assigned_issues
|
Chris@0
|
54 project = Project.find(1)
|
Chris@0
|
55 v = Version.create!(:project => project, :name => 'Progress')
|
Chris@0
|
56 add_issue(v)
|
Chris@0
|
57 add_issue(v, :done_ratio => 0)
|
Chris@0
|
58 assert_progress_equal 0, v.completed_pourcent
|
Chris@0
|
59 assert_progress_equal 0, v.closed_pourcent
|
Chris@0
|
60 end
|
Chris@909
|
61
|
Chris@0
|
62 def test_progress_should_be_100_with_closed_assigned_issues
|
Chris@0
|
63 project = Project.find(1)
|
Chris@0
|
64 status = IssueStatus.find(:first, :conditions => {:is_closed => true})
|
Chris@0
|
65 v = Version.create!(:project => project, :name => 'Progress')
|
Chris@0
|
66 add_issue(v, :status => status)
|
Chris@0
|
67 add_issue(v, :status => status, :done_ratio => 20)
|
Chris@0
|
68 add_issue(v, :status => status, :done_ratio => 70, :estimated_hours => 25)
|
Chris@0
|
69 add_issue(v, :status => status, :estimated_hours => 15)
|
Chris@0
|
70 assert_progress_equal 100.0, v.completed_pourcent
|
Chris@0
|
71 assert_progress_equal 100.0, v.closed_pourcent
|
Chris@0
|
72 end
|
Chris@909
|
73
|
Chris@0
|
74 def test_progress_should_consider_done_ratio_of_open_assigned_issues
|
Chris@0
|
75 project = Project.find(1)
|
Chris@0
|
76 v = Version.create!(:project => project, :name => 'Progress')
|
Chris@0
|
77 add_issue(v)
|
Chris@0
|
78 add_issue(v, :done_ratio => 20)
|
Chris@0
|
79 add_issue(v, :done_ratio => 70)
|
Chris@0
|
80 assert_progress_equal (0.0 + 20.0 + 70.0)/3, v.completed_pourcent
|
Chris@0
|
81 assert_progress_equal 0, v.closed_pourcent
|
Chris@0
|
82 end
|
Chris@909
|
83
|
Chris@0
|
84 def test_progress_should_consider_closed_issues_as_completed
|
Chris@0
|
85 project = Project.find(1)
|
Chris@0
|
86 v = Version.create!(:project => project, :name => 'Progress')
|
Chris@0
|
87 add_issue(v)
|
Chris@0
|
88 add_issue(v, :done_ratio => 20)
|
Chris@0
|
89 add_issue(v, :status => IssueStatus.find(:first, :conditions => {:is_closed => true}))
|
Chris@0
|
90 assert_progress_equal (0.0 + 20.0 + 100.0)/3, v.completed_pourcent
|
Chris@0
|
91 assert_progress_equal (100.0)/3, v.closed_pourcent
|
Chris@0
|
92 end
|
Chris@909
|
93
|
Chris@0
|
94 def test_progress_should_consider_estimated_hours_to_weigth_issues
|
Chris@0
|
95 project = Project.find(1)
|
Chris@0
|
96 v = Version.create!(:project => project, :name => 'Progress')
|
Chris@0
|
97 add_issue(v, :estimated_hours => 10)
|
Chris@0
|
98 add_issue(v, :estimated_hours => 20, :done_ratio => 30)
|
Chris@0
|
99 add_issue(v, :estimated_hours => 40, :done_ratio => 10)
|
Chris@0
|
100 add_issue(v, :estimated_hours => 25, :status => IssueStatus.find(:first, :conditions => {:is_closed => true}))
|
Chris@0
|
101 assert_progress_equal (10.0*0 + 20.0*0.3 + 40*0.1 + 25.0*1)/95.0*100, v.completed_pourcent
|
Chris@0
|
102 assert_progress_equal 25.0/95.0*100, v.closed_pourcent
|
Chris@0
|
103 end
|
Chris@909
|
104
|
Chris@0
|
105 def test_progress_should_consider_average_estimated_hours_to_weigth_unestimated_issues
|
Chris@0
|
106 project = Project.find(1)
|
Chris@0
|
107 v = Version.create!(:project => project, :name => 'Progress')
|
Chris@0
|
108 add_issue(v, :done_ratio => 20)
|
Chris@0
|
109 add_issue(v, :status => IssueStatus.find(:first, :conditions => {:is_closed => true}))
|
Chris@0
|
110 add_issue(v, :estimated_hours => 10, :done_ratio => 30)
|
Chris@0
|
111 add_issue(v, :estimated_hours => 40, :done_ratio => 10)
|
Chris@0
|
112 assert_progress_equal (25.0*0.2 + 25.0*1 + 10.0*0.3 + 40.0*0.1)/100.0*100, v.completed_pourcent
|
Chris@0
|
113 assert_progress_equal 25.0/100.0*100, v.closed_pourcent
|
Chris@0
|
114 end
|
chris@22
|
115
|
Chris@1115
|
116 def test_should_sort_scheduled_then_unscheduled_versions
|
Chris@1115
|
117 Version.delete_all
|
Chris@1115
|
118 v4 = Version.create!(:project_id => 1, :name => 'v4')
|
Chris@1115
|
119 v3 = Version.create!(:project_id => 1, :name => 'v2', :effective_date => '2012-07-14')
|
Chris@1115
|
120 v2 = Version.create!(:project_id => 1, :name => 'v1')
|
Chris@1115
|
121 v1 = Version.create!(:project_id => 1, :name => 'v3', :effective_date => '2012-08-02')
|
Chris@1115
|
122 v5 = Version.create!(:project_id => 1, :name => 'v5', :effective_date => '2012-07-02')
|
Chris@1115
|
123
|
Chris@1115
|
124 assert_equal [v5, v3, v1, v2, v4], [v1, v2, v3, v4, v5].sort
|
Chris@1115
|
125 assert_equal [v5, v3, v1, v2, v4], Version.sorted.all
|
Chris@1115
|
126 end
|
Chris@1115
|
127
|
Chris@1115
|
128 def test_completed_should_be_false_when_due_today
|
Chris@1115
|
129 version = Version.create!(:project_id => 1, :effective_date => Date.today, :name => 'Due today')
|
Chris@1115
|
130 assert_equal false, version.completed?
|
Chris@1115
|
131 end
|
Chris@1115
|
132
|
chris@22
|
133 context "#behind_schedule?" do
|
chris@22
|
134 setup do
|
chris@22
|
135 ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests
|
Chris@1115
|
136 @project = Project.create!(:name => 'test0', :identifier => 'test0')
|
Chris@1115
|
137 @project.trackers << Tracker.create!(:name => 'track')
|
chris@22
|
138
|
Chris@1115
|
139 @version = Version.create!(:project => @project, :effective_date => nil, :name => 'version')
|
chris@22
|
140 end
|
Chris@909
|
141
|
chris@22
|
142 should "be false if there are no issues assigned" do
|
chris@22
|
143 @version.update_attribute(:effective_date, Date.yesterday)
|
chris@22
|
144 assert_equal false, @version.behind_schedule?
|
chris@22
|
145 end
|
chris@22
|
146
|
chris@22
|
147 should "be false if there is no effective_date" do
|
chris@22
|
148 assert_equal false, @version.behind_schedule?
|
chris@22
|
149 end
|
chris@22
|
150
|
chris@22
|
151 should "be false if all of the issues are ahead of schedule" do
|
chris@22
|
152 @version.update_attribute(:effective_date, 7.days.from_now.to_date)
|
Chris@1115
|
153 add_issue(@version, :start_date => 7.days.ago, :done_ratio => 60) # 14 day span, 60% done, 50% time left
|
Chris@1115
|
154 add_issue(@version, :start_date => 7.days.ago, :done_ratio => 60) # 14 day span, 60% done, 50% time left
|
chris@22
|
155 assert_equal 60, @version.completed_pourcent
|
chris@22
|
156 assert_equal false, @version.behind_schedule?
|
chris@22
|
157 end
|
chris@22
|
158
|
chris@22
|
159 should "be true if any of the issues are behind schedule" do
|
chris@22
|
160 @version.update_attribute(:effective_date, 7.days.from_now.to_date)
|
Chris@1115
|
161 add_issue(@version, :start_date => 7.days.ago, :done_ratio => 60) # 14 day span, 60% done, 50% time left
|
Chris@1115
|
162 add_issue(@version, :start_date => 7.days.ago, :done_ratio => 20) # 14 day span, 20% done, 50% time left
|
chris@22
|
163 assert_equal 40, @version.completed_pourcent
|
chris@22
|
164 assert_equal true, @version.behind_schedule?
|
chris@22
|
165 end
|
chris@22
|
166
|
chris@22
|
167 should "be false if all of the issues are complete" do
|
chris@22
|
168 @version.update_attribute(:effective_date, 7.days.from_now.to_date)
|
Chris@1115
|
169 add_issue(@version, :start_date => 14.days.ago, :done_ratio => 100, :status => IssueStatus.find(5)) # 7 day span
|
Chris@1115
|
170 add_issue(@version, :start_date => 14.days.ago, :done_ratio => 100, :status => IssueStatus.find(5)) # 7 day span
|
chris@22
|
171 assert_equal 100, @version.completed_pourcent
|
chris@22
|
172 assert_equal false, @version.behind_schedule?
|
chris@22
|
173 end
|
chris@22
|
174 end
|
chris@22
|
175
|
Chris@0
|
176 context "#estimated_hours" do
|
Chris@0
|
177 setup do
|
Chris@0
|
178 @version = Version.create!(:project_id => 1, :name => '#estimated_hours')
|
Chris@0
|
179 end
|
Chris@909
|
180
|
Chris@0
|
181 should "return 0 with no assigned issues" do
|
Chris@0
|
182 assert_equal 0, @version.estimated_hours
|
Chris@0
|
183 end
|
Chris@909
|
184
|
Chris@0
|
185 should "return 0 with no estimated hours" do
|
Chris@0
|
186 add_issue(@version)
|
Chris@0
|
187 assert_equal 0, @version.estimated_hours
|
Chris@0
|
188 end
|
Chris@909
|
189
|
Chris@0
|
190 should "return the sum of estimated hours" do
|
Chris@0
|
191 add_issue(@version, :estimated_hours => 2.5)
|
Chris@0
|
192 add_issue(@version, :estimated_hours => 5)
|
Chris@0
|
193 assert_equal 7.5, @version.estimated_hours
|
Chris@0
|
194 end
|
Chris@909
|
195
|
Chris@0
|
196 should "return the sum of leaves estimated hours" do
|
Chris@0
|
197 parent = add_issue(@version)
|
Chris@0
|
198 add_issue(@version, :estimated_hours => 2.5, :parent_issue_id => parent.id)
|
Chris@0
|
199 add_issue(@version, :estimated_hours => 5, :parent_issue_id => parent.id)
|
Chris@0
|
200 assert_equal 7.5, @version.estimated_hours
|
Chris@0
|
201 end
|
Chris@0
|
202 end
|
Chris@0
|
203
|
Chris@0
|
204 test "should update all issue's fixed_version associations in case the hierarchy changed XXX" do
|
Chris@0
|
205 User.current = User.find(1) # Need the admin's permissions
|
Chris@909
|
206
|
Chris@0
|
207 @version = Version.find(7)
|
Chris@0
|
208 # Separate hierarchy
|
Chris@0
|
209 project_1_issue = Issue.find(1)
|
Chris@0
|
210 project_1_issue.fixed_version = @version
|
Chris@119
|
211 assert project_1_issue.save, project_1_issue.errors.full_messages.to_s
|
Chris@909
|
212
|
Chris@0
|
213 project_5_issue = Issue.find(6)
|
Chris@0
|
214 project_5_issue.fixed_version = @version
|
Chris@0
|
215 assert project_5_issue.save
|
Chris@909
|
216
|
Chris@0
|
217 # Project
|
Chris@0
|
218 project_2_issue = Issue.find(4)
|
Chris@0
|
219 project_2_issue.fixed_version = @version
|
Chris@0
|
220 assert project_2_issue.save
|
Chris@0
|
221
|
Chris@0
|
222 # Update the sharing
|
Chris@0
|
223 @version.sharing = 'none'
|
Chris@0
|
224 assert @version.save
|
Chris@0
|
225
|
Chris@0
|
226 # Project 1 now out of the shared scope
|
Chris@0
|
227 project_1_issue.reload
|
Chris@0
|
228 assert_equal nil, project_1_issue.fixed_version, "Fixed version is still set after changing the Version's sharing"
|
Chris@909
|
229
|
Chris@0
|
230 # Project 5 now out of the shared scope
|
Chris@0
|
231 project_5_issue.reload
|
Chris@0
|
232 assert_equal nil, project_5_issue.fixed_version, "Fixed version is still set after changing the Version's sharing"
|
Chris@0
|
233
|
Chris@0
|
234 # Project 2 issue remains
|
Chris@0
|
235 project_2_issue.reload
|
Chris@0
|
236 assert_equal @version, project_2_issue.fixed_version
|
Chris@0
|
237 end
|
Chris@909
|
238
|
Chris@0
|
239 private
|
Chris@909
|
240
|
Chris@0
|
241 def add_issue(version, attributes={})
|
Chris@0
|
242 Issue.create!({:project => version.project,
|
Chris@0
|
243 :fixed_version => version,
|
Chris@0
|
244 :subject => 'Test',
|
Chris@0
|
245 :author => User.find(:first),
|
Chris@0
|
246 :tracker => version.project.trackers.find(:first)}.merge(attributes))
|
Chris@0
|
247 end
|
Chris@909
|
248
|
Chris@0
|
249 def assert_progress_equal(expected_float, actual_float, message="")
|
Chris@0
|
250 assert_in_delta(expected_float, actual_float, 0.000001, message="")
|
Chris@0
|
251 end
|
Chris@0
|
252 end
|