diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2022-03-12 04:14:25 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-03-12 04:14:25 +0100 |
commit | bc320d6cec5900fa2fd9d3a46480e242111caaca (patch) | |
tree | 2606d8af9cc2b7bb66f05ea54ffa4834ad8ebeee /spec | |
parent | 1745b71840036675332f356180aa2d69ad77b0ba (diff) |
Fix `POST /api/v1/emails/confirmations` not being available after sign-up (#17743)
Diffstat (limited to 'spec')
-rw-r--r-- | spec/controllers/api/v1/emails/confirmations_controller_spec.rb | 64 |
1 files changed, 64 insertions, 0 deletions
diff --git a/spec/controllers/api/v1/emails/confirmations_controller_spec.rb b/spec/controllers/api/v1/emails/confirmations_controller_spec.rb new file mode 100644 index 000000000..15ac31cbc --- /dev/null +++ b/spec/controllers/api/v1/emails/confirmations_controller_spec.rb @@ -0,0 +1,64 @@ +require 'rails_helper' + +RSpec.describe Api::V1::Emails::ConfirmationsController, type: :controller do + let(:confirmed_at) { nil } + let(:user) { Fabricate(:user, confirmed_at: confirmed_at) } + let(:app) { Fabricate(:application) } + let(:token) { Fabricate(:accessible_access_token, resource_owner_id: user.id, scopes: scopes, application: app) } + let(:scopes) { 'write' } + + describe '#create' do + context 'with an oauth token' do + before do + allow(controller).to receive(:doorkeeper_token) { token } + end + + context 'from a random app' do + it 'returns http forbidden' do + post :create + expect(response).to have_http_status(:forbidden) + end + end + + context 'from an app that created the account' do + before do + user.update(created_by_application: token.application) + end + + context 'when the account is already confirmed' do + let(:confirmed_at) { Time.now.utc } + + it 'returns http forbidden' do + post :create + expect(response).to have_http_status(:forbidden) + end + + context 'but user changed e-mail and has not confirmed it' do + before do + user.update(email: 'foo@bar.com') + end + + it 'returns http success' do + post :create + expect(response).to have_http_status(:success) + end + end + end + + context 'when the account is unconfirmed' do + it 'returns http success' do + post :create + expect(response).to have_http_status(:success) + end + end + end + end + + context 'without an oauth token' do + it 'returns http unauthorized' do + post :create + expect(response).to have_http_status(:unauthorized) + end + end + end +end |