Moved tests about session reset to functional tests.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@9014 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
parent
65b4d8a649
commit
39aa4cac86
|
@ -61,6 +61,13 @@ class AccountControllerTest < ActionController::TestCase
|
||||||
assert_error_tag :content => /Something wrong/
|
assert_error_tag :content => /Something wrong/
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def test_login_should_reset_session
|
||||||
|
@controller.expects(:reset_session).once
|
||||||
|
|
||||||
|
post :login, :username => 'jsmith', :password => 'jsmith'
|
||||||
|
assert_response 302
|
||||||
|
end
|
||||||
|
|
||||||
if Object.const_defined?(:OpenID)
|
if Object.const_defined?(:OpenID)
|
||||||
|
|
||||||
def test_login_with_openid_for_existing_user
|
def test_login_with_openid_for_existing_user
|
||||||
|
@ -171,6 +178,14 @@ class AccountControllerTest < ActionController::TestCase
|
||||||
assert_nil @request.session[:user_id]
|
assert_nil @request.session[:user_id]
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def test_logout_should_reset_session
|
||||||
|
@controller.expects(:reset_session).once
|
||||||
|
|
||||||
|
@request.session[:user_id] = 2
|
||||||
|
get :logout
|
||||||
|
assert_response 302
|
||||||
|
end
|
||||||
|
|
||||||
def test_get_register_with_registration_on
|
def test_get_register_with_registration_on
|
||||||
with_settings :self_registration => '3' do
|
with_settings :self_registration => '3' do
|
||||||
get :register
|
get :register
|
||||||
|
|
|
@ -179,22 +179,4 @@ class AccountTest < ActionController::IntegrationTest
|
||||||
assert_equal 66, user.auth_source_id
|
assert_equal 66, user.auth_source_id
|
||||||
assert user.hashed_password.blank?
|
assert user.hashed_password.blank?
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_login_and_logout_should_clear_session
|
|
||||||
get '/login'
|
|
||||||
sid = session[:session_id]
|
|
||||||
|
|
||||||
post '/login', :username => 'admin', :password => 'admin'
|
|
||||||
assert_redirected_to '/my/page'
|
|
||||||
assert_not_equal sid, session[:session_id], "login should reset session"
|
|
||||||
assert_equal 1, session[:user_id]
|
|
||||||
sid = session[:session_id]
|
|
||||||
|
|
||||||
get '/'
|
|
||||||
assert_equal sid, session[:session_id]
|
|
||||||
|
|
||||||
get '/logout'
|
|
||||||
assert_not_equal sid, session[:session_id], "logout should reset session"
|
|
||||||
assert_nil session[:user_id]
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in New Issue