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