Opened 13 years ago
Closed 12 years ago
#24871 closed defect (bug) (fixed)
Imagick Preserve Alpha Unit Tests Broken
| Reported by: | bpetty | Owned by: | boonebgorges |
|---|---|---|---|
| Priority: | low | Milestone: | 4.1 |
| Component: | Media | Version: | 3.5.2 |
| Severity: | minor | Keywords: | has-patch |
| Cc: | Focuses: |
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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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.