diff --git a/app/controllers/verification/letter_controller.rb b/app/controllers/verification/letter_controller.rb index a1146520c..e302a1bc6 100644 --- a/app/controllers/verification/letter_controller.rb +++ b/app/controllers/verification/letter_controller.rb @@ -1,7 +1,7 @@ class Verification::LetterController < ApplicationController before_action :authenticate_user! before_action :verify_resident! - before_action :verify_phone_or_email! + before_action :verify_phone! skip_authorization_check def new @@ -39,7 +39,7 @@ class Verification::LetterController < ApplicationController params.require(:letter).permit(:verification_code) end - def verify_phone_or_email! + def verify_phone! unless current_user.confirmed_phone? redirect_to verified_user_path, alert: t('verification.letter.alert.unconfirmed_code') end diff --git a/spec/features/verification/letter_spec.rb b/spec/features/verification/letter_spec.rb index e2ebe5aa5..9fc97267f 100644 --- a/spec/features/verification/letter_spec.rb +++ b/spec/features/verification/letter_spec.rb @@ -57,17 +57,6 @@ feature 'Verify Letter' do expect(page).to have_content "We could not verify your address with the Census please try again later" end - scenario 'Send letter level 2 user verified with email' do - user = create(:user, residence_verified_at: Time.now, confirmed_phone: "611111111") - - login_as(user) - visit new_letter_path - - click_button "Send me a letter" - - expect(page).to have_content "Thank you for requesting a code maximum security in a few days we will send it to the address on your census data. Remember that you can save shipping collecting your code in any of the Office of Citizen Services." - end - scenario "Deny access unless verified residence" do user = create(:user)