-
Notifications
You must be signed in to change notification settings - Fork 992
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.
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
Migrate Library Contents API to FastAPI #18838
base: dev
Are you sure you want to change the base?
Conversation
Updating test_update.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least some of the test failures are due to wrong Library Folder ID decoding.
Looks like LibraryFolderDatabaseIdField
is not stripping the initial "F" before decoding here:
galaxy/lib/galaxy/schema/fields.py
Line 81 in 5e332f0
BeforeValidator(lambda database_id: Security.security.decode_id(database_id)), |
I think the test failures are because I forgot to add the model for the folder ids on library_id! I have added them to the new commit. |
…nt caused in test_materialize_library_dataset
Have a look at other endpoints that upload files, for example this one:
|
What did you do?
lib/galaxy_test/api/
library_contents
API logic so all the logic is contained in theLibraryContentsService
classrelated to #10889
How to test the changes?
(Select all options that apply)
License