users_controller_test.rb
391 lines
| 11.7 KiB
| text/x-ruby
|
RubyLexer
|
r5030 | # Redmine - project management software | ||
# Copyright (C) 2006-2011 Jean-Philippe Lang | ||||
|
r974 | # | ||
# This program is free software; you can redistribute it and/or | ||||
# modify it under the terms of the GNU General Public License | ||||
# as published by the Free Software Foundation; either version 2 | ||||
# of the License, or (at your option) any later version. | ||||
|
r6508 | # | ||
|
r974 | # This program is distributed in the hope that it will be useful, | ||
# but WITHOUT ANY WARRANTY; without even the implied warranty of | ||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||||
# GNU General Public License for more details. | ||||
|
r6508 | # | ||
|
r974 | # You should have received a copy of the GNU General Public License | ||
# along with this program; if not, write to the Free Software | ||||
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. | ||||
|
r4395 | require File.expand_path('../../test_helper', __FILE__) | ||
|
r974 | require 'users_controller' | ||
# Re-raise errors caught by the controller. | ||||
class UsersController; def rescue_action(e) raise e end; end | ||||
|
r2773 | class UsersControllerTest < ActionController::TestCase | ||
|
r2431 | include Redmine::I18n | ||
|
r6508 | |||
|
r8832 | fixtures :users, :projects, :members, :member_roles, :roles, :custom_fields, :custom_values, :groups_users, :auth_sources | ||
|
r6508 | |||
|
r974 | def setup | ||
@controller = UsersController.new | ||||
@request = ActionController::TestRequest.new | ||||
@response = ActionController::TestResponse.new | ||||
User.current = nil | ||||
@request.session[:user_id] = 1 # admin | ||||
end | ||||
|
r6508 | |||
|
r974 | def test_index | ||
get :index | ||||
assert_response :success | ||||
|
r2877 | assert_template 'index' | ||
|
r2315 | end | ||
|
r974 | |||
|
r2877 | def test_index | ||
get :index | ||||
|
r974 | assert_response :success | ||
|
r2877 | assert_template 'index' | ||
|
r974 | assert_not_nil assigns(:users) | ||
# active users only | ||||
assert_nil assigns(:users).detect {|u| !u.active?} | ||||
end | ||||
|
r6508 | |||
|
r7961 | def test_index_with_status_filter | ||
get :index, :status => 3 | ||||
assert_response :success | ||||
assert_template 'index' | ||||
assert_not_nil assigns(:users) | ||||
assert_equal [3], assigns(:users).map(&:status).uniq | ||||
end | ||||
|
r2877 | def test_index_with_name_filter | ||
get :index, :name => 'john' | ||||
|
r1944 | assert_response :success | ||
|
r2877 | assert_template 'index' | ||
|
r1944 | users = assigns(:users) | ||
assert_not_nil users | ||||
assert_equal 1, users.size | ||||
assert_equal 'John', users.first.firstname | ||||
end | ||||
|
r6508 | |||
|
r5030 | def test_index_with_group_filter | ||
get :index, :group_id => '10' | ||||
assert_response :success | ||||
assert_template 'index' | ||||
users = assigns(:users) | ||||
assert users.any? | ||||
assert_equal([], (users - Group.find(10).users)) | ||||
end | ||||
|
r6508 | |||
|
r2874 | def test_show | ||
@request.session[:user_id] = nil | ||||
get :show, :id => 2 | ||||
assert_response :success | ||||
assert_template 'show' | ||||
assert_not_nil assigns(:user) | ||||
|
r6508 | |||
|
r4268 | assert_tag 'li', :content => /Phone number/ | ||
end | ||||
|
r6508 | |||
|
r4268 | def test_show_should_not_display_hidden_custom_fields | ||
@request.session[:user_id] = nil | ||||
UserCustomField.find_by_name('Phone number').update_attribute :visible, false | ||||
get :show, :id => 2 | ||||
assert_response :success | ||||
assert_template 'show' | ||||
assert_not_nil assigns(:user) | ||||
|
r6508 | |||
|
r4268 | assert_no_tag 'li', :content => /Phone number/ | ||
|
r2874 | end | ||
def test_show_should_not_fail_when_custom_values_are_nil | ||||
user = User.find(2) | ||||
# Create a custom field to illustrate the issue | ||||
custom_field = CustomField.create!(:name => 'Testing', :field_format => 'text') | ||||
custom_value = user.custom_values.build(:custom_field => custom_field).save! | ||||
get :show, :id => 2 | ||||
assert_response :success | ||||
end | ||||
def test_show_inactive | ||||
|
r3379 | @request.session[:user_id] = nil | ||
|
r2874 | get :show, :id => 5 | ||
assert_response 404 | ||||
end | ||||
|
r6508 | |||
|
r2874 | def test_show_should_not_reveal_users_with_no_visible_activity_or_project | ||
@request.session[:user_id] = nil | ||||
get :show, :id => 9 | ||||
assert_response 404 | ||||
end | ||||
|
r6508 | |||
|
r3379 | def test_show_inactive_by_admin | ||
@request.session[:user_id] = 1 | ||||
get :show, :id => 5 | ||||
assert_response 200 | ||||
assert_not_nil assigns(:user) | ||||
end | ||||
|
r6508 | |||
|
r3821 | def test_show_displays_memberships_based_on_project_visibility | ||
@request.session[:user_id] = 1 | ||||
get :show, :id => 2 | ||||
assert_response :success | ||||
memberships = assigns(:memberships) | ||||
assert_not_nil memberships | ||||
project_ids = memberships.map(&:project_id) | ||||
assert project_ids.include?(2) #private project admin can see | ||||
end | ||||
|
r6508 | |||
|
r4430 | def test_show_current_should_require_authentication | ||
@request.session[:user_id] = nil | ||||
get :show, :id => 'current' | ||||
assert_response 302 | ||||
end | ||||
|
r6508 | |||
|
r4430 | def test_show_current | ||
@request.session[:user_id] = 2 | ||||
get :show, :id => 'current' | ||||
assert_response :success | ||||
assert_template 'show' | ||||
assert_equal User.find(2), assigns(:user) | ||||
end | ||||
|
r6508 | |||
|
r4387 | def test_new | ||
get :new | ||||
|
r6508 | |||
|
r4387 | assert_response :success | ||
assert_template :new | ||||
assert assigns(:user) | ||||
|
r4386 | end | ||
|
r6508 | |||
|
r4386 | def test_create | ||
Setting.bcc_recipients = '1' | ||||
|
r6508 | |||
|
r4386 | assert_difference 'User.count' do | ||
assert_difference 'ActionMailer::Base.deliveries.size' do | ||||
post :create, | ||||
:user => { | ||||
:firstname => 'John', | ||||
:lastname => 'Doe', | ||||
:login => 'jdoe', | ||||
:password => 'secret', | ||||
:password_confirmation => 'secret', | ||||
:mail => 'jdoe@gmail.com', | ||||
:mail_notification => 'none' | ||||
}, | ||||
:send_information => '1' | ||||
end | ||||
end | ||||
|
r6508 | |||
|
r4386 | user = User.first(:order => 'id DESC') | ||
assert_redirected_to :controller => 'users', :action => 'edit', :id => user.id | ||||
|
r6508 | |||
|
r4386 | assert_equal 'John', user.firstname | ||
assert_equal 'Doe', user.lastname | ||||
assert_equal 'jdoe', user.login | ||||
assert_equal 'jdoe@gmail.com', user.mail | ||||
assert_equal 'none', user.mail_notification | ||||
assert user.check_password?('secret') | ||||
|
r6508 | |||
|
r4386 | mail = ActionMailer::Base.deliveries.last | ||
assert_not_nil mail | ||||
assert_equal [user.mail], mail.bcc | ||||
|
r8966 | assert_mail_body_match 'secret', mail | ||
|
r4101 | end | ||
|
r6508 | |||
|
r8947 | def test_create_with_preferences | ||
assert_difference 'User.count' do | ||||
post :create, | ||||
:user => { | ||||
:firstname => 'John', | ||||
:lastname => 'Doe', | ||||
:login => 'jdoe', | ||||
:password => 'secret', | ||||
:password_confirmation => 'secret', | ||||
:mail => 'jdoe@gmail.com', | ||||
:mail_notification => 'none' | ||||
}, | ||||
:pref => { | ||||
'hide_mail' => '1', | ||||
'time_zone' => 'Paris', | ||||
'comments_sorting' => 'desc', | ||||
'warn_on_leaving_unsaved' => '0' | ||||
} | ||||
end | ||||
user = User.first(:order => 'id DESC') | ||||
assert_equal 'jdoe', user.login | ||||
assert_equal true, user.pref.hide_mail | ||||
assert_equal 'Paris', user.pref.time_zone | ||||
assert_equal 'desc', user.pref[:comments_sorting] | ||||
assert_equal '0', user.pref[:warn_on_leaving_unsaved] | ||||
end | ||||
|
r4387 | def test_create_with_failure | ||
assert_no_difference 'User.count' do | ||||
post :create, :user => {} | ||||
end | ||||
|
r6508 | |||
|
r4387 | assert_response :success | ||
assert_template 'new' | ||||
end | ||||
def test_edit | ||||
get :edit, :id => 2 | ||||
|
r6508 | |||
|
r4387 | assert_response :success | ||
assert_template 'edit' | ||||
assert_equal User.find(2), assigns(:user) | ||||
end | ||||
|
r4101 | |||
|
r4116 | def test_update | ||
|
r2708 | ActionMailer::Base.deliveries.clear | ||
|
r4382 | put :update, :id => 2, :user => {:firstname => 'Changed', :mail_notification => 'only_assigned'}, :pref => {:hide_mail => '1', :comments_sorting => 'desc'} | ||
|
r4109 | |||
user = User.find(2) | ||||
assert_equal 'Changed', user.firstname | ||||
|
r4382 | assert_equal 'only_assigned', user.mail_notification | ||
|
r4109 | assert_equal true, user.pref[:hide_mail] | ||
assert_equal 'desc', user.pref[:comments_sorting] | ||||
|
r2708 | assert ActionMailer::Base.deliveries.empty? | ||
end | ||||
|
r4387 | |||
def test_update_with_failure | ||||
assert_no_difference 'User.count' do | ||||
put :update, :id => 2, :user => {:firstname => ''} | ||||
end | ||||
|
r6508 | |||
|
r4387 | assert_response :success | ||
assert_template 'edit' | ||||
end | ||||
|
r6508 | |||
|
r4385 | def test_update_with_group_ids_should_assign_groups | ||
put :update, :id => 2, :user => {:group_ids => ['10']} | ||||
|
r6508 | |||
|
r4385 | user = User.find(2) | ||
assert_equal [10], user.group_ids | ||||
end | ||||
|
r6508 | |||
|
r4116 | def test_update_with_activation_should_send_a_notification | ||
|
r2708 | u = User.new(:firstname => 'Foo', :lastname => 'Bar', :mail => 'foo.bar@somenet.foo', :language => 'fr') | ||
u.login = 'foo' | ||||
u.status = User::STATUS_REGISTERED | ||||
u.save! | ||||
ActionMailer::Base.deliveries.clear | ||||
Setting.bcc_recipients = '1' | ||||
|
r6508 | |||
|
r4116 | put :update, :id => u.id, :user => {:status => User::STATUS_ACTIVE} | ||
|
r2708 | assert u.reload.active? | ||
mail = ActionMailer::Base.deliveries.last | ||||
assert_not_nil mail | ||||
assert_equal ['foo.bar@somenet.foo'], mail.bcc | ||||
|
r8966 | assert_mail_body_match ll('fr', :notice_account_activated), mail | ||
|
r2708 | end | ||
|
r6508 | |||
|
r4379 | def test_update_with_password_change_should_send_a_notification | ||
|
r2708 | ActionMailer::Base.deliveries.clear | ||
Setting.bcc_recipients = '1' | ||||
|
r6508 | |||
|
r4379 | put :update, :id => 2, :user => {:password => 'newpass', :password_confirmation => 'newpass'}, :send_information => '1' | ||
|
r2708 | u = User.find(2) | ||
|
r4379 | assert u.check_password?('newpass') | ||
|
r6508 | |||
|
r2708 | mail = ActionMailer::Base.deliveries.last | ||
assert_not_nil mail | ||||
assert_equal [u.mail], mail.bcc | ||||
|
r8966 | assert_mail_body_match 'newpass', mail | ||
|
r2708 | end | ||
|
r3952 | |||
|
r9111 | def test_update_user_switchin_from_auth_source_to_password_authentication | ||
|
r3952 | # Configure as auth source | ||
u = User.find(2) | ||||
u.auth_source = AuthSource.find(1) | ||||
u.save! | ||||
|
r9111 | put :update, :id => u.id, :user => {:auth_source_id => '', :password => 'newpass', :password_confirmation => 'newpass'} | ||
|
r3952 | |||
assert_equal nil, u.reload.auth_source | ||||
|
r4379 | assert u.check_password?('newpass') | ||
|
r3952 | end | ||
|
r6508 | |||
|
r4609 | def test_destroy | ||
assert_difference 'User.count', -1 do | ||||
delete :destroy, :id => 2 | ||||
end | ||||
assert_redirected_to '/users' | ||||
assert_nil User.find_by_id(2) | ||||
end | ||||
def test_destroy_should_be_denied_for_non_admin_users | ||||
@request.session[:user_id] = 3 | ||||
|
r6508 | |||
|
r4609 | assert_no_difference 'User.count' do | ||
get :destroy, :id => 2 | ||||
end | ||||
assert_response 403 | ||||
end | ||||
|
r6508 | |||
|
r8026 | def test_create_membership | ||
assert_difference 'Member.count' do | ||||
post :edit_membership, :id => 7, :membership => { :project_id => 3, :role_ids => [2]} | ||||
end | ||||
assert_redirected_to :action => 'edit', :id => '7', :tab => 'memberships' | ||||
member = Member.first(:order => 'id DESC') | ||||
assert_equal User.find(7), member.principal | ||||
assert_equal [2], member.role_ids | ||||
assert_equal 3, member.project_id | ||||
end | ||||
|
r9111 | def test_create_membership_js_format | ||
assert_difference 'Member.count' do | ||||
post :edit_membership, :id => 7, :membership => {:project_id => 3, :role_ids => [2]}, :format => 'js' | ||||
end | ||||
assert_response :success | ||||
assert_select_rjs :replace_html, 'tab-content-memberships' | ||||
member = Member.first(:order => 'id DESC') | ||||
assert_equal User.find(7), member.principal | ||||
assert_equal [2], member.role_ids | ||||
assert_equal 3, member.project_id | ||||
end | ||||
def test_create_membership_js_format_with_failure | ||||
assert_no_difference 'Member.count' do | ||||
post :edit_membership, :id => 7, :membership => {:project_id => 3}, :format => 'js' | ||||
end | ||||
assert_response :success | ||||
assert @response.body.match(/alert/i), "Alert message not sent" | ||||
assert @response.body.match(/role can't be empty/i), "Error message not sent" | ||||
end | ||||
|
r8026 | def test_update_membership | ||
assert_no_difference 'Member.count' do | ||||
put :edit_membership, :id => 2, :membership_id => 1, :membership => { :role_ids => [2]} | ||||
end | ||||
|
r2315 | assert_redirected_to :action => 'edit', :id => '2', :tab => 'memberships' | ||
|
r2627 | assert_equal [2], Member.find(1).role_ids | ||
|
r974 | end | ||
|
r6508 | |||
|
r9111 | def test_update_membership_js_format | ||
assert_no_difference 'Member.count' do | ||||
put :edit_membership, :id => 2, :membership_id => 1, :membership => {:role_ids => [2]}, :format => 'js' | ||||
end | ||||
assert_response :success | ||||
assert_select_rjs :replace_html, 'tab-content-memberships' | ||||
end | ||||
|
r974 | def test_destroy_membership | ||
|
r8026 | assert_difference 'Member.count', -1 do | ||
delete :destroy_membership, :id => 2, :membership_id => 1 | ||||
end | ||||
|
r2315 | assert_redirected_to :action => 'edit', :id => '2', :tab => 'memberships' | ||
|
r974 | assert_nil Member.find_by_id(1) | ||
end | ||||
|
r9111 | |||
def test_destroy_membership_js_format | ||||
assert_difference 'Member.count', -1 do | ||||
delete :destroy_membership, :id => 2, :membership_id => 1, :format => 'js' | ||||
end | ||||
assert_response :success | ||||
assert_select_rjs :replace_html, 'tab-content-memberships' | ||||
end | ||||
|
r974 | end | ||