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

Migrate Library Contents API to FastAPI #18838

Open
wants to merge 71 commits into
base: dev
Choose a base branch
from
Open

Migrate Library Contents API to FastAPI #18838

wants to merge 71 commits into from
+1,949 −611

Conversation

arash77
Copy link
Collaborator

@arash77 arash77 commented Sep 17, 2024

What did you do?

  • Include an API test under lib/galaxy_test/api/
  • Refactor the legacy library_contents API logic so all the logic is contained in the LibraryContentsService class
  • Create pydantic models for the responses and payloads
  • Create the FastAPI routes that will replace the old ones

related to #10889

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@arash77 arash77 marked this pull request as ready for review September 26, 2024 12:46
@github-actions github-actions bot added this to the 24.2 milestone Sep 26, 2024
@arash77
Refactor library content routes and controllers
156ede6
@arash77
Refactor library content routes and controllers
b80f249
@arash77
Refactor library content routes and controllers
4dee8ed
@arash77
Remove old library_contents API route configurations
b90179d
@arash77
make the review process easy
0c6837d
@arash77
Refactor library content routes and controllers
fdcc850
@arash77
remove unused imports
f423125
@arash77
move _create_response function to services
3d41893
@arash77
Fix url_builder
4cf20ed
@arash77
Add json=true to library_content api tests
4873336
@arash77
update OpenAPI schema
3995e6e
@arash77
Update OpenAPI schema
f3670d9
@arash77
Reformating populators.py
75c6489
@arash77
Fix test_libraries erros
a9151e9
@arash77
add LibraryFolderDatabaseIdField Model for library_id
07db06a
@arash77
Fix schema for uploaded_by being None in show library contents endpoi…
c0e589b 
…nt caused in test_materialize_library_dataset
@arash77
Add function for supporting files in library_contents
9e47626
@arash77
remove unused enumerate and files field into LibraryContentsFileCreat…
3e22733 
…ePayload
@arash77
Refactor for better readability
6d59a46
@arash77
send data as json when there is no files on creating new_library_data…
d43e6b8 
…set for tests
@arash77
update files to upload_files in create payload
4d30692
@arash77
remove unused import
9d0bd72
@arash77
send none files library_contents create in population with json
f4cffb3
@arash77
Apply suggestions: s/title/description in schema, don't do try/except…
0a86966 
… in delete logic, rename type aliases for consistency
@arash77
remove unused import
d7c3b81
@arash77
update schema
89a056e
@arash77
Merge branch 'LC_test' into FastAPI_library_contents
8d417fb
@arash77
schema linting
b03ce07
@arash77
Using DecodedDatabaseIdField instead of `Union[DecodedDatabaseIdFie…
110e467 
…ld, LibraryFolderDatabaseIdField]` for library_id and fix `_create_dataset_in_folder_in_library` function that is leading to failed tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mvdbeek mvdbeek mvdbeek left review comments

@davelopez davelopez davelopez left review comments

Assignees
No one assigned
Labels
area/API area/testing/api area/testing
Projects
None yet
Milestone
24.2
Development

Successfully merging this pull request may close these issues.
3 participants
@arash77 @davelopez @mvdbeek