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 WorkflowsControllerTest < ActionController::TestCase
|
Chris@0
|
21 fixtures :roles, :trackers, :workflows, :users, :issue_statuses
|
Chris@909
|
22
|
Chris@0
|
23 def setup
|
Chris@0
|
24 User.current = nil
|
Chris@0
|
25 @request.session[:user_id] = 1 # admin
|
Chris@0
|
26 end
|
Chris@909
|
27
|
Chris@0
|
28 def test_index
|
Chris@0
|
29 get :index
|
Chris@0
|
30 assert_response :success
|
Chris@0
|
31 assert_template 'index'
|
Chris@909
|
32
|
Chris@1464
|
33 count = WorkflowTransition.where(:role_id => 1, :tracker_id => 2).count
|
Chris@0
|
34 assert_tag :tag => 'a', :content => count.to_s,
|
Chris@0
|
35 :attributes => { :href => '/workflows/edit?role_id=1&tracker_id=2' }
|
Chris@0
|
36 end
|
Chris@909
|
37
|
Chris@0
|
38 def test_get_edit
|
Chris@0
|
39 get :edit
|
Chris@0
|
40 assert_response :success
|
Chris@0
|
41 assert_template 'edit'
|
Chris@0
|
42 assert_not_nil assigns(:roles)
|
Chris@0
|
43 assert_not_nil assigns(:trackers)
|
Chris@0
|
44 end
|
Chris@909
|
45
|
Chris@0
|
46 def test_get_edit_with_role_and_tracker
|
Chris@1115
|
47 WorkflowTransition.delete_all
|
Chris@1115
|
48 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 2, :new_status_id => 3)
|
Chris@1115
|
49 WorkflowTransition.create!(:role_id => 2, :tracker_id => 1, :old_status_id => 3, :new_status_id => 5)
|
Chris@909
|
50
|
Chris@0
|
51 get :edit, :role_id => 2, :tracker_id => 1
|
Chris@0
|
52 assert_response :success
|
Chris@0
|
53 assert_template 'edit'
|
Chris@909
|
54
|
Chris@0
|
55 # used status only
|
Chris@0
|
56 assert_not_nil assigns(:statuses)
|
Chris@0
|
57 assert_equal [2, 3, 5], assigns(:statuses).collect(&:id)
|
Chris@909
|
58
|
Chris@0
|
59 # allowed transitions
|
Chris@0
|
60 assert_tag :tag => 'input', :attributes => { :type => 'checkbox',
|
Chris@245
|
61 :name => 'issue_status[3][5][]',
|
Chris@245
|
62 :value => 'always',
|
Chris@0
|
63 :checked => 'checked' }
|
Chris@0
|
64 # not allowed
|
Chris@0
|
65 assert_tag :tag => 'input', :attributes => { :type => 'checkbox',
|
Chris@245
|
66 :name => 'issue_status[3][2][]',
|
Chris@245
|
67 :value => 'always',
|
Chris@0
|
68 :checked => nil }
|
Chris@0
|
69 # unused
|
Chris@0
|
70 assert_no_tag :tag => 'input', :attributes => { :type => 'checkbox',
|
Chris@245
|
71 :name => 'issue_status[1][1][]' }
|
Chris@0
|
72 end
|
Chris@909
|
73
|
Chris@0
|
74 def test_get_edit_with_role_and_tracker_and_all_statuses
|
Chris@1115
|
75 WorkflowTransition.delete_all
|
Chris@909
|
76
|
Chris@0
|
77 get :edit, :role_id => 2, :tracker_id => 1, :used_statuses_only => '0'
|
Chris@0
|
78 assert_response :success
|
Chris@0
|
79 assert_template 'edit'
|
Chris@909
|
80
|
Chris@0
|
81 assert_not_nil assigns(:statuses)
|
Chris@0
|
82 assert_equal IssueStatus.count, assigns(:statuses).size
|
Chris@909
|
83
|
Chris@0
|
84 assert_tag :tag => 'input', :attributes => { :type => 'checkbox',
|
Chris@245
|
85 :name => 'issue_status[1][1][]',
|
Chris@245
|
86 :value => 'always',
|
Chris@0
|
87 :checked => nil }
|
Chris@0
|
88 end
|
Chris@909
|
89
|
Chris@0
|
90 def test_post_edit
|
Chris@245
|
91 post :edit, :role_id => 2, :tracker_id => 1,
|
Chris@245
|
92 :issue_status => {
|
Chris@245
|
93 '4' => {'5' => ['always']},
|
Chris@245
|
94 '3' => {'1' => ['always'], '2' => ['always']}
|
Chris@245
|
95 }
|
Chris@0
|
96 assert_redirected_to '/workflows/edit?role_id=2&tracker_id=1'
|
Chris@909
|
97
|
Chris@1464
|
98 assert_equal 3, WorkflowTransition.where(:tracker_id => 1, :role_id => 2).count
|
Chris@1464
|
99 assert_not_nil WorkflowTransition.where(:role_id => 2, :tracker_id => 1, :old_status_id => 3, :new_status_id => 2).first
|
Chris@1464
|
100 assert_nil WorkflowTransition.where(:role_id => 2, :tracker_id => 1, :old_status_id => 5, :new_status_id => 4).first
|
Chris@0
|
101 end
|
Chris@909
|
102
|
Chris@245
|
103 def test_post_edit_with_additional_transitions
|
Chris@245
|
104 post :edit, :role_id => 2, :tracker_id => 1,
|
Chris@245
|
105 :issue_status => {
|
Chris@245
|
106 '4' => {'5' => ['always']},
|
Chris@245
|
107 '3' => {'1' => ['author'], '2' => ['assignee'], '4' => ['author', 'assignee']}
|
Chris@245
|
108 }
|
Chris@245
|
109 assert_redirected_to '/workflows/edit?role_id=2&tracker_id=1'
|
Chris@909
|
110
|
Chris@1464
|
111 assert_equal 4, WorkflowTransition.where(:tracker_id => 1, :role_id => 2).count
|
Chris@909
|
112
|
Chris@1464
|
113 w = WorkflowTransition.where(:role_id => 2, :tracker_id => 1, :old_status_id => 4, :new_status_id => 5).first
|
Chris@245
|
114 assert ! w.author
|
Chris@245
|
115 assert ! w.assignee
|
Chris@1464
|
116 w = WorkflowTransition.where(:role_id => 2, :tracker_id => 1, :old_status_id => 3, :new_status_id => 1).first
|
Chris@245
|
117 assert w.author
|
Chris@245
|
118 assert ! w.assignee
|
Chris@1464
|
119 w = WorkflowTransition.where(:role_id => 2, :tracker_id => 1, :old_status_id => 3, :new_status_id => 2).first
|
Chris@245
|
120 assert ! w.author
|
Chris@245
|
121 assert w.assignee
|
Chris@1464
|
122 w = WorkflowTransition.where(:role_id => 2, :tracker_id => 1, :old_status_id => 3, :new_status_id => 4).first
|
Chris@245
|
123 assert w.author
|
Chris@245
|
124 assert w.assignee
|
Chris@245
|
125 end
|
Chris@909
|
126
|
Chris@0
|
127 def test_clear_workflow
|
Chris@1464
|
128 assert WorkflowTransition.where(:role_id => 1, :tracker_id => 2).count > 0
|
Chris@0
|
129
|
Chris@1464
|
130 post :edit, :role_id => 1, :tracker_id => 2
|
Chris@1464
|
131 assert_equal 0, WorkflowTransition.where(:role_id => 1, :tracker_id => 2).count
|
Chris@1115
|
132 end
|
Chris@1115
|
133
|
Chris@1115
|
134 def test_get_permissions
|
Chris@1115
|
135 get :permissions
|
Chris@1115
|
136
|
Chris@1115
|
137 assert_response :success
|
Chris@1115
|
138 assert_template 'permissions'
|
Chris@1115
|
139 assert_not_nil assigns(:roles)
|
Chris@1115
|
140 assert_not_nil assigns(:trackers)
|
Chris@1115
|
141 end
|
Chris@1115
|
142
|
Chris@1115
|
143 def test_get_permissions_with_role_and_tracker
|
Chris@1115
|
144 WorkflowPermission.delete_all
|
Chris@1115
|
145 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :field_name => 'assigned_to_id', :rule => 'required')
|
Chris@1115
|
146 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :field_name => 'fixed_version_id', :rule => 'required')
|
Chris@1115
|
147 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 3, :field_name => 'fixed_version_id', :rule => 'readonly')
|
Chris@1115
|
148
|
Chris@1115
|
149 get :permissions, :role_id => 1, :tracker_id => 2
|
Chris@1115
|
150 assert_response :success
|
Chris@1115
|
151 assert_template 'permissions'
|
Chris@1115
|
152
|
Chris@1115
|
153 assert_select 'input[name=role_id][value=1]'
|
Chris@1115
|
154 assert_select 'input[name=tracker_id][value=2]'
|
Chris@1115
|
155
|
Chris@1115
|
156 # Required field
|
Chris@1115
|
157 assert_select 'select[name=?]', 'permissions[assigned_to_id][2]' do
|
Chris@1115
|
158 assert_select 'option[value=]'
|
Chris@1115
|
159 assert_select 'option[value=][selected=selected]', 0
|
Chris@1115
|
160 assert_select 'option[value=readonly]', :text => 'Read-only'
|
Chris@1115
|
161 assert_select 'option[value=readonly][selected=selected]', 0
|
Chris@1115
|
162 assert_select 'option[value=required]', :text => 'Required'
|
Chris@1115
|
163 assert_select 'option[value=required][selected=selected]'
|
Chris@1115
|
164 end
|
Chris@1115
|
165
|
Chris@1115
|
166 # Read-only field
|
Chris@1115
|
167 assert_select 'select[name=?]', 'permissions[fixed_version_id][3]' do
|
Chris@1115
|
168 assert_select 'option[value=]'
|
Chris@1115
|
169 assert_select 'option[value=][selected=selected]', 0
|
Chris@1115
|
170 assert_select 'option[value=readonly]', :text => 'Read-only'
|
Chris@1115
|
171 assert_select 'option[value=readonly][selected=selected]'
|
Chris@1115
|
172 assert_select 'option[value=required]', :text => 'Required'
|
Chris@1115
|
173 assert_select 'option[value=required][selected=selected]', 0
|
Chris@1115
|
174 end
|
Chris@1115
|
175
|
Chris@1115
|
176 # Other field
|
Chris@1115
|
177 assert_select 'select[name=?]', 'permissions[due_date][3]' do
|
Chris@1115
|
178 assert_select 'option[value=]'
|
Chris@1115
|
179 assert_select 'option[value=][selected=selected]', 0
|
Chris@1115
|
180 assert_select 'option[value=readonly]', :text => 'Read-only'
|
Chris@1115
|
181 assert_select 'option[value=readonly][selected=selected]', 0
|
Chris@1115
|
182 assert_select 'option[value=required]', :text => 'Required'
|
Chris@1115
|
183 assert_select 'option[value=required][selected=selected]', 0
|
Chris@1115
|
184 end
|
Chris@1115
|
185 end
|
Chris@1115
|
186
|
Chris@1115
|
187 def test_get_permissions_with_required_custom_field_should_not_show_required_option
|
Chris@1115
|
188 cf = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :tracker_ids => [1], :is_required => true)
|
Chris@1115
|
189
|
Chris@1115
|
190 get :permissions, :role_id => 1, :tracker_id => 1
|
Chris@1115
|
191 assert_response :success
|
Chris@1115
|
192 assert_template 'permissions'
|
Chris@1115
|
193
|
Chris@1115
|
194 # Custom field that is always required
|
Chris@1115
|
195 # The default option is "(Required)"
|
Chris@1115
|
196 assert_select 'select[name=?]', "permissions[#{cf.id}][3]" do
|
Chris@1115
|
197 assert_select 'option[value=]'
|
Chris@1115
|
198 assert_select 'option[value=readonly]', :text => 'Read-only'
|
Chris@1115
|
199 assert_select 'option[value=required]', 0
|
Chris@1115
|
200 end
|
Chris@1115
|
201 end
|
Chris@1115
|
202
|
Chris@1464
|
203 def test_get_permissions_should_disable_hidden_custom_fields
|
Chris@1464
|
204 cf1 = IssueCustomField.generate!(:tracker_ids => [1], :visible => true)
|
Chris@1464
|
205 cf2 = IssueCustomField.generate!(:tracker_ids => [1], :visible => false, :role_ids => [1])
|
Chris@1464
|
206 cf3 = IssueCustomField.generate!(:tracker_ids => [1], :visible => false, :role_ids => [1, 2])
|
Chris@1464
|
207
|
Chris@1464
|
208 get :permissions, :role_id => 2, :tracker_id => 1
|
Chris@1464
|
209 assert_response :success
|
Chris@1464
|
210 assert_template 'permissions'
|
Chris@1464
|
211
|
Chris@1464
|
212 assert_select 'select[name=?]:not(.disabled)', "permissions[#{cf1.id}][1]"
|
Chris@1464
|
213 assert_select 'select[name=?]:not(.disabled)', "permissions[#{cf3.id}][1]"
|
Chris@1464
|
214
|
Chris@1464
|
215 assert_select 'select[name=?][disabled=disabled]', "permissions[#{cf2.id}][1]" do
|
Chris@1464
|
216 assert_select 'option[value=][selected=selected]', :text => 'Hidden'
|
Chris@1464
|
217 end
|
Chris@1464
|
218 end
|
Chris@1464
|
219
|
Chris@1115
|
220 def test_get_permissions_with_role_and_tracker_and_all_statuses
|
Chris@1115
|
221 WorkflowTransition.delete_all
|
Chris@1115
|
222
|
Chris@1115
|
223 get :permissions, :role_id => 1, :tracker_id => 2, :used_statuses_only => '0'
|
Chris@1115
|
224 assert_response :success
|
Chris@1115
|
225 assert_equal IssueStatus.sorted.all, assigns(:statuses)
|
Chris@1115
|
226 end
|
Chris@1115
|
227
|
Chris@1115
|
228 def test_post_permissions
|
Chris@1115
|
229 WorkflowPermission.delete_all
|
Chris@1115
|
230
|
Chris@1115
|
231 post :permissions, :role_id => 1, :tracker_id => 2, :permissions => {
|
Chris@1115
|
232 'assigned_to_id' => {'1' => '', '2' => 'readonly', '3' => ''},
|
Chris@1115
|
233 'fixed_version_id' => {'1' => 'required', '2' => 'readonly', '3' => ''},
|
Chris@1115
|
234 'due_date' => {'1' => '', '2' => '', '3' => ''},
|
Chris@1115
|
235 }
|
Chris@1115
|
236 assert_redirected_to '/workflows/permissions?role_id=1&tracker_id=2'
|
Chris@1115
|
237
|
Chris@1115
|
238 workflows = WorkflowPermission.all
|
Chris@1115
|
239 assert_equal 3, workflows.size
|
Chris@1115
|
240 workflows.each do |workflow|
|
Chris@1115
|
241 assert_equal 1, workflow.role_id
|
Chris@1115
|
242 assert_equal 2, workflow.tracker_id
|
Chris@1115
|
243 end
|
Chris@1115
|
244 assert workflows.detect {|wf| wf.old_status_id == 2 && wf.field_name == 'assigned_to_id' && wf.rule == 'readonly'}
|
Chris@1115
|
245 assert workflows.detect {|wf| wf.old_status_id == 1 && wf.field_name == 'fixed_version_id' && wf.rule == 'required'}
|
Chris@1115
|
246 assert workflows.detect {|wf| wf.old_status_id == 2 && wf.field_name == 'fixed_version_id' && wf.rule == 'readonly'}
|
Chris@1115
|
247 end
|
Chris@1115
|
248
|
Chris@1115
|
249 def test_post_permissions_should_clear_permissions
|
Chris@1115
|
250 WorkflowPermission.delete_all
|
Chris@1115
|
251 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :field_name => 'assigned_to_id', :rule => 'required')
|
Chris@1115
|
252 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :field_name => 'fixed_version_id', :rule => 'required')
|
Chris@1115
|
253 wf1 = WorkflowPermission.create!(:role_id => 1, :tracker_id => 3, :old_status_id => 2, :field_name => 'fixed_version_id', :rule => 'required')
|
Chris@1115
|
254 wf2 = WorkflowPermission.create!(:role_id => 2, :tracker_id => 2, :old_status_id => 3, :field_name => 'fixed_version_id', :rule => 'readonly')
|
Chris@1115
|
255
|
Chris@1115
|
256 post :permissions, :role_id => 1, :tracker_id => 2
|
Chris@1115
|
257 assert_redirected_to '/workflows/permissions?role_id=1&tracker_id=2'
|
Chris@1115
|
258
|
Chris@1115
|
259 workflows = WorkflowPermission.all
|
Chris@1115
|
260 assert_equal 2, workflows.size
|
Chris@1115
|
261 assert wf1.reload
|
Chris@1115
|
262 assert wf2.reload
|
Chris@0
|
263 end
|
Chris@909
|
264
|
Chris@0
|
265 def test_get_copy
|
Chris@0
|
266 get :copy
|
Chris@0
|
267 assert_response :success
|
Chris@0
|
268 assert_template 'copy'
|
Chris@1115
|
269 assert_select 'select[name=source_tracker_id]' do
|
Chris@1115
|
270 assert_select 'option[value=1]', :text => 'Bug'
|
Chris@1115
|
271 end
|
Chris@1115
|
272 assert_select 'select[name=source_role_id]' do
|
Chris@1115
|
273 assert_select 'option[value=2]', :text => 'Developer'
|
Chris@1115
|
274 end
|
Chris@1115
|
275 assert_select 'select[name=?]', 'target_tracker_ids[]' do
|
Chris@1115
|
276 assert_select 'option[value=3]', :text => 'Support request'
|
Chris@1115
|
277 end
|
Chris@1115
|
278 assert_select 'select[name=?]', 'target_role_ids[]' do
|
Chris@1115
|
279 assert_select 'option[value=1]', :text => 'Manager'
|
Chris@1115
|
280 end
|
Chris@0
|
281 end
|
Chris@909
|
282
|
Chris@0
|
283 def test_post_copy_one_to_one
|
Chris@0
|
284 source_transitions = status_transitions(:tracker_id => 1, :role_id => 2)
|
Chris@909
|
285
|
Chris@0
|
286 post :copy, :source_tracker_id => '1', :source_role_id => '2',
|
Chris@0
|
287 :target_tracker_ids => ['3'], :target_role_ids => ['1']
|
Chris@0
|
288 assert_response 302
|
Chris@0
|
289 assert_equal source_transitions, status_transitions(:tracker_id => 3, :role_id => 1)
|
Chris@0
|
290 end
|
Chris@909
|
291
|
Chris@0
|
292 def test_post_copy_one_to_many
|
Chris@0
|
293 source_transitions = status_transitions(:tracker_id => 1, :role_id => 2)
|
Chris@909
|
294
|
Chris@0
|
295 post :copy, :source_tracker_id => '1', :source_role_id => '2',
|
Chris@0
|
296 :target_tracker_ids => ['2', '3'], :target_role_ids => ['1', '3']
|
Chris@0
|
297 assert_response 302
|
Chris@0
|
298 assert_equal source_transitions, status_transitions(:tracker_id => 2, :role_id => 1)
|
Chris@0
|
299 assert_equal source_transitions, status_transitions(:tracker_id => 3, :role_id => 1)
|
Chris@0
|
300 assert_equal source_transitions, status_transitions(:tracker_id => 2, :role_id => 3)
|
Chris@0
|
301 assert_equal source_transitions, status_transitions(:tracker_id => 3, :role_id => 3)
|
Chris@0
|
302 end
|
Chris@909
|
303
|
Chris@0
|
304 def test_post_copy_many_to_many
|
Chris@0
|
305 source_t2 = status_transitions(:tracker_id => 2, :role_id => 2)
|
Chris@0
|
306 source_t3 = status_transitions(:tracker_id => 3, :role_id => 2)
|
Chris@909
|
307
|
Chris@0
|
308 post :copy, :source_tracker_id => 'any', :source_role_id => '2',
|
Chris@0
|
309 :target_tracker_ids => ['2', '3'], :target_role_ids => ['1', '3']
|
Chris@0
|
310 assert_response 302
|
Chris@0
|
311 assert_equal source_t2, status_transitions(:tracker_id => 2, :role_id => 1)
|
Chris@0
|
312 assert_equal source_t3, status_transitions(:tracker_id => 3, :role_id => 1)
|
Chris@0
|
313 assert_equal source_t2, status_transitions(:tracker_id => 2, :role_id => 3)
|
Chris@0
|
314 assert_equal source_t3, status_transitions(:tracker_id => 3, :role_id => 3)
|
Chris@0
|
315 end
|
Chris@909
|
316
|
Chris@1464
|
317 def test_post_copy_with_incomplete_source_specification_should_fail
|
Chris@1464
|
318 assert_no_difference 'WorkflowRule.count' do
|
Chris@1464
|
319 post :copy,
|
Chris@1464
|
320 :source_tracker_id => '', :source_role_id => '2',
|
Chris@1464
|
321 :target_tracker_ids => ['2', '3'], :target_role_ids => ['1', '3']
|
Chris@1464
|
322 assert_response 200
|
Chris@1464
|
323 assert_select 'div.flash.error', :text => 'Please select a source tracker or role'
|
Chris@1464
|
324 end
|
Chris@1464
|
325 end
|
Chris@1464
|
326
|
Chris@1464
|
327 def test_post_copy_with_incomplete_target_specification_should_fail
|
Chris@1464
|
328 assert_no_difference 'WorkflowRule.count' do
|
Chris@1464
|
329 post :copy,
|
Chris@1464
|
330 :source_tracker_id => '1', :source_role_id => '2',
|
Chris@1464
|
331 :target_tracker_ids => ['2', '3']
|
Chris@1464
|
332 assert_response 200
|
Chris@1464
|
333 assert_select 'div.flash.error', :text => 'Please select target tracker(s) and role(s)'
|
Chris@1464
|
334 end
|
Chris@1464
|
335 end
|
Chris@1464
|
336
|
Chris@0
|
337 # Returns an array of status transitions that can be compared
|
Chris@0
|
338 def status_transitions(conditions)
|
Chris@1464
|
339 WorkflowTransition.
|
Chris@1464
|
340 where(conditions).
|
Chris@1464
|
341 order('tracker_id, role_id, old_status_id, new_status_id').
|
Chris@1464
|
342 collect {|w| [w.old_status, w.new_status_id]}
|
Chris@0
|
343 end
|
Chris@0
|
344 end
|