Skip to content

Commit

Permalink
Do not return User UUID in requesting_issuer_uuid when generating use…
Browse files Browse the repository at this point in the history
…r report (#11553)

changelog: Internal, Reporting, Do not return User UUID in requesting_issuer_uuid when generating user report
  • Loading branch information
mitchellhenke authored and AShukla-GSA committed Nov 25, 2024
1 parent 65d03a5 commit 8779514
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion lib/data_requests/deployed/create_user_report.rb
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ def mfa_configurations_report
end

def requesting_issuer_uuid
return user.uuid if requesting_issuers.blank?
user.agency_identities.where(agency: requesting_agencies).first&.uuid ||
"NonSPUser##{user.id}"
end
Expand Down
2 changes: 1 addition & 1 deletion spec/lib/data_requests/deployed/create_user_report_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

expect(result[:user_id]).to eq(user.id)
expect(result[:login_uuid]).to eq(user.uuid)
expect(result[:requesting_issuer_uuid]).to eq(user.uuid)
expect(result[:requesting_issuer_uuid]).to eq("NonSPUser##{user.id}")
expect(result[:email_addresses]).to be_a(Array)
expect(result[:mfa_configurations]).to be_a(Hash)
expect(result[:user_events]).to be_a(Array)
Expand Down

0 comments on commit 8779514

Please sign in to comment.