Chore: Increase test coverage for accounts controller (#475)
This commit is contained in:
parent
5210d201b6
commit
a30a8867a3
1 changed files with 17 additions and 1 deletions
|
@ -9,10 +9,11 @@ RSpec.describe 'Accounts API', type: :request do
|
||||||
|
|
||||||
before do
|
before do
|
||||||
allow(AccountBuilder).to receive(:new).and_return(account_builder)
|
allow(AccountBuilder).to receive(:new).and_return(account_builder)
|
||||||
allow(account_builder).to receive(:perform).and_return(user)
|
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'calls account builder' do
|
it 'calls account builder' do
|
||||||
|
allow(account_builder).to receive(:perform).and_return(user)
|
||||||
|
|
||||||
params = { account_name: 'test', email: email }
|
params = { account_name: 'test', email: email }
|
||||||
|
|
||||||
post api_v1_accounts_url,
|
post api_v1_accounts_url,
|
||||||
|
@ -23,6 +24,21 @@ RSpec.describe 'Accounts API', type: :request do
|
||||||
expect(account_builder).to have_received(:perform)
|
expect(account_builder).to have_received(:perform)
|
||||||
expect(response.headers.keys).to include('access-token', 'token-type', 'client', 'expiry', 'uid')
|
expect(response.headers.keys).to include('access-token', 'token-type', 'client', 'expiry', 'uid')
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it 'renders error response on invalid params' do
|
||||||
|
allow(account_builder).to receive(:perform).and_return(nil)
|
||||||
|
|
||||||
|
params = { account_name: nil, email: nil }
|
||||||
|
|
||||||
|
post api_v1_accounts_url,
|
||||||
|
params: params,
|
||||||
|
as: :json
|
||||||
|
|
||||||
|
expect(AccountBuilder).to have_received(:new).with(params)
|
||||||
|
expect(account_builder).to have_received(:perform)
|
||||||
|
expect(response).to have_http_status(:forbidden)
|
||||||
|
expect(response.body).to eq({ message: I18n.t('errors.signup.failed') }.to_json)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in a new issue