To check out this repository please hg clone the following URL, or open the URL using EasyMercurial or your preferred Mercurial client.
root / .svn / pristine / 85 / 85cb4b5b2aa4587d3f8e85f0dc40fcad929d49ad.svn-base @ 1297:0a574315af3e
History | View | Annotate | Download (13.3 KB)
| 1 | 1296:038ba2d95de8 | Chris | # Redmine - project management software |
|---|---|---|---|
| 2 | # Copyright (C) 2006-2012 Jean-Philippe Lang |
||
| 3 | # |
||
| 4 | # This program is free software; you can redistribute it and/or |
||
| 5 | # modify it under the terms of the GNU General Public License |
||
| 6 | # as published by the Free Software Foundation; either version 2 |
||
| 7 | # of the License, or (at your option) any later version. |
||
| 8 | # |
||
| 9 | # This program is distributed in the hope that it will be useful, |
||
| 10 | # but WITHOUT ANY WARRANTY; without even the implied warranty of |
||
| 11 | # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
||
| 12 | # GNU General Public License for more details. |
||
| 13 | # |
||
| 14 | # You should have received a copy of the GNU General Public License |
||
| 15 | # along with this program; if not, write to the Free Software |
||
| 16 | # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
||
| 17 | |||
| 18 | require File.expand_path('../../test_helper', __FILE__)
|
||
| 19 | require 'users_controller' |
||
| 20 | |||
| 21 | # Re-raise errors caught by the controller. |
||
| 22 | class UsersController; def rescue_action(e) raise e end; end |
||
| 23 | |||
| 24 | class UsersControllerTest < ActionController::TestCase |
||
| 25 | include Redmine::I18n |
||
| 26 | |||
| 27 | fixtures :users, :projects, :members, :member_roles, :roles, |
||
| 28 | :custom_fields, :custom_values, :groups_users, |
||
| 29 | :auth_sources |
||
| 30 | |||
| 31 | def setup |
||
| 32 | @controller = UsersController.new |
||
| 33 | @request = ActionController::TestRequest.new |
||
| 34 | @response = ActionController::TestResponse.new |
||
| 35 | User.current = nil |
||
| 36 | @request.session[:user_id] = 1 # admin |
||
| 37 | end |
||
| 38 | |||
| 39 | def test_index |
||
| 40 | get :index |
||
| 41 | assert_response :success |
||
| 42 | assert_template 'index' |
||
| 43 | end |
||
| 44 | |||
| 45 | def test_index |
||
| 46 | get :index |
||
| 47 | assert_response :success |
||
| 48 | assert_template 'index' |
||
| 49 | assert_not_nil assigns(:users) |
||
| 50 | # active users only |
||
| 51 | assert_nil assigns(:users).detect {|u| !u.active?}
|
||
| 52 | end |
||
| 53 | |||
| 54 | def test_index_with_status_filter |
||
| 55 | get :index, :status => 3 |
||
| 56 | assert_response :success |
||
| 57 | assert_template 'index' |
||
| 58 | assert_not_nil assigns(:users) |
||
| 59 | assert_equal [3], assigns(:users).map(&:status).uniq |
||
| 60 | end |
||
| 61 | |||
| 62 | def test_index_with_name_filter |
||
| 63 | get :index, :name => 'john' |
||
| 64 | assert_response :success |
||
| 65 | assert_template 'index' |
||
| 66 | users = assigns(:users) |
||
| 67 | assert_not_nil users |
||
| 68 | assert_equal 1, users.size |
||
| 69 | assert_equal 'John', users.first.firstname |
||
| 70 | end |
||
| 71 | |||
| 72 | def test_index_with_group_filter |
||
| 73 | get :index, :group_id => '10' |
||
| 74 | assert_response :success |
||
| 75 | assert_template 'index' |
||
| 76 | users = assigns(:users) |
||
| 77 | assert users.any? |
||
| 78 | assert_equal([], (users - Group.find(10).users)) |
||
| 79 | assert_select 'select[name=group_id]' do |
||
| 80 | assert_select 'option[value=10][selected=selected]' |
||
| 81 | end |
||
| 82 | end |
||
| 83 | |||
| 84 | def test_show |
||
| 85 | @request.session[:user_id] = nil |
||
| 86 | get :show, :id => 2 |
||
| 87 | assert_response :success |
||
| 88 | assert_template 'show' |
||
| 89 | assert_not_nil assigns(:user) |
||
| 90 | |||
| 91 | assert_tag 'li', :content => /Phone number/ |
||
| 92 | end |
||
| 93 | |||
| 94 | def test_show_should_not_display_hidden_custom_fields |
||
| 95 | @request.session[:user_id] = nil |
||
| 96 | UserCustomField.find_by_name('Phone number').update_attribute :visible, false
|
||
| 97 | get :show, :id => 2 |
||
| 98 | assert_response :success |
||
| 99 | assert_template 'show' |
||
| 100 | assert_not_nil assigns(:user) |
||
| 101 | |||
| 102 | assert_no_tag 'li', :content => /Phone number/ |
||
| 103 | end |
||
| 104 | |||
| 105 | def test_show_should_not_fail_when_custom_values_are_nil |
||
| 106 | user = User.find(2) |
||
| 107 | |||
| 108 | # Create a custom field to illustrate the issue |
||
| 109 | custom_field = CustomField.create!(:name => 'Testing', :field_format => 'text') |
||
| 110 | custom_value = user.custom_values.build(:custom_field => custom_field).save! |
||
| 111 | |||
| 112 | get :show, :id => 2 |
||
| 113 | assert_response :success |
||
| 114 | end |
||
| 115 | |||
| 116 | def test_show_inactive |
||
| 117 | @request.session[:user_id] = nil |
||
| 118 | get :show, :id => 5 |
||
| 119 | assert_response 404 |
||
| 120 | end |
||
| 121 | |||
| 122 | def test_show_should_not_reveal_users_with_no_visible_activity_or_project |
||
| 123 | @request.session[:user_id] = nil |
||
| 124 | get :show, :id => 9 |
||
| 125 | assert_response 404 |
||
| 126 | end |
||
| 127 | |||
| 128 | def test_show_inactive_by_admin |
||
| 129 | @request.session[:user_id] = 1 |
||
| 130 | get :show, :id => 5 |
||
| 131 | assert_response 200 |
||
| 132 | assert_not_nil assigns(:user) |
||
| 133 | end |
||
| 134 | |||
| 135 | def test_show_displays_memberships_based_on_project_visibility |
||
| 136 | @request.session[:user_id] = 1 |
||
| 137 | get :show, :id => 2 |
||
| 138 | assert_response :success |
||
| 139 | memberships = assigns(:memberships) |
||
| 140 | assert_not_nil memberships |
||
| 141 | project_ids = memberships.map(&:project_id) |
||
| 142 | assert project_ids.include?(2) #private project admin can see |
||
| 143 | end |
||
| 144 | |||
| 145 | def test_show_current_should_require_authentication |
||
| 146 | @request.session[:user_id] = nil |
||
| 147 | get :show, :id => 'current' |
||
| 148 | assert_response 302 |
||
| 149 | end |
||
| 150 | |||
| 151 | def test_show_current |
||
| 152 | @request.session[:user_id] = 2 |
||
| 153 | get :show, :id => 'current' |
||
| 154 | assert_response :success |
||
| 155 | assert_template 'show' |
||
| 156 | assert_equal User.find(2), assigns(:user) |
||
| 157 | end |
||
| 158 | |||
| 159 | def test_new |
||
| 160 | get :new |
||
| 161 | assert_response :success |
||
| 162 | assert_template :new |
||
| 163 | assert assigns(:user) |
||
| 164 | end |
||
| 165 | |||
| 166 | def test_create |
||
| 167 | Setting.bcc_recipients = '1' |
||
| 168 | |||
| 169 | assert_difference 'User.count' do |
||
| 170 | assert_difference 'ActionMailer::Base.deliveries.size' do |
||
| 171 | post :create, |
||
| 172 | :user => {
|
||
| 173 | :firstname => 'John', |
||
| 174 | :lastname => 'Doe', |
||
| 175 | :login => 'jdoe', |
||
| 176 | :password => 'secret123', |
||
| 177 | :password_confirmation => 'secret123', |
||
| 178 | :mail => 'jdoe@gmail.com', |
||
| 179 | :mail_notification => 'none' |
||
| 180 | }, |
||
| 181 | :send_information => '1' |
||
| 182 | end |
||
| 183 | end |
||
| 184 | |||
| 185 | user = User.first(:order => 'id DESC') |
||
| 186 | assert_redirected_to :controller => 'users', :action => 'edit', :id => user.id |
||
| 187 | |||
| 188 | assert_equal 'John', user.firstname |
||
| 189 | assert_equal 'Doe', user.lastname |
||
| 190 | assert_equal 'jdoe', user.login |
||
| 191 | assert_equal 'jdoe@gmail.com', user.mail |
||
| 192 | assert_equal 'none', user.mail_notification |
||
| 193 | assert user.check_password?('secret123')
|
||
| 194 | |||
| 195 | mail = ActionMailer::Base.deliveries.last |
||
| 196 | assert_not_nil mail |
||
| 197 | assert_equal [user.mail], mail.bcc |
||
| 198 | assert_mail_body_match 'secret', mail |
||
| 199 | end |
||
| 200 | |||
| 201 | def test_create_with_preferences |
||
| 202 | assert_difference 'User.count' do |
||
| 203 | post :create, |
||
| 204 | :user => {
|
||
| 205 | :firstname => 'John', |
||
| 206 | :lastname => 'Doe', |
||
| 207 | :login => 'jdoe', |
||
| 208 | :password => 'secret123', |
||
| 209 | :password_confirmation => 'secret123', |
||
| 210 | :mail => 'jdoe@gmail.com', |
||
| 211 | :mail_notification => 'none' |
||
| 212 | }, |
||
| 213 | :pref => {
|
||
| 214 | 'hide_mail' => '1', |
||
| 215 | 'time_zone' => 'Paris', |
||
| 216 | 'comments_sorting' => 'desc', |
||
| 217 | 'warn_on_leaving_unsaved' => '0' |
||
| 218 | } |
||
| 219 | end |
||
| 220 | user = User.first(:order => 'id DESC') |
||
| 221 | assert_equal 'jdoe', user.login |
||
| 222 | assert_equal true, user.pref.hide_mail |
||
| 223 | assert_equal 'Paris', user.pref.time_zone |
||
| 224 | assert_equal 'desc', user.pref[:comments_sorting] |
||
| 225 | assert_equal '0', user.pref[:warn_on_leaving_unsaved] |
||
| 226 | end |
||
| 227 | |||
| 228 | def test_create_with_failure |
||
| 229 | assert_no_difference 'User.count' do |
||
| 230 | post :create, :user => {}
|
||
| 231 | end |
||
| 232 | assert_response :success |
||
| 233 | assert_template 'new' |
||
| 234 | end |
||
| 235 | |||
| 236 | def test_edit |
||
| 237 | get :edit, :id => 2 |
||
| 238 | assert_response :success |
||
| 239 | assert_template 'edit' |
||
| 240 | assert_equal User.find(2), assigns(:user) |
||
| 241 | end |
||
| 242 | |||
| 243 | def test_update |
||
| 244 | ActionMailer::Base.deliveries.clear |
||
| 245 | put :update, :id => 2, |
||
| 246 | :user => {:firstname => 'Changed', :mail_notification => 'only_assigned'},
|
||
| 247 | :pref => {:hide_mail => '1', :comments_sorting => 'desc'}
|
||
| 248 | user = User.find(2) |
||
| 249 | assert_equal 'Changed', user.firstname |
||
| 250 | assert_equal 'only_assigned', user.mail_notification |
||
| 251 | assert_equal true, user.pref[:hide_mail] |
||
| 252 | assert_equal 'desc', user.pref[:comments_sorting] |
||
| 253 | assert ActionMailer::Base.deliveries.empty? |
||
| 254 | end |
||
| 255 | |||
| 256 | def test_update_with_failure |
||
| 257 | assert_no_difference 'User.count' do |
||
| 258 | put :update, :id => 2, :user => {:firstname => ''}
|
||
| 259 | end |
||
| 260 | assert_response :success |
||
| 261 | assert_template 'edit' |
||
| 262 | end |
||
| 263 | |||
| 264 | def test_update_with_group_ids_should_assign_groups |
||
| 265 | put :update, :id => 2, :user => {:group_ids => ['10']}
|
||
| 266 | user = User.find(2) |
||
| 267 | assert_equal [10], user.group_ids |
||
| 268 | end |
||
| 269 | |||
| 270 | def test_update_with_activation_should_send_a_notification |
||
| 271 | u = User.new(:firstname => 'Foo', :lastname => 'Bar', :mail => 'foo.bar@somenet.foo', :language => 'fr') |
||
| 272 | u.login = 'foo' |
||
| 273 | u.status = User::STATUS_REGISTERED |
||
| 274 | u.save! |
||
| 275 | ActionMailer::Base.deliveries.clear |
||
| 276 | Setting.bcc_recipients = '1' |
||
| 277 | |||
| 278 | put :update, :id => u.id, :user => {:status => User::STATUS_ACTIVE}
|
||
| 279 | assert u.reload.active? |
||
| 280 | mail = ActionMailer::Base.deliveries.last |
||
| 281 | assert_not_nil mail |
||
| 282 | assert_equal ['foo.bar@somenet.foo'], mail.bcc |
||
| 283 | assert_mail_body_match ll('fr', :notice_account_activated), mail
|
||
| 284 | end |
||
| 285 | |||
| 286 | def test_update_with_password_change_should_send_a_notification |
||
| 287 | ActionMailer::Base.deliveries.clear |
||
| 288 | Setting.bcc_recipients = '1' |
||
| 289 | |||
| 290 | put :update, :id => 2, :user => {:password => 'newpass123', :password_confirmation => 'newpass123'}, :send_information => '1'
|
||
| 291 | u = User.find(2) |
||
| 292 | assert u.check_password?('newpass123')
|
||
| 293 | |||
| 294 | mail = ActionMailer::Base.deliveries.last |
||
| 295 | assert_not_nil mail |
||
| 296 | assert_equal [u.mail], mail.bcc |
||
| 297 | assert_mail_body_match 'newpass123', mail |
||
| 298 | end |
||
| 299 | |||
| 300 | def test_update_user_switchin_from_auth_source_to_password_authentication |
||
| 301 | # Configure as auth source |
||
| 302 | u = User.find(2) |
||
| 303 | u.auth_source = AuthSource.find(1) |
||
| 304 | u.save! |
||
| 305 | |||
| 306 | put :update, :id => u.id, :user => {:auth_source_id => '', :password => 'newpass123', :password_confirmation => 'newpass123'}
|
||
| 307 | |||
| 308 | assert_equal nil, u.reload.auth_source |
||
| 309 | assert u.check_password?('newpass123')
|
||
| 310 | end |
||
| 311 | |||
| 312 | def test_update_notified_project |
||
| 313 | get :edit, :id => 2 |
||
| 314 | assert_response :success |
||
| 315 | assert_template 'edit' |
||
| 316 | u = User.find(2) |
||
| 317 | assert_equal [1, 2, 5], u.projects.collect{|p| p.id}.sort
|
||
| 318 | assert_equal [1, 2, 5], u.notified_projects_ids.sort |
||
| 319 | assert_tag :tag => 'input', |
||
| 320 | :attributes => {
|
||
| 321 | :id => 'notified_project_ids_', |
||
| 322 | :value => 1, |
||
| 323 | } |
||
| 324 | assert_equal 'all', u.mail_notification |
||
| 325 | put :update, :id => 2, |
||
| 326 | :user => {
|
||
| 327 | :mail_notification => 'selected', |
||
| 328 | }, |
||
| 329 | :notified_project_ids => [1, 2] |
||
| 330 | u = User.find(2) |
||
| 331 | assert_equal 'selected', u.mail_notification |
||
| 332 | assert_equal [1, 2], u.notified_projects_ids.sort |
||
| 333 | end |
||
| 334 | |||
| 335 | def test_destroy |
||
| 336 | assert_difference 'User.count', -1 do |
||
| 337 | delete :destroy, :id => 2 |
||
| 338 | end |
||
| 339 | assert_redirected_to '/users' |
||
| 340 | assert_nil User.find_by_id(2) |
||
| 341 | end |
||
| 342 | |||
| 343 | def test_destroy_should_be_denied_for_non_admin_users |
||
| 344 | @request.session[:user_id] = 3 |
||
| 345 | |||
| 346 | assert_no_difference 'User.count' do |
||
| 347 | get :destroy, :id => 2 |
||
| 348 | end |
||
| 349 | assert_response 403 |
||
| 350 | end |
||
| 351 | |||
| 352 | def test_destroy_should_redirect_to_back_url_param |
||
| 353 | assert_difference 'User.count', -1 do |
||
| 354 | delete :destroy, :id => 2, :back_url => '/users?name=foo' |
||
| 355 | end |
||
| 356 | assert_redirected_to '/users?name=foo' |
||
| 357 | end |
||
| 358 | |||
| 359 | def test_create_membership |
||
| 360 | assert_difference 'Member.count' do |
||
| 361 | post :edit_membership, :id => 7, :membership => { :project_id => 3, :role_ids => [2]}
|
||
| 362 | end |
||
| 363 | assert_redirected_to :action => 'edit', :id => '7', :tab => 'memberships' |
||
| 364 | member = Member.first(:order => 'id DESC') |
||
| 365 | assert_equal User.find(7), member.principal |
||
| 366 | assert_equal [2], member.role_ids |
||
| 367 | assert_equal 3, member.project_id |
||
| 368 | end |
||
| 369 | |||
| 370 | def test_create_membership_js_format |
||
| 371 | assert_difference 'Member.count' do |
||
| 372 | post :edit_membership, :id => 7, :membership => {:project_id => 3, :role_ids => [2]}, :format => 'js'
|
||
| 373 | assert_response :success |
||
| 374 | assert_template 'edit_membership' |
||
| 375 | assert_equal 'text/javascript', response.content_type |
||
| 376 | end |
||
| 377 | member = Member.first(:order => 'id DESC') |
||
| 378 | assert_equal User.find(7), member.principal |
||
| 379 | assert_equal [2], member.role_ids |
||
| 380 | assert_equal 3, member.project_id |
||
| 381 | assert_include 'tab-content-memberships', response.body |
||
| 382 | end |
||
| 383 | |||
| 384 | def test_create_membership_js_format_with_failure |
||
| 385 | assert_no_difference 'Member.count' do |
||
| 386 | post :edit_membership, :id => 7, :membership => {:project_id => 3}, :format => 'js'
|
||
| 387 | assert_response :success |
||
| 388 | assert_template 'edit_membership' |
||
| 389 | assert_equal 'text/javascript', response.content_type |
||
| 390 | end |
||
| 391 | assert_include 'alert', response.body, "Alert message not sent" |
||
| 392 | assert_include 'Role can\\\'t be empty', response.body, "Error message not sent" |
||
| 393 | end |
||
| 394 | |||
| 395 | def test_update_membership |
||
| 396 | assert_no_difference 'Member.count' do |
||
| 397 | put :edit_membership, :id => 2, :membership_id => 1, :membership => { :role_ids => [2]}
|
||
| 398 | assert_redirected_to :action => 'edit', :id => '2', :tab => 'memberships' |
||
| 399 | end |
||
| 400 | assert_equal [2], Member.find(1).role_ids |
||
| 401 | end |
||
| 402 | |||
| 403 | def test_update_membership_js_format |
||
| 404 | assert_no_difference 'Member.count' do |
||
| 405 | put :edit_membership, :id => 2, :membership_id => 1, :membership => {:role_ids => [2]}, :format => 'js'
|
||
| 406 | assert_response :success |
||
| 407 | assert_template 'edit_membership' |
||
| 408 | assert_equal 'text/javascript', response.content_type |
||
| 409 | end |
||
| 410 | assert_equal [2], Member.find(1).role_ids |
||
| 411 | assert_include 'tab-content-memberships', response.body |
||
| 412 | end |
||
| 413 | |||
| 414 | def test_destroy_membership |
||
| 415 | assert_difference 'Member.count', -1 do |
||
| 416 | delete :destroy_membership, :id => 2, :membership_id => 1 |
||
| 417 | end |
||
| 418 | assert_redirected_to :action => 'edit', :id => '2', :tab => 'memberships' |
||
| 419 | assert_nil Member.find_by_id(1) |
||
| 420 | end |
||
| 421 | |||
| 422 | def test_destroy_membership_js_format |
||
| 423 | assert_difference 'Member.count', -1 do |
||
| 424 | delete :destroy_membership, :id => 2, :membership_id => 1, :format => 'js' |
||
| 425 | assert_response :success |
||
| 426 | assert_template 'destroy_membership' |
||
| 427 | assert_equal 'text/javascript', response.content_type |
||
| 428 | end |
||
| 429 | assert_nil Member.find_by_id(1) |
||
| 430 | assert_include 'tab-content-memberships', response.body |
||
| 431 | end |
||
| 432 | end |