Skip to content

Add test for login authorized user functionality#453

Merged
krishnaa05 merged 19 commits intomasterfrom
chore/only_login_authorized_users
Apr 21, 2026
Merged

Add test for login authorized user functionality#453
krishnaa05 merged 19 commits intomasterfrom
chore/only_login_authorized_users

Conversation

@krishnaa05
Copy link
Copy Markdown
Contributor

@krishnaa05 krishnaa05 commented Jan 22, 2026

Link to JIRA ticket if there is one: GQE-370

New Features

Breaking Changes

Bug Fixes

Improvements

Dependency updates

Deployment changes

@krishnaa05 krishnaa05 added the wip label Jan 22, 2026
@krishnaa05 krishnaa05 changed the title Add test for logion authorized user functionality Add test for login authorized user functionality Jan 22, 2026
@krishnaa05 krishnaa05 removed the wip label Jan 23, 2026
@github-actions
Copy link
Copy Markdown

Test summary after running integration tests

filepath passed failed skipped SUBTOTAL
tests/test_fence_admin.py 0 2 0 2
tests/test_user_login_activation.py 2 0 1 3
TOTAL 2 2 1 5

Test summary after rerunning failed integration tests

filepath failed SUBTOTAL
tests/test_fence_admin.py 2 2
TOTAL 2 2

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

filepath passed skipped SUBTOTAL
tests/test_user_login_activation.py 2 1 3
tests/test_fence_admin.py 2 0 2
TOTAL 4 1 5

Please find the detailed integration test report here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

Test summary after running integration tests

filepath passed failed skipped SUBTOTAL
tests/test_oauth2.py 15 0 0 15
tests/test_dicom_viewer.py 4 0 0 4
tests/test_centralized_auth.py 16 0 0 16
tests/test_graph_submit_and_query.py 13 0 1 14
tests/test_gen3_workflow.py 13 0 0 13
tests/test_pfb_export.py 1 0 0 1
tests/test_guppy_service.py 8 0 0 8
tests/test_data_upload.py 7 1 1 9
tests/test_presigned_url.py 7 0 0 7
tests/test_audit_service.py 3 0 3 6
tests/test_dbgap.py 4 0 1 5
tests/test_study_viewer.py 3 0 0 3
tests/test_requestor.py 5 0 0 5
tests/test_user_token.py 5 0 0 5
tests/test_drs_endpoint.py 4 0 0 4
tests/test_user_data_library.py 2 0 1 3
tests/test_oidc_client.py 2 0 0 2
tests/test_login_page.py 2 0 0 2
tests/test_user_login_activation.py 2 0 1 3
tests/test_discoverypage.py 1 0 0 1
tests/test_workspace.py 1 0 0 1
tests/test_client_credentials.py 1 0 0 1
tests/test_gen3client.py 1 0 0 1
tests/test_register_user.py 2 0 0 2
tests/test_fence_admin.py 2 0 0 2
tests/test_manifest_service.py 1 0 0 1
tests/test_aggregate_mds.py 1 0 0 1
tests/test_google_data_access.py 1 0 0 1
tests/test_homepage.py 1 0 0 1
tests/test_ras_authn.py 0 0 3 3
tests/test_gen3ff_landing_page.py 0 0 2 2
tests/test_etl.py 0 0 1 1
tests/test_env_sanity.py 0 0 1 1
TOTAL 128 1 15 144

Test summary after rerunning failed integration tests

filepath passed skipped SUBTOTAL
tests/test_data_upload.py 8 1 9
TOTAL 8 1 9

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

Comment thread gen3-integration-tests/tests/test_user_login_activation.py
@github-actions
Copy link
Copy Markdown

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

@github-actions
Copy link
Copy Markdown

filepath passed skipped SUBTOTAL
tests/test_homepage.py 1 0 1
tests/test_user_login_activation.py 2 1 3
TOTAL 3 1 4

Please find the detailed integration test report here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

filepath passed skipped SUBTOTAL
tests/test_user_login_activation.py 2 1 3
TOTAL 2 1 3

Please find the detailed integration test report here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

filepath passed skipped SUBTOTAL
tests/test_user_login_activation.py 2 1 3
tests/test_fence_admin.py 2 0 2
TOTAL 4 1 5

Please find the detailed integration test report here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

filepath passed skipped SUBTOTAL
tests/test_user_login_activation.py 2 1 3
tests/test_fence_admin.py 2 0 2
TOTAL 4 1 5

Please find the detailed integration test report here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

filepath passed skipped SUBTOTAL
tests/test_user_login_activation.py 2 1 3
tests/test_fence_admin.py 2 0 2
TOTAL 4 1 5

Please find the detailed integration test report here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

filepath passed skipped SUBTOTAL
tests/test_user_login_activation.py 2 1 3
tests/test_fence_admin.py 2 0 2
TOTAL 4 1 5

Please find the detailed integration test report here

Please find the Github Action logs here

Copy link
Copy Markdown
Contributor

@pieterlukasse pieterlukasse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @krishnaa05 . I've added some comments and code suggestions.

Comment thread gen3-integration-tests/pages/login.py
Comment thread gen3-integration-tests/tests/test_fence_admin.py Outdated
Comment thread gen3-integration-tests/tests/test_fence_admin.py Outdated
Comment thread gen3-integration-tests/tests/test_user_login_activation.py
Comment thread gen3-integration-tests/tests/test_user_login_activation.py
krishnaa05 and others added 4 commits April 17, 2026 08:11
Co-authored-by: pieterlukasse <pieterlukasse@gmail.com>
Co-authored-by: pieterlukasse <pieterlukasse@gmail.com>
Co-authored-by: pieterlukasse <pieterlukasse@gmail.com>
@github-actions
Copy link
Copy Markdown

filepath passed skipped SUBTOTAL
tests/test_fence_admin.py 2 0 2
tests/test_user_login_activation.py 2 1 3
TOTAL 4 1 5

Please find the detailed integration test report here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

filepath passed skipped SUBTOTAL
tests/test_user_login_activation.py 2 1 3
tests/test_fence_admin.py 2 0 2
TOTAL 4 1 5

Please find the detailed integration test report here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

Test summary after running integration tests

filepath passed failed skipped SUBTOTAL
tests/test_user_login_activation.py 1 1 1 3
tests/test_fence_admin.py 2 0 0 2
TOTAL 3 1 1 5

Test summary after rerunning failed integration tests

filepath failed SUBTOTAL
tests/test_user_login_activation.py 1 1
TOTAL 1 1

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

@github-actions
Copy link
Copy Markdown

filepath passed skipped SUBTOTAL
tests/test_fence_admin.py 2 0 2
tests/test_user_login_activation.py 2 1 3
TOTAL 4 1 5

Please find the detailed integration test report here

Please find the Github Action logs here

@krishnaa05 krishnaa05 merged commit c124e25 into master Apr 21, 2026
5 checks passed
@krishnaa05 krishnaa05 deleted the chore/only_login_authorized_users branch April 21, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants