Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests_Image_Editor_Imagick Issue #202

Open
forgetwp opened this issue Nov 27, 2023 · 4 comments
Open

Tests_Image_Editor_Imagick Issue #202

forgetwp opened this issue Nov 27, 2023 · 4 comments
Labels

Comments

@forgetwp
Copy link

I've had this issue for couple of weeks now and I cannot seems to find any solution.

1) Tests_Image_Editor_Imagick::test_remove_pdf_alpha_channel_should_remove_the_alpha_channel_in_preview The intermediate size could not be retrieved. Failed asserting that false is of type "array".

/wptest/tests/phpunit/tests/image/editorImagick.php:680
/wptest/vendor/phpunit/phpunit/phpunit:107

I pulled latest git few times now, and during prepare I do see a lot of deprecated NPM warnings, but at the end it does say

Done.
Replacing variables in wp-tests-config.php
Success: Prepared environment.

However, during the test, it failing on Tests_Image_Editor_Imagick.

Worth mentioning it does also produce a lot of warnings about PHP10 unit
is deprecated and will no longer be possible in PHPUnit 10

Node.js v16.20.2
All extensions installed and checked.
Tested on PHP 7.4, 8.2, 8.3 and always the same issue.
MariaDB 10.11.6

Any help is much appreciated.

@forgetwp forgetwp added the bug label Nov 27, 2023
@forgetwp
Copy link
Author

Updated to the latest PHPUnit Test Runner Tool. Test again with PHP 7.4 all the way to 8.3 with node v21.6.1 (npm v10.2.4) - same issue.

Found this change back in 2023, so it could be related? - https://core.trac.wordpress.org/changeset/56271

@forgetwp
Copy link
Author

forgetwp commented Feb 7, 2024

Just want to add comment,
I tried with the latest commit today and this issue still exist, not sure how we could bypass this or fix this on our end.

@javiercasares
Copy link
Contributor

The test errors should be reviews by the Test Team. I'll pass the information and see if they can give us some support.

@swissspidy
Copy link
Member

I opened https://core.trac.wordpress.org/ticket/60798 to investigate this and we found a potential solution already. I suggest following that core ticket for updates in case you're interested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants