annotate .svn/pristine/08/08ba366b8c38c360363f9f7f4597d533502606b0.svn-base @ 1295:622f24f53b42 redmine-2.3

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