Opened 4 years ago
Last modified 4 years ago
#52451 new enhancement
Make the media sideload functions testable and add tests
Reported by: | hellofromTonya | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Media | Keywords: | |
Focuses: | Cc: |
Description
Goals:
- Make the media "sideload" functions testable
- Provide code coverage for these functions
To accomplish this:
- These functions will first need to be refactored.
- Then unit tests can be built to provide code coverage.
Sideload functions include:
media_handle_sideload()
media_sideload_image()
A guide for the refactoring work:
- All changes will focus on testability
- No changes should impact or alter the current behavior, extensibility, or backwards compatibility
- Care will be given to maintainability and readability (i.e. code quality)
Note: See
TracTickets for help on using
tickets.
For reference, the drivers for this ticket were discussed in #core-media slack (links below) initiated by Ticket #50972.
Links to slack discussions prior to opening this ticket: