Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions app/controllers/idv/link_sent_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,13 @@ def extra_view_variables
private

def confirm_upload_step_complete
return if flow_session['Idv::Steps::UploadStep']
return if flow_session[:flow_path] == 'hybrid'

redirect_to idv_doc_auth_url
if flow_session[:flow_path] == 'standard'
redirect_to idv_document_capture_url
else
redirect_to idv_doc_auth_url
end
end

def confirm_document_capture_needed
Expand Down
2 changes: 1 addition & 1 deletion app/services/idv/steps/upload_step.rb
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ def application
def bypass_send_link_steps
mark_step_complete(:link_sent)

flow_session[:flow_path] = @flow.flow_path
flow_session[:flow_path] = 'standard'
redirect_to idv_document_capture_url

form_response(destination: :document_capture)
Expand Down
25 changes: 18 additions & 7 deletions spec/controllers/idv/link_sent_controller_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@
{ 'document_capture_session_uuid' => 'fd14e181-6fb1-4cdc-92e0-ef66dad0df4e',
:threatmetrix_session_id => 'c90ae7a5-6629-4e77-b97c-f1987c2df7d0',
:flow_path => 'hybrid',
:phone_for_mobile_flow => '201-555-1212',
'Idv::Steps::UploadStep' => true }
:phone_for_mobile_flow => '201-555-1212' }
end

let(:user) { create(:user) }
Expand Down Expand Up @@ -79,13 +78,25 @@
)
end

context 'upload step is not complete' do
it 'redirects to idv_doc_auth_url' do
flow_session['Idv::Steps::UploadStep'] = nil
context '#confirm_upload_step_complete' do
context 'no flow_path' do
it 'redirects to idv_doc_auth_url' do
flow_session[:flow_path] = nil

get :show
get :show

expect(response).to redirect_to(idv_doc_auth_url)
end
end

context 'flow_path is standard' do
it 'redirects to idv_document_capture_url' do
flow_session[:flow_path] = 'standard'

get :show

expect(response).to redirect_to(idv_doc_auth_url)
expect(response).to redirect_to(idv_document_capture_url)
end
end
end

Expand Down