Refactor Workflow API routes - Part 1

[heisner-tillman]

This is a part of #10889.

Summary

• Refactored API routes:
  • POST: /api/workflows/{workflow_id}/invocations
    • Added pydantic models to input
    • Added pydantic models to return
    • Removed the mapping to the legacy routes
  • PUT: /api/workflows/{workflow_id}/refactor
    • Added pydantic models to input
    • Added pydantic models to return
    • Removed the mapping to the legacy routes
  • GET: /api/workflows/{workflow_id}
    • Added pydantic models to input
    • Added pydantic models to return
    • Removed the mapping to the legacy routes

How to test the changes?

• This is a refactoring of components with existing test coverage.
• 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

• 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

Update: It is ready now.

[heisner-tillman]

The selenium test failure might be related though https://github.com/galaxyproject/galaxy/actions/runs/8029094805/job/21939309255?pr=17463

pydantic_core._pydantic_core.ValidationError: 1 validation error for StoredWorkflowDetailed
    | source_metadata
    |   Input should be a valid string [type=string_type, input_value={'url': 'https://raw.gith...ata/test_workflow_1.ga'}, input_type=dict]
    |     For further information visit https://errors.pydantic.dev/2.5/v/string_type

Yes, it expects a dict there and not a str. I fixed it now.

Thanks for pointing it out!

[jdavcs]

@davelopez can we merge this?

[davelopez]

Yes, It looks pretty good to me! 👍

[github-actions]

This PR was merged without a "kind/" label, please correct.