2017-04-22 04:23:17 +02:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'rails_helper'
|
|
|
|
|
|
|
|
describe Settings::TwoFactorAuthenticationsController do
|
|
|
|
render_views
|
|
|
|
|
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
|
|
|
|
describe 'GET #show' do
|
2017-05-29 18:07:07 +02:00
|
|
|
context 'when signed in' do
|
|
|
|
before do
|
|
|
|
sign_in user, scope: :user
|
|
|
|
end
|
2017-04-22 04:23:17 +02:00
|
|
|
|
2017-05-29 18:07:07 +02:00
|
|
|
describe 'when user requires otp for login already' do
|
|
|
|
it 'returns http success' do
|
|
|
|
user.update(otp_required_for_login: true)
|
|
|
|
get :show
|
|
|
|
|
2018-04-21 21:35:07 +02:00
|
|
|
expect(response).to have_http_status(200)
|
2017-05-29 18:07:07 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'when user does not require otp for login' do
|
|
|
|
it 'returns http success' do
|
|
|
|
user.update(otp_required_for_login: false)
|
|
|
|
get :show
|
|
|
|
|
2018-04-21 21:35:07 +02:00
|
|
|
expect(response).to have_http_status(200)
|
2017-05-29 18:07:07 +02:00
|
|
|
end
|
2017-04-22 04:23:17 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-05-29 18:07:07 +02:00
|
|
|
context 'when not signed in' do
|
|
|
|
it 'redirects' do
|
2017-04-22 04:23:17 +02:00
|
|
|
get :show
|
2017-05-29 18:07:07 +02:00
|
|
|
expect(response).to redirect_to '/auth/sign_in'
|
2017-04-22 04:23:17 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'POST #create' do
|
2017-05-29 18:07:07 +02:00
|
|
|
context 'when signed in' do
|
|
|
|
before do
|
|
|
|
sign_in user, scope: :user
|
|
|
|
end
|
2017-04-22 04:23:17 +02:00
|
|
|
|
2017-05-29 18:07:07 +02:00
|
|
|
describe 'when user requires otp for login already' do
|
|
|
|
it 'redirects to show page' do
|
|
|
|
user.update(otp_required_for_login: true)
|
|
|
|
post :create
|
|
|
|
|
|
|
|
expect(response).to redirect_to(settings_two_factor_authentication_path)
|
|
|
|
end
|
2017-04-22 04:23:17 +02:00
|
|
|
end
|
|
|
|
|
2017-05-29 18:07:07 +02:00
|
|
|
describe 'when creation succeeds' do
|
|
|
|
it 'updates user secret' do
|
|
|
|
before = user.otp_secret
|
|
|
|
post :create
|
2017-04-22 04:23:17 +02:00
|
|
|
|
2017-05-29 18:07:07 +02:00
|
|
|
expect(user.reload.otp_secret).not_to eq(before)
|
|
|
|
expect(response).to redirect_to(new_settings_two_factor_authentication_confirmation_path)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when not signed in' do
|
|
|
|
it 'redirects' do
|
|
|
|
get :show
|
|
|
|
expect(response).to redirect_to '/auth/sign_in'
|
2017-04-22 04:23:17 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'POST #destroy' do
|
|
|
|
before do
|
|
|
|
user.update(otp_required_for_login: true)
|
|
|
|
end
|
2017-05-29 18:07:07 +02:00
|
|
|
|
2017-06-25 23:51:46 +02:00
|
|
|
context 'when signed in' do
|
|
|
|
before do
|
|
|
|
sign_in user, scope: :user
|
|
|
|
end
|
2017-04-22 04:23:17 +02:00
|
|
|
|
2017-06-25 23:51:46 +02:00
|
|
|
it 'turns off otp requirement with correct code' do
|
|
|
|
expect_any_instance_of(User).to receive(:validate_and_consume_otp!) do |value, arg|
|
|
|
|
expect(value).to eq user
|
|
|
|
expect(arg).to eq '123456'
|
|
|
|
true
|
|
|
|
end
|
|
|
|
|
|
|
|
post :destroy, params: { form_two_factor_confirmation: { code: '123456' } }
|
|
|
|
|
|
|
|
expect(response).to redirect_to(settings_two_factor_authentication_path)
|
|
|
|
user.reload
|
|
|
|
expect(user.otp_required_for_login).to eq(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not turn off otp if code is incorrect' do
|
|
|
|
expect_any_instance_of(User).to receive(:validate_and_consume_otp!) do |value, arg|
|
|
|
|
expect(value).to eq user
|
|
|
|
expect(arg).to eq '057772'
|
|
|
|
false
|
|
|
|
end
|
|
|
|
|
|
|
|
post :destroy, params: { form_two_factor_confirmation: { code: '057772' } }
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.otp_required_for_login).to eq(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'raises ActionController::ParameterMissing if code is missing' do
|
2019-08-30 01:34:47 +02:00
|
|
|
post :destroy
|
|
|
|
expect(response).to have_http_status(400)
|
2017-06-25 23:51:46 +02:00
|
|
|
end
|
2017-04-22 04:23:17 +02:00
|
|
|
end
|
2017-05-29 18:07:07 +02:00
|
|
|
|
|
|
|
it 'redirects if not signed in' do
|
|
|
|
get :show
|
|
|
|
expect(response).to redirect_to '/auth/sign_in'
|
|
|
|
end
|
2017-04-22 04:23:17 +02:00
|
|
|
end
|
|
|
|
end
|