[wp-trac] [WordPress Trac] #52451: Make the media sideload functions testable and add tests

WordPress Trac noreply at wordpress.org
Thu Feb 4 17:27:08 UTC 2021


#52451: Make the media sideload functions testable and add tests
----------------------------+------------------------------
 Reporter:  hellofromTonya  |       Owner:  (none)
     Type:  enhancement     |      Status:  new
 Priority:  normal          |   Milestone:  Awaiting Review
Component:  Media           |     Version:
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:
----------------------------+------------------------------

Comment (by hellofromTonya):

 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:

 - [https://wordpress.slack.com/archives/C02SX62S6/p1603981803493000
 Comments by @johnbillion]
 >Might be difficult for the same reason the new filter in
 `media_sideload_image` didn't get a test, those functions aren't very
 testable without refectoring
 >
 >The sideloading functions could all do with a refactor to make them
 testable

 - [https://wordpress.slack.com/archives/C02SX62S6/p1612452539251000
 Discussion in Media meeting starts here].

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/52451#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list