Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Expands tests for various users #1049

Merged
merged 12 commits into from
Sep 5, 2024
Merged

Expands tests for various users #1049

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
382da93
Remove netid from factory user
K8Sewell Sep 3, 2024
0350562
Add context section for non yale user
K8Sewell Sep 3, 2024
46cc7af
Supplements test coverage
K8Sewell Sep 4, 2024
b8e41be
Change param sent to be the one expected by management
K8Sewell Sep 4, 2024
2625f84
Add contexts for non yale users
K8Sewell Sep 4, 2024
e15bba4
Add owp tests
K8Sewell Sep 5, 2024
5687c0b
Use uid not netid
K8Sewell Sep 5, 2024
5cbb56e
Remove comment
K8Sewell Sep 5, 2024
b708522
Add tests for terms and conditions
K8Sewell Sep 5, 2024
1e8f92f
Switch from netid to uid
K8Sewell Sep 5, 2024
04f997a
Update tests to use uid not netid
K8Sewell Sep 5, 2024
c4fc385
Undo including flappy spec
K8Sewell Sep 5, 2024
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
Prev Previous commit
Next Next commit
Switch from netid to uid
  • Loading branch information
@K8Sewell
K8Sewell committed Sep 5, 2024
commit 1e8f92f1dd510868ea5f2c4d06b3c67d3bd93513
2 changes: 1 addition & 1 deletion app/helpers/access_helper.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ def retrieve_admin_fulltext_credentials(document)
return nil if current_user.nil?
# #{ENV['MANAGEMENT_HOST']}
# for local debugging - http://yul-dc-management-1:3001/management or http://yul-dc_management_1:3001/management
url = URI.parse("#{ENV['MANAGEMENT_HOST']}/api/permission_sets/#{document}/#{current_user.netid}")
url = URI.parse("#{ENV['MANAGEMENT_HOST']}/api/permission_sets/#{document}/#{current_user.uid}")
response = Net::HTTP.get_response(url, { 'Authorization' => "Bearer #{ENV['OWP_AUTH_TOKEN']}" })
JSON.parse(response.body)
end
Expand Down