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
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ class AuthorizationConfirmationController < ApplicationController
def show
analytics.track_event(Analytics::AUTHENTICATION_CONFIRMATION)
@sp = ServiceProvider.find_by(issuer: sp_session[:issuer])
@email = EmailContext.new(current_user).last_sign_in_email_address.email
end

def update
Expand Down
2 changes: 1 addition & 1 deletion app/views/users/authorization_confirmation/show.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
<%= t("help_text.requested_attributes.email") %>
</span>
<span class='padding-left-2'>
<%= current_user.email %>
<%= @email %>
</span>
</li>
</ul>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,14 @@ def create_user_and_remember_device
user1
end

it 'it confirms the user wants to continue to the SP after signing in again' do
sign_in_user(user1)
it 'it confirms the user wants to continue to SP with signin email after signing in again' do
second_email = create(:email_address, user: user1)
sign_in_user(user1, second_email.email)
visit_idp_from_ial1_oidc_sp
expect(current_url).to match(user_authorization_confirmation_path)
expect(page).to have_content second_email.email

continue_as(user1.email)
continue_as(second_email.email)
expect(current_url).to match('http://localhost:7654/auth/result')
end

Expand Down
9 changes: 5 additions & 4 deletions spec/features/saml/authorization_confirmation_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -32,14 +32,15 @@ def create_user_and_remember_device
user1
end

it 'it confirms the user wants to continue to the SP after signing in again' do
sign_in_user(user1)
it 'it confirms the user wants to continue to SP with signin email after signing in again' do
second_email = create(:email_address, user: user1)
sign_in_user(user1, second_email.email)

visit request_url

expect(current_url).to match(user_authorization_confirmation_path)
continue_as(user1.email)
expect(page).to have_content second_email.email

continue_as(second_email.email)
expect(current_url).to eq(request_url)
end

Expand Down
5 changes: 3 additions & 2 deletions spec/support/features/session_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -171,8 +171,9 @@ def sign_up_with_backup_codes_and_set_password
user
end

def sign_in_user(user = create(:user))
signin(user.email_addresses.first.email, user.password)
def sign_in_user(user = create(:user), email = nil)
email ||= user.email_addresses.first.email
signin(email, user.password)
user
end

Expand Down