From 459394a02021ae927d135669e7472a4c7c32eaec Mon Sep 17 00:00:00 2001 From: Shuhei Kitagawa Date: Thu, 21 Jun 2018 10:40:23 +0900 Subject: Add missing tests for confirmations controller (#7866) --- .../auth/confirmations_controller_spec.rb | 43 ++++++++++++++++++++++ 1 file changed, 43 insertions(+) (limited to 'spec/controllers') diff --git a/spec/controllers/auth/confirmations_controller_spec.rb b/spec/controllers/auth/confirmations_controller_spec.rb index b3af5e0ec..35eed4f51 100644 --- a/spec/controllers/auth/confirmations_controller_spec.rb +++ b/spec/controllers/auth/confirmations_controller_spec.rb @@ -3,6 +3,8 @@ require 'rails_helper' describe Auth::ConfirmationsController, type: :controller do + render_views + describe 'GET #new' do it 'returns http success' do @request.env['devise.mapping'] = Devise.mappings[:user] @@ -48,4 +50,45 @@ describe Auth::ConfirmationsController, type: :controller do end end end + + describe 'GET #finish_signup' do + subject { get :finish_signup } + + let(:user) { Fabricate(:user) } + before do + sign_in user, scope: :user + @request.env['devise.mapping'] = Devise.mappings[:user] + end + + it 'renders finish_signup' do + is_expected.to render_template :finish_signup + expect(assigns(:user)).to have_attributes id: user.id + end + end + + describe 'PATCH #finish_signup' do + subject { patch :finish_signup, params: { user: { email: email }} } + + let(:user) { Fabricate(:user) } + before do + sign_in user, scope: :user + @request.env['devise.mapping'] = Devise.mappings[:user] + end + + context 'when email is valid' do + let(:email) { 'new_' + user.email } + + it 'redirects to root_path' do + is_expected.to redirect_to root_path + end + end + + context 'when email is invalid' do + let(:email) { '' } + + it 'renders finish_signup' do + is_expected.to render_template :finish_signup + end + end + end end -- cgit