370 lines
12 KiB
Ruby
370 lines
12 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'rails_helper'
|
|
|
|
RSpec.describe Auth::RegistrationsController do
|
|
render_views
|
|
|
|
shared_examples 'checks for enabled registrations' do |path|
|
|
it 'redirects if it is in single user mode while it is open for registration' do
|
|
Fabricate(:account)
|
|
Setting.registrations_mode = 'open'
|
|
allow(Rails.configuration.x).to receive(:single_user_mode).and_return(true)
|
|
|
|
get path
|
|
|
|
expect(response).to redirect_to '/'
|
|
expect(Rails.configuration.x).to have_received(:single_user_mode)
|
|
end
|
|
|
|
it 'redirects if it is not open for registration while it is not in single user mode' do
|
|
Setting.registrations_mode = 'none'
|
|
allow(Rails.configuration.x).to receive(:single_user_mode).and_return(false)
|
|
|
|
get path
|
|
|
|
expect(response).to redirect_to '/'
|
|
expect(Rails.configuration.x).to have_received(:single_user_mode)
|
|
end
|
|
end
|
|
|
|
describe 'GET #edit' do
|
|
before do
|
|
request.env['devise.mapping'] = Devise.mappings[:user]
|
|
sign_in(Fabricate(:user))
|
|
get :edit
|
|
end
|
|
|
|
it 'returns http success' do
|
|
expect(response).to have_http_status(200)
|
|
end
|
|
|
|
it 'returns private cache control header' do
|
|
expect(response.headers['Cache-Control']).to include('private, no-store')
|
|
end
|
|
end
|
|
|
|
describe 'PUT #update' do
|
|
let(:current_password) { 'current password' }
|
|
let(:user) { Fabricate(:user, password: current_password) }
|
|
|
|
before do
|
|
request.env['devise.mapping'] = Devise.mappings[:user]
|
|
sign_in(user, scope: :user)
|
|
end
|
|
|
|
it 'returns http success' do
|
|
put :update
|
|
expect(response).to have_http_status(200)
|
|
end
|
|
|
|
it 'returns private cache control headers' do
|
|
put :update
|
|
expect(response.headers['Cache-Control']).to include('private, no-store')
|
|
end
|
|
|
|
it 'can update the user email' do
|
|
expect do
|
|
put :update, params: {
|
|
user: {
|
|
email: 'newemail@example.com',
|
|
current_password: current_password,
|
|
},
|
|
}
|
|
expect(response).to redirect_to(edit_user_registration_path)
|
|
end.to change { user.reload.unconfirmed_email }.to('newemail@example.com')
|
|
end
|
|
|
|
it 'requires the current password to update the email' do
|
|
expect do
|
|
put :update, params: {
|
|
user: {
|
|
email: 'newemail@example.com',
|
|
current_password: 'something',
|
|
},
|
|
}
|
|
expect(response).to have_http_status(200)
|
|
end.to_not(change { user.reload.unconfirmed_email })
|
|
end
|
|
|
|
it 'can update the user password' do
|
|
expect do
|
|
put :update, params: {
|
|
user: {
|
|
password: 'new password',
|
|
password_confirmation: 'new password',
|
|
current_password: current_password,
|
|
},
|
|
}
|
|
expect(response).to redirect_to(edit_user_registration_path)
|
|
end.to(change { user.reload.encrypted_password })
|
|
end
|
|
|
|
it 'requires the password confirmation' do
|
|
expect do
|
|
put :update, params: {
|
|
user: {
|
|
password: 'new password',
|
|
password_confirmation: 'something else',
|
|
current_password: current_password,
|
|
},
|
|
}
|
|
expect(response).to have_http_status(200)
|
|
end.to_not(change { user.reload.encrypted_password })
|
|
end
|
|
|
|
it 'requires the current password to update the password' do
|
|
expect do
|
|
put :update, params: {
|
|
user: {
|
|
password: 'new password',
|
|
password_confirmation: 'new password',
|
|
current_password: 'something',
|
|
},
|
|
}
|
|
expect(response).to have_http_status(200)
|
|
end.to_not(change { user.reload.encrypted_password })
|
|
end
|
|
|
|
context 'when suspended' do
|
|
let(:user) { Fabricate(:user, account_attributes: { username: 'test', suspended_at: Time.now.utc }) }
|
|
|
|
it 'returns http forbidden' do
|
|
put :update
|
|
expect(response).to have_http_status(403)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'GET #new' do
|
|
before do
|
|
request.env['devise.mapping'] = Devise.mappings[:user]
|
|
end
|
|
|
|
context 'with open registrations' do
|
|
it 'returns http success' do
|
|
Setting.registrations_mode = 'open'
|
|
get :new
|
|
expect(response).to have_http_status(200)
|
|
end
|
|
end
|
|
|
|
include_examples 'checks for enabled registrations', :new
|
|
end
|
|
|
|
describe 'POST #create' do
|
|
let(:accept_language) { 'de' }
|
|
|
|
before do
|
|
session[:registration_form_time] = 5.seconds.ago
|
|
|
|
request.env['devise.mapping'] = Devise.mappings[:user]
|
|
end
|
|
|
|
around do |example|
|
|
I18n.with_locale(I18n.locale) do
|
|
example.run
|
|
end
|
|
end
|
|
|
|
context 'when an accept language is present in headers' do
|
|
subject do
|
|
Setting.registrations_mode = 'open'
|
|
request.headers['Accept-Language'] = accept_language
|
|
post :create, params: { user: { account_attributes: { username: 'test' }, email: 'test@example.com', password: '12345678', password_confirmation: '12345678', agreement: 'true' } }
|
|
end
|
|
|
|
it 'redirects to setup' do
|
|
subject
|
|
expect(response).to redirect_to auth_setup_path
|
|
end
|
|
|
|
it 'creates user' do
|
|
subject
|
|
user = User.find_by(email: 'test@example.com')
|
|
expect(user).to_not be_nil
|
|
expect(user.locale).to eq(accept_language)
|
|
end
|
|
end
|
|
|
|
context 'when user has not agreed to terms of service' do
|
|
subject do
|
|
Setting.registrations_mode = 'open'
|
|
request.headers['Accept-Language'] = accept_language
|
|
post :create, params: { user: { account_attributes: { username: 'test' }, email: 'test@example.com', password: '12345678', password_confirmation: '12345678', agreement: 'false' } }
|
|
end
|
|
|
|
it 'does not create user' do
|
|
subject
|
|
user = User.find_by(email: 'test@example.com')
|
|
expect(user).to be_nil
|
|
end
|
|
end
|
|
|
|
context 'when user has an email address requiring approval' do
|
|
subject do
|
|
request.headers['Accept-Language'] = accept_language
|
|
post :create, params: { user: { account_attributes: { username: 'test' }, email: 'test@example.com', password: '12345678', password_confirmation: '12345678', agreement: 'true' } }
|
|
end
|
|
|
|
before do
|
|
Setting.registrations_mode = 'open'
|
|
Fabricate(:email_domain_block, allow_with_approval: true, domain: 'example.com')
|
|
end
|
|
|
|
it 'creates unapproved user and redirects to setup' do
|
|
subject
|
|
expect(response).to redirect_to auth_setup_path
|
|
|
|
user = User.find_by(email: 'test@example.com')
|
|
expect(user).to_not be_nil
|
|
expect(user.locale).to eq(accept_language)
|
|
expect(user.approved).to be(false)
|
|
end
|
|
end
|
|
|
|
context 'when user has an email address requiring approval through a MX record' do
|
|
subject do
|
|
request.headers['Accept-Language'] = accept_language
|
|
post :create, params: { user: { account_attributes: { username: 'test' }, email: 'test@example.com', password: '12345678', password_confirmation: '12345678', agreement: 'true' } }
|
|
end
|
|
|
|
before do
|
|
Setting.registrations_mode = 'open'
|
|
Fabricate(:email_domain_block, allow_with_approval: true, domain: 'mail.example.com')
|
|
allow(User).to receive(:skip_mx_check?).and_return(false)
|
|
|
|
resolver = instance_double(Resolv::DNS, :timeouts= => nil)
|
|
|
|
allow(resolver).to receive(:getresources)
|
|
.with('example.com', Resolv::DNS::Resource::IN::MX)
|
|
.and_return([instance_double(Resolv::DNS::Resource::MX, exchange: 'mail.example.com')])
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::A).and_return([])
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::AAAA).and_return([])
|
|
allow(resolver).to receive(:getresources).with('mail.example.com', Resolv::DNS::Resource::IN::A).and_return([instance_double(Resolv::DNS::Resource::IN::A, address: '2.3.4.5')])
|
|
allow(resolver).to receive(:getresources).with('mail.example.com', Resolv::DNS::Resource::IN::AAAA).and_return([instance_double(Resolv::DNS::Resource::IN::AAAA, address: 'fd00::2')])
|
|
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
|
|
end
|
|
|
|
it 'creates unapproved user and redirects to setup' do
|
|
subject
|
|
expect(response).to redirect_to auth_setup_path
|
|
|
|
user = User.find_by(email: 'test@example.com')
|
|
expect(user).to_not be_nil
|
|
expect(user.locale).to eq(accept_language)
|
|
expect(user.approved).to be(false)
|
|
end
|
|
end
|
|
|
|
context 'with Approval-based registrations without invite' do
|
|
subject do
|
|
Setting.registrations_mode = 'approved'
|
|
request.headers['Accept-Language'] = accept_language
|
|
post :create, params: { user: { account_attributes: { username: 'test' }, email: 'test@example.com', password: '12345678', password_confirmation: '12345678', agreement: 'true' } }
|
|
end
|
|
|
|
it 'redirects to setup' do
|
|
subject
|
|
expect(response).to redirect_to auth_setup_path
|
|
end
|
|
|
|
it 'creates user' do
|
|
subject
|
|
user = User.find_by(email: 'test@example.com')
|
|
expect(user).to_not be_nil
|
|
expect(user.locale).to eq(accept_language)
|
|
expect(user.approved).to be(false)
|
|
end
|
|
end
|
|
|
|
context 'with Approval-based registrations with expired invite' do
|
|
subject do
|
|
Setting.registrations_mode = 'approved'
|
|
request.headers['Accept-Language'] = accept_language
|
|
invite = Fabricate(:invite, max_uses: nil, expires_at: 1.hour.ago)
|
|
post :create, params: { user: { account_attributes: { username: 'test' }, email: 'test@example.com', password: '12345678', password_confirmation: '12345678', invite_code: invite.code, agreement: 'true' } }
|
|
end
|
|
|
|
it 'redirects to setup' do
|
|
subject
|
|
expect(response).to redirect_to auth_setup_path
|
|
end
|
|
|
|
it 'creates user' do
|
|
subject
|
|
user = User.find_by(email: 'test@example.com')
|
|
expect(user).to_not be_nil
|
|
expect(user.locale).to eq(accept_language)
|
|
expect(user.approved).to be(false)
|
|
end
|
|
end
|
|
|
|
context 'with Approval-based registrations with valid invite and required invite text' do
|
|
subject do
|
|
inviter = Fabricate(:user, confirmed_at: 2.days.ago)
|
|
Setting.registrations_mode = 'approved'
|
|
Setting.require_invite_text = true
|
|
request.headers['Accept-Language'] = accept_language
|
|
invite = Fabricate(:invite, user: inviter, max_uses: nil, expires_at: 1.hour.from_now)
|
|
post :create, params: { user: { account_attributes: { username: 'test' }, email: 'test@example.com', password: '12345678', password_confirmation: '12345678', invite_code: invite.code, agreement: 'true' } }
|
|
end
|
|
|
|
it 'redirects to setup' do
|
|
subject
|
|
expect(response).to redirect_to auth_setup_path
|
|
end
|
|
|
|
it 'creates user' do
|
|
subject
|
|
user = User.find_by(email: 'test@example.com')
|
|
expect(user).to_not be_nil
|
|
expect(user.locale).to eq(accept_language)
|
|
expect(user.approved).to be(true)
|
|
end
|
|
end
|
|
|
|
context 'with an already taken username' do
|
|
subject do
|
|
Setting.registrations_mode = 'open'
|
|
post :create, params: { user: { account_attributes: { username: 'test' }, email: 'test@example.com', password: '12345678', password_confirmation: '12345678', agreement: 'true' } }
|
|
end
|
|
|
|
before do
|
|
Fabricate(:account, username: 'test')
|
|
end
|
|
|
|
it 'responds with an error message about the username' do
|
|
subject
|
|
|
|
expect(response).to have_http_status(:success)
|
|
expect(username_error_text).to eq(I18n.t('errors.messages.taken'))
|
|
end
|
|
|
|
def username_error_text
|
|
Nokogiri::Slop(response.body).css('.user_account_username .error').text
|
|
end
|
|
end
|
|
|
|
include_examples 'checks for enabled registrations', :create
|
|
end
|
|
|
|
describe 'DELETE #destroy' do
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
before do
|
|
request.env['devise.mapping'] = Devise.mappings[:user]
|
|
sign_in(user, scope: :user)
|
|
delete :destroy
|
|
end
|
|
|
|
it 'returns http not found' do
|
|
expect(response).to have_http_status(404)
|
|
end
|
|
|
|
it 'does not delete user' do
|
|
expect(User.find(user.id)).to_not be_nil
|
|
end
|
|
end
|
|
end
|