Opened 11 years ago
Closed 10 years ago
#24871 closed defect (bug) (fixed)
Imagick Preserve Alpha Unit Tests Broken
Reported by: | bpetty | Owned by: | boonebgorges |
---|---|---|---|
Milestone: | 4.1 | Priority: | low |
Severity: | minor | Version: | 3.5.2 |
Component: | Media | Keywords: | has-patch |
Focuses: | Cc: |
Description
I'm still not entirely sure why the following two tests aren't working correctly, however, alpha in the resulting files they are testing for do in fact preserve alpha correctly.
- Tests_Image_Editor_Imagick::test_image_preserves_alpha_on_resize
- Tests_Image_Editor_Imagick::test_image_preserves_alpha
It might be helpful though to at least update the code in WP_Image_UnitTestCase::assertImageAlphaAtPoint() to implement this in each editor engine so the Imagick tests actually use Imagick to check for alpha instead of GD, and in the process, that might resolve this.
Attachments (2)
Change History (11)
This ticket was mentioned in Slack in #core by boone. View the logs.
10 years ago
This ticket was mentioned in Slack in #core by voldemortensen. View the logs.
10 years ago
This ticket was mentioned in Slack in #core by voldemortensen. View the logs.
10 years ago
Note: See
TracTickets for help on using
tickets.
I think it's broken in some versions of imagemagick but I'm not sure. Currently I do think some stuff needs to be rewritten but I haven't had the change yet to do so.