Skip to content

PD-202 improve metadata functions - #317

Open
mfshao wants to merge 4 commits into
masterfrom
feat/improve-query
Open

mfshao wants to merge 4 commits into
masterfrom
feat/improve-query

Conversation

@mfshao

@mfshao mfshao commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/PD-202

New Features

Breaking Changes

Bug Fixes

Improvements

  • gen3.metadata.Gen3Metadata.query() no longer requires a query parameter, if omitted, all metadata record with MDS will be returned (subject to limit and offset)
  • gen3.metadata.Gen3Metadata() no longer requires a gen3.auth.Gen3Auth object during initialization. If functions that requires auth will be called, an message will be posted to the user and the call will return directly

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.

@github-actions

Copy link
Copy Markdown

Integration Tests

Test summary after running integration tests

filepath passed failed skipped SUBTOTAL
tests/test_graph_submit_and_query.py 12 1 1 14
tests/test_data_upload.py 8 0 1 9
tests/test_presigned_url.py 8 0 0 8
tests/test_centralized_auth.py 5 0 0 5
tests/test_audit_service.py 1 0 0 1
tests/test_dbgap.py 4 0 1 5
tests/test_gen3_sdk.py 1 0 0 1
tests/test_ras_passport.py 0 0 2 2
tests/test_google_data_access.py 0 1 0 1
TOTAL 39 2 5 46

Test summary after rerunning failed integration tests

filepath passed failed SUBTOTAL
tests/test_graph_submit_and_query.py 1 0 1
tests/test_google_data_access.py 0 1 1
TOTAL 1 1 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

Failure Analysis

Please find the detailed test analysis report here

This branch has not been deployed

No deployments
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.

1 participant