![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 5 years ago
Last modified 5 years ago
#52451 new enhancement
Make the media sideload functions testable and add tests
| Reported by: | hellofromTonya | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Awaiting Review |
| Component: | Media | Version: | |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
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: