annotate .svn/pristine/fe/fe48a02cf4f3380a268baa6d0b7204b03f3af240.svn-base @ 1494:e248c7af89ec redmine-2.4

Update to Redmine SVN revision 12979 on 2.4-stable branch
author Chris Cannam
date Mon, 17 Mar 2014 08:54:02 +0000
parents 261b3d9a4903
children
rev   line source
Chris@1464 1 # Redmine - project management software
Chris@1464 2 # Copyright (C) 2006-2013 Jean-Philippe Lang
Chris@1464 3 #
Chris@1464 4 # This program is free software; you can redistribute it and/or
Chris@1464 5 # modify it under the terms of the GNU General Public License
Chris@1464 6 # as published by the Free Software Foundation; either version 2
Chris@1464 7 # of the License, or (at your option) any later version.
Chris@1464 8 #
Chris@1464 9 # This program is distributed in the hope that it will be useful,
Chris@1464 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
Chris@1464 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
Chris@1464 12 # GNU General Public License for more details.
Chris@1464 13 #
Chris@1464 14 # You should have received a copy of the GNU General Public License
Chris@1464 15 # along with this program; if not, write to the Free Software
Chris@1464 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
Chris@1464 17
Chris@1464 18 require File.expand_path('../../test_helper', __FILE__)
Chris@1464 19
Chris@1464 20 class WorkflowsControllerTest < ActionController::TestCase
Chris@1464 21 fixtures :roles, :trackers, :workflows, :users, :issue_statuses
Chris@1464 22
Chris@1464 23 def setup
Chris@1464 24 User.current = nil
Chris@1464 25 @request.session[:user_id] = 1 # admin
Chris@1464 26 end
Chris@1464 27
Chris@1464 28 def test_index
Chris@1464 29 get :index
Chris@1464 30 assert_response :success
Chris@1464 31 assert_template 'index'
Chris@1464 32
Chris@1464 33 count = WorkflowTransition.where(:role_id => 1, :tracker_id => 2).count
Chris@1464 34 assert_tag :tag => 'a', :content => count.to_s,
Chris@1464 35 :attributes => { :href => '/workflows/edit?role_id=1&amp;tracker_id=2' }
Chris@1464 36 end
Chris@1464 37
Chris@1464 38 def test_get_edit
Chris@1464 39 get :edit
Chris@1464 40 assert_response :success
Chris@1464 41 assert_template 'edit'
Chris@1464 42 assert_not_nil assigns(:roles)
Chris@1464 43 assert_not_nil assigns(:trackers)
Chris@1464 44 end
Chris@1464 45
Chris@1464 46 def test_get_edit_with_role_and_tracker
Chris@1464 47 WorkflowTransition.delete_all
Chris@1464 48 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 2, :new_status_id => 3)
Chris@1464 49 WorkflowTransition.create!(:role_id => 2, :tracker_id => 1, :old_status_id => 3, :new_status_id => 5)
Chris@1464 50
Chris@1464 51 get :edit, :role_id => 2, :tracker_id => 1
Chris@1464 52 assert_response :success
Chris@1464 53 assert_template 'edit'
Chris@1464 54
Chris@1464 55 # used status only
Chris@1464 56 assert_not_nil assigns(:statuses)
Chris@1464 57 assert_equal [2, 3, 5], assigns(:statuses).collect(&:id)
Chris@1464 58
Chris@1464 59 # allowed transitions
Chris@1464 60 assert_tag :tag => 'input', :attributes => { :type => 'checkbox',
Chris@1464 61 :name => 'issue_status[3][5][]',
Chris@1464 62 :value => 'always',
Chris@1464 63 :checked => 'checked' }
Chris@1464 64 # not allowed
Chris@1464 65 assert_tag :tag => 'input', :attributes => { :type => 'checkbox',
Chris@1464 66 :name => 'issue_status[3][2][]',
Chris@1464 67 :value => 'always',
Chris@1464 68 :checked => nil }
Chris@1464 69 # unused
Chris@1464 70 assert_no_tag :tag => 'input', :attributes => { :type => 'checkbox',
Chris@1464 71 :name => 'issue_status[1][1][]' }
Chris@1464 72 end
Chris@1464 73
Chris@1464 74 def test_get_edit_with_role_and_tracker_and_all_statuses
Chris@1464 75 WorkflowTransition.delete_all
Chris@1464 76
Chris@1464 77 get :edit, :role_id => 2, :tracker_id => 1, :used_statuses_only => '0'
Chris@1464 78 assert_response :success
Chris@1464 79 assert_template 'edit'
Chris@1464 80
Chris@1464 81 assert_not_nil assigns(:statuses)
Chris@1464 82 assert_equal IssueStatus.count, assigns(:statuses).size
Chris@1464 83
Chris@1464 84 assert_tag :tag => 'input', :attributes => { :type => 'checkbox',
Chris@1464 85 :name => 'issue_status[1][1][]',
Chris@1464 86 :value => 'always',
Chris@1464 87 :checked => nil }
Chris@1464 88 end
Chris@1464 89
Chris@1464 90 def test_post_edit
Chris@1464 91 post :edit, :role_id => 2, :tracker_id => 1,
Chris@1464 92 :issue_status => {
Chris@1464 93 '4' => {'5' => ['always']},
Chris@1464 94 '3' => {'1' => ['always'], '2' => ['always']}
Chris@1464 95 }
Chris@1464 96 assert_redirected_to '/workflows/edit?role_id=2&tracker_id=1'
Chris@1464 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@1464 101 end
Chris@1464 102
Chris@1464 103 def test_post_edit_with_additional_transitions
Chris@1464 104 post :edit, :role_id => 2, :tracker_id => 1,
Chris@1464 105 :issue_status => {
Chris@1464 106 '4' => {'5' => ['always']},
Chris@1464 107 '3' => {'1' => ['author'], '2' => ['assignee'], '4' => ['author', 'assignee']}
Chris@1464 108 }
Chris@1464 109 assert_redirected_to '/workflows/edit?role_id=2&tracker_id=1'
Chris@1464 110
Chris@1464 111 assert_equal 4, WorkflowTransition.where(:tracker_id => 1, :role_id => 2).count
Chris@1464 112
Chris@1464 113 w = WorkflowTransition.where(:role_id => 2, :tracker_id => 1, :old_status_id => 4, :new_status_id => 5).first
Chris@1464 114 assert ! w.author
Chris@1464 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@1464 117 assert w.author
Chris@1464 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@1464 120 assert ! w.author
Chris@1464 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@1464 123 assert w.author
Chris@1464 124 assert w.assignee
Chris@1464 125 end
Chris@1464 126
Chris@1464 127 def test_clear_workflow
Chris@1464 128 assert WorkflowTransition.where(:role_id => 1, :tracker_id => 2).count > 0
Chris@1464 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@1464 132 end
Chris@1464 133
Chris@1464 134 def test_get_permissions
Chris@1464 135 get :permissions
Chris@1464 136
Chris@1464 137 assert_response :success
Chris@1464 138 assert_template 'permissions'
Chris@1464 139 assert_not_nil assigns(:roles)
Chris@1464 140 assert_not_nil assigns(:trackers)
Chris@1464 141 end
Chris@1464 142
Chris@1464 143 def test_get_permissions_with_role_and_tracker
Chris@1464 144 WorkflowPermission.delete_all
Chris@1464 145 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :field_name => 'assigned_to_id', :rule => 'required')
Chris@1464 146 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :field_name => 'fixed_version_id', :rule => 'required')
Chris@1464 147 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 3, :field_name => 'fixed_version_id', :rule => 'readonly')
Chris@1464 148
Chris@1464 149 get :permissions, :role_id => 1, :tracker_id => 2
Chris@1464 150 assert_response :success
Chris@1464 151 assert_template 'permissions'
Chris@1464 152
Chris@1464 153 assert_select 'input[name=role_id][value=1]'
Chris@1464 154 assert_select 'input[name=tracker_id][value=2]'
Chris@1464 155
Chris@1464 156 # Required field
Chris@1464 157 assert_select 'select[name=?]', 'permissions[assigned_to_id][2]' do
Chris@1464 158 assert_select 'option[value=]'
Chris@1464 159 assert_select 'option[value=][selected=selected]', 0
Chris@1464 160 assert_select 'option[value=readonly]', :text => 'Read-only'
Chris@1464 161 assert_select 'option[value=readonly][selected=selected]', 0
Chris@1464 162 assert_select 'option[value=required]', :text => 'Required'
Chris@1464 163 assert_select 'option[value=required][selected=selected]'
Chris@1464 164 end
Chris@1464 165
Chris@1464 166 # Read-only field
Chris@1464 167 assert_select 'select[name=?]', 'permissions[fixed_version_id][3]' do
Chris@1464 168 assert_select 'option[value=]'
Chris@1464 169 assert_select 'option[value=][selected=selected]', 0
Chris@1464 170 assert_select 'option[value=readonly]', :text => 'Read-only'
Chris@1464 171 assert_select 'option[value=readonly][selected=selected]'
Chris@1464 172 assert_select 'option[value=required]', :text => 'Required'
Chris@1464 173 assert_select 'option[value=required][selected=selected]', 0
Chris@1464 174 end
Chris@1464 175
Chris@1464 176 # Other field
Chris@1464 177 assert_select 'select[name=?]', 'permissions[due_date][3]' do
Chris@1464 178 assert_select 'option[value=]'
Chris@1464 179 assert_select 'option[value=][selected=selected]', 0
Chris@1464 180 assert_select 'option[value=readonly]', :text => 'Read-only'
Chris@1464 181 assert_select 'option[value=readonly][selected=selected]', 0
Chris@1464 182 assert_select 'option[value=required]', :text => 'Required'
Chris@1464 183 assert_select 'option[value=required][selected=selected]', 0
Chris@1464 184 end
Chris@1464 185 end
Chris@1464 186
Chris@1464 187 def test_get_permissions_with_required_custom_field_should_not_show_required_option
Chris@1464 188 cf = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :tracker_ids => [1], :is_required => true)
Chris@1464 189
Chris@1464 190 get :permissions, :role_id => 1, :tracker_id => 1
Chris@1464 191 assert_response :success
Chris@1464 192 assert_template 'permissions'
Chris@1464 193
Chris@1464 194 # Custom field that is always required
Chris@1464 195 # The default option is "(Required)"
Chris@1464 196 assert_select 'select[name=?]', "permissions[#{cf.id}][3]" do
Chris@1464 197 assert_select 'option[value=]'
Chris@1464 198 assert_select 'option[value=readonly]', :text => 'Read-only'
Chris@1464 199 assert_select 'option[value=required]', 0
Chris@1464 200 end
Chris@1464 201 end
Chris@1464 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@1464 220 def test_get_permissions_with_role_and_tracker_and_all_statuses
Chris@1464 221 WorkflowTransition.delete_all
Chris@1464 222
Chris@1464 223 get :permissions, :role_id => 1, :tracker_id => 2, :used_statuses_only => '0'
Chris@1464 224 assert_response :success
Chris@1464 225 assert_equal IssueStatus.sorted.all, assigns(:statuses)
Chris@1464 226 end
Chris@1464 227
Chris@1464 228 def test_post_permissions
Chris@1464 229 WorkflowPermission.delete_all
Chris@1464 230
Chris@1464 231 post :permissions, :role_id => 1, :tracker_id => 2, :permissions => {
Chris@1464 232 'assigned_to_id' => {'1' => '', '2' => 'readonly', '3' => ''},
Chris@1464 233 'fixed_version_id' => {'1' => 'required', '2' => 'readonly', '3' => ''},
Chris@1464 234 'due_date' => {'1' => '', '2' => '', '3' => ''},
Chris@1464 235 }
Chris@1464 236 assert_redirected_to '/workflows/permissions?role_id=1&tracker_id=2'
Chris@1464 237
Chris@1464 238 workflows = WorkflowPermission.all
Chris@1464 239 assert_equal 3, workflows.size
Chris@1464 240 workflows.each do |workflow|
Chris@1464 241 assert_equal 1, workflow.role_id
Chris@1464 242 assert_equal 2, workflow.tracker_id
Chris@1464 243 end
Chris@1464 244 assert workflows.detect {|wf| wf.old_status_id == 2 && wf.field_name == 'assigned_to_id' && wf.rule == 'readonly'}
Chris@1464 245 assert workflows.detect {|wf| wf.old_status_id == 1 && wf.field_name == 'fixed_version_id' && wf.rule == 'required'}
Chris@1464 246 assert workflows.detect {|wf| wf.old_status_id == 2 && wf.field_name == 'fixed_version_id' && wf.rule == 'readonly'}
Chris@1464 247 end
Chris@1464 248
Chris@1464 249 def test_post_permissions_should_clear_permissions
Chris@1464 250 WorkflowPermission.delete_all
Chris@1464 251 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :field_name => 'assigned_to_id', :rule => 'required')
Chris@1464 252 WorkflowPermission.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :field_name => 'fixed_version_id', :rule => 'required')
Chris@1464 253 wf1 = WorkflowPermission.create!(:role_id => 1, :tracker_id => 3, :old_status_id => 2, :field_name => 'fixed_version_id', :rule => 'required')
Chris@1464 254 wf2 = WorkflowPermission.create!(:role_id => 2, :tracker_id => 2, :old_status_id => 3, :field_name => 'fixed_version_id', :rule => 'readonly')
Chris@1464 255
Chris@1464 256 post :permissions, :role_id => 1, :tracker_id => 2
Chris@1464 257 assert_redirected_to '/workflows/permissions?role_id=1&tracker_id=2'
Chris@1464 258
Chris@1464 259 workflows = WorkflowPermission.all
Chris@1464 260 assert_equal 2, workflows.size
Chris@1464 261 assert wf1.reload
Chris@1464 262 assert wf2.reload
Chris@1464 263 end
Chris@1464 264
Chris@1464 265 def test_get_copy
Chris@1464 266 get :copy
Chris@1464 267 assert_response :success
Chris@1464 268 assert_template 'copy'
Chris@1464 269 assert_select 'select[name=source_tracker_id]' do
Chris@1464 270 assert_select 'option[value=1]', :text => 'Bug'
Chris@1464 271 end
Chris@1464 272 assert_select 'select[name=source_role_id]' do
Chris@1464 273 assert_select 'option[value=2]', :text => 'Developer'
Chris@1464 274 end
Chris@1464 275 assert_select 'select[name=?]', 'target_tracker_ids[]' do
Chris@1464 276 assert_select 'option[value=3]', :text => 'Support request'
Chris@1464 277 end
Chris@1464 278 assert_select 'select[name=?]', 'target_role_ids[]' do
Chris@1464 279 assert_select 'option[value=1]', :text => 'Manager'
Chris@1464 280 end
Chris@1464 281 end
Chris@1464 282
Chris@1464 283 def test_post_copy_one_to_one
Chris@1464 284 source_transitions = status_transitions(:tracker_id => 1, :role_id => 2)
Chris@1464 285
Chris@1464 286 post :copy, :source_tracker_id => '1', :source_role_id => '2',
Chris@1464 287 :target_tracker_ids => ['3'], :target_role_ids => ['1']
Chris@1464 288 assert_response 302
Chris@1464 289 assert_equal source_transitions, status_transitions(:tracker_id => 3, :role_id => 1)
Chris@1464 290 end
Chris@1464 291
Chris@1464 292 def test_post_copy_one_to_many
Chris@1464 293 source_transitions = status_transitions(:tracker_id => 1, :role_id => 2)
Chris@1464 294
Chris@1464 295 post :copy, :source_tracker_id => '1', :source_role_id => '2',
Chris@1464 296 :target_tracker_ids => ['2', '3'], :target_role_ids => ['1', '3']
Chris@1464 297 assert_response 302
Chris@1464 298 assert_equal source_transitions, status_transitions(:tracker_id => 2, :role_id => 1)
Chris@1464 299 assert_equal source_transitions, status_transitions(:tracker_id => 3, :role_id => 1)
Chris@1464 300 assert_equal source_transitions, status_transitions(:tracker_id => 2, :role_id => 3)
Chris@1464 301 assert_equal source_transitions, status_transitions(:tracker_id => 3, :role_id => 3)
Chris@1464 302 end
Chris@1464 303
Chris@1464 304 def test_post_copy_many_to_many
Chris@1464 305 source_t2 = status_transitions(:tracker_id => 2, :role_id => 2)
Chris@1464 306 source_t3 = status_transitions(:tracker_id => 3, :role_id => 2)
Chris@1464 307
Chris@1464 308 post :copy, :source_tracker_id => 'any', :source_role_id => '2',
Chris@1464 309 :target_tracker_ids => ['2', '3'], :target_role_ids => ['1', '3']
Chris@1464 310 assert_response 302
Chris@1464 311 assert_equal source_t2, status_transitions(:tracker_id => 2, :role_id => 1)
Chris@1464 312 assert_equal source_t3, status_transitions(:tracker_id => 3, :role_id => 1)
Chris@1464 313 assert_equal source_t2, status_transitions(:tracker_id => 2, :role_id => 3)
Chris@1464 314 assert_equal source_t3, status_transitions(:tracker_id => 3, :role_id => 3)
Chris@1464 315 end
Chris@1464 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@1464 337 # Returns an array of status transitions that can be compared
Chris@1464 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 all.
Chris@1464 343 collect {|w| [w.old_status, w.new_status_id]}
Chris@1464 344 end
Chris@1464 345 end