diff --git a/.reek b/.reek index dc06ece04f5..ddb112d50fc 100644 --- a/.reek +++ b/.reek @@ -106,7 +106,6 @@ UtilityFunction: - WorkerHealthChecker::Middleware#call - UserEncryptedAttributeOverrides#create_fingerprint - LocaleHelper#locale_url_param - - Verify::Base#mock_vendor_partial - IdvSession#timed_out_vendor_error 'app/controllers': InstanceVariableAssumption: diff --git a/app/view_models/verify/base.rb b/app/view_models/verify/base.rb index 46d0c291aa0..c31b052e948 100644 --- a/app/view_models/verify/base.rb +++ b/app/view_models/verify/base.rb @@ -11,14 +11,6 @@ def initialize(error: nil, remaining_attempts:, idv_form:, timed_out: nil) attr_reader :error, :remaining_attempts, :idv_form - def mock_vendor_partial - if FeatureManagement.no_pii_mode? - 'verify/sessions/no_pii_warning' - else - 'shared/null' - end - end - def title I18n.t("idv.titles.#{step_name}") end diff --git a/app/views/verify/finance/new.html.slim b/app/views/verify/finance/new.html.slim index 1e4f881b2f0..f4107356784 100644 --- a/app/views/verify/finance/new.html.slim +++ b/app/views/verify/finance/new.html.slim @@ -1,5 +1,4 @@ - title @view_model.title -= render @view_model.mock_vendor_partial h1.h3.my0 = @view_model.title p.mt-tiny.mb0 diff --git a/app/views/verify/finance_other/new.html.slim b/app/views/verify/finance_other/new.html.slim index 29f6e7f4665..fc71e2a30b9 100644 --- a/app/views/verify/finance_other/new.html.slim +++ b/app/views/verify/finance_other/new.html.slim @@ -1,5 +1,4 @@ - title @view_model.title -= render @view_model.mock_vendor_partial h1.h3.my0 = @view_model.title p.mt-tiny.mb0 = t('idv.messages.finance.intro_account') diff --git a/app/views/verify/phone/new.html.slim b/app/views/verify/phone/new.html.slim index 39dd10e522d..24cfe4bef20 100644 --- a/app/views/verify/phone/new.html.slim +++ b/app/views/verify/phone/new.html.slim @@ -1,5 +1,4 @@ - title @view_model.title -= render @view_model.mock_vendor_partial h1.h2.my0 = t('idv.titles.session.phone') p.mt-tiny.mb2 = t('idv.messages.phone.intro') diff --git a/app/views/verify/sessions/_no_pii_warning.html.slim b/app/views/verify/sessions/_no_pii_warning.html.slim deleted file mode 100644 index b05b9d379ca..00000000000 --- a/app/views/verify/sessions/_no_pii_warning.html.slim +++ /dev/null @@ -1 +0,0 @@ -.mt1.mb2.h6.caps.bold.red = t('idv.messages.sessions.no_pii') diff --git a/app/views/verify/sessions/new.html.slim b/app/views/verify/sessions/new.html.slim index 59fe0958e9c..4f18b4307bd 100644 --- a/app/views/verify/sessions/new.html.slim +++ b/app/views/verify/sessions/new.html.slim @@ -1,5 +1,4 @@ - title @view_model.title -= render @view_model.mock_vendor_partial h1.h3.my0 = @view_model.title = simple_form_for(@view_model.idv_form, url: verify_session_path, diff --git a/spec/view_models/verify/base_spec.rb b/spec/view_models/verify/base_spec.rb index a5dcd367dc8..497b3275899 100644 --- a/spec/view_models/verify/base_spec.rb +++ b/spec/view_models/verify/base_spec.rb @@ -1,28 +1,6 @@ require 'rails_helper' RSpec.describe Verify::Base do - describe '#mock_vendor_partial' do - context 'idv vendor is mock' do - it 'returns no pii warning partial' do - allow(Figaro.env).to receive(:proofing_vendors).and_return('mock') - - partial = Verify::Base.new(remaining_attempts: 1, idv_form: nil).mock_vendor_partial - - expect(partial).to eq 'verify/sessions/no_pii_warning' - end - end - - context 'idv vendor is not mock' do - it 'returns null partial' do - allow(Figaro.env).to receive(:proofing_vendors).and_return('other') - - partial = Verify::Base.new(remaining_attempts: 1, idv_form: nil).mock_vendor_partial - - expect(partial).to eq 'shared/null' - end - end - end - describe '#message' do let(:timed_out) { false } let(:view_model) do