[wp-trac] [WordPress Trac] #41148: Test for ticket 39875 can throw PHP error

[WordPress Trac]

#41148: Test for ticket 39875 can throw PHP error
--------------------------+------------------------------
 Reporter:  schlessera    |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Media         |     Version:  4.8
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:
--------------------------+------------------------------

Comment (by johnbillion):

 Related: #40536, #40537.

 Replying to [comment:6 schlessera]:
 > Alternatively to the asserts I added, we could add a check to see
 whether the proper dependencies are installed, and skip the tests with a
 corresponding notice if not.
 There's already a bunch of test skipping in place for other tests that
 depend on Imagick. The same pattern can be used.

 In the long term, I think tests that require Imagick, GS, etc should fail
 (instead of be skipped) if the dependencies are not installed.

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