Skip to content

fix(folders): more than one per PR - #608

Open
jbarno wants to merge 1 commit into
masterfrom
feat/all_folders
Open

fix(folders): more than one per PR#608
jbarno wants to merge 1 commit into
masterfrom
feat/all_folders

Conversation

@jbarno

@jbarno jbarno commented Jul 22, 2026

Copy link
Copy Markdown

Link to JIRA ticket if there is one:

New Features

Breaking Changes

Bug Fixes

Improvements

Dependency updates

Deployment changes

@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.

result = modify_env_for_manifest_pr(namespace, updated_folder, repo)
assert result.lower() == "success"
for i in range(len(updated_folders)):
folder = updated_folder[i]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Removing this would result in setting up the env multiple times with each manifest folder. I dont think we should be modifying this logic

@github-actions

Copy link
Copy Markdown

Integration Tests on Kind

filepath passed skipped SUBTOTAL
tests/test_gen3_workflow.py 25 4 29
TOTAL 25 4 29

@github-actions

Copy link
Copy Markdown

Integration Tests

Test summary after running integration tests

filepath passed failed skipped SUBTOTAL
tests/test_oauth2.py 15 0 0 15
tests/test_gen3_workflow.py 28 0 1 29
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 12 1 1 14
tests/test_audit_service.py 3 0 3 6
tests/test_data_upload.py 8 0 1 9
tests/test_pfb_export.py 1 0 0 1
tests/test_guppy_service.py 13 0 0 13
tests/test_presigned_url.py 7 0 0 7
tests/test_drs_endpoint.py 6 0 0 6
tests/test_requestor.py 6 0 0 6
tests/test_user_token.py 5 0 0 5
tests/test_dbgap.py 4 0 1 5
tests/test_user_data_library.py 2 0 2 4
tests/test_user_login_activation.py 2 0 1 3
tests/test_study_viewer.py 3 0 0 3
tests/test_fence_admin.py 2 0 0 2
tests/test_register_user.py 1 1 0 2
tests/test_discoverypage.py 1 0 0 1
tests/test_oidc_client.py 1 1 0 2
tests/test_workspace.py 1 0 0 1
tests/test_aggregate_mds.py 1 0 0 1
tests/test_client_credentials.py 1 0 0 1
tests/test_manifest_service.py 1 0 0 1
tests/test_login_page.py 2 0 0 2
tests/test_gen3_sdk.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_gen3client.py 1 0 0 1
tests/test_ras_authn.py 0 0 3 3
tests/test_ras_passport.py 0 0 3 3
tests/test_gen3ff_landing_page.py 0 0 2 2
tests/test_env_sanity.py 0 0 1 1
tests/test_etl.py 0 0 1 1
TOTAL 150 3 20 173

Test summary after rerunning failed integration tests

filepath passed SUBTOTAL
tests/test_register_user.py 1 1
tests/test_graph_submit_and_query.py 1 1
tests/test_oidc_client.py 1 1
TOTAL 3 3

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants