Refactor Workflow API routes - Part 2

[heisner-tillman]

This is a part of #10889 and an extension of #17463.

Summary

• Refactored API routes:

  • [x] PUT: /api/workflows/menu

    • [x] Added pydantic models to input
    • [x] Added pydantic models to return
    • [x] Removed the mapping to the legacy route
    • [x] Write appropriate tests

  • [x] GET: /api/workflows/download/{workflow_id}, /api/workflows/{workflow_id}/download

    • [x] Added pydantic models to input
    • [x] Added pydantic models to return
    • [x] Removed the mapping to the legacy routes

How to test the changes?

• [x] This is a refactoring of components with existing test coverage.
• [x] Instructions for manual testing are as follows: You can find the interactive API documentation here: http://127.0.0.1:8080/api/docs#/workflows

License

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

[heisner-tillman]

This is not ready for review. I just wanted to create this pull request such that there is no duplication effort

[jdavcs]

This is not ready for review. I just wanted to create this pull request such that there is no duplication effort

Converting to draft.

[davelopez]

Test failures seem relevant

[heisner-tillman]

Test failures seem relevant

I rebased on dev and they disappeared. Are you sure?

[davelopez]

Was something related to workflow API, but I didn't dig much. Glad they werent related though :)