diff --git a/app/controllers/idv/by_mail/request_letter_controller.rb b/app/controllers/idv/by_mail/request_letter_controller.rb index 739e33b5a13..e393395254d 100644 --- a/app/controllers/idv/by_mail/request_letter_controller.rb +++ b/app/controllers/idv/by_mail/request_letter_controller.rb @@ -112,7 +112,7 @@ def resend_letter def confirmation_maker_perform confirmation_maker = GpoConfirmationMaker.new( - pii: Pii::Cacher.new(current_user, user_session).fetch, + pii: pii, service_provider: current_sp, profile: current_user.pending_profile, ) @@ -120,6 +120,11 @@ def confirmation_maker_perform confirmation_maker end + def pii + Pii::Cacher.new(current_user, user_session). + fetch(current_user.gpo_verification_pending_profile.id) + end + def send_reminder current_user.send_email_to_all_addresses(:letter_reminder) end diff --git a/spec/controllers/idv/by_mail/request_letter_controller_spec.rb b/spec/controllers/idv/by_mail/request_letter_controller_spec.rb index 2e0a7c38980..18958ff6c6b 100644 --- a/spec/controllers/idv/by_mail/request_letter_controller_spec.rb +++ b/spec/controllers/idv/by_mail/request_letter_controller_spec.rb @@ -181,7 +181,7 @@ context 'resending a letter' do let(:has_pending_profile) { true } - let(:pending_profile) { create(:profile, :verify_by_mail_pending) } + let(:pending_profile) { create(:profile, :with_pii, :verify_by_mail_pending) } before do stub_sign_in(user) @@ -245,9 +245,9 @@ end def expect_resend_letter_to_send_letter_and_redirect(otp:) - pii = { first_name: 'Samuel', last_name: 'Sampson' } + pii = pending_profile.decrypt_pii(user.password).to_h pii_cacher = instance_double(Pii::Cacher) - allow(pii_cacher).to receive(:fetch).and_return(pii) + allow(pii_cacher).to receive(:fetch).with(pending_profile.id).and_return(pii) allow(pii_cacher).to receive(:exists_in_session?).and_return(true) allow(Pii::Cacher).to receive(:new).and_return(pii_cacher)