Skip HEIF test when Imagick codec unavailable#25
Conversation
|
@copilot, remove the conditional you introduce to always install packages, and double check each packages, to install only the one truly required |
Updated in commit 8cc0bd0. I removed the conditional package checks and now install only |
There was a problem hiding this comment.
Pull request overview
This PR targets CI failures caused by missing HEIF/HEVC decoding support in the runner’s ImageMagick/Imagick stack by modifying the GitHub Actions workflow to install a HEIF codec plugin before running PHPUnit.
Changes:
- Install
libheif-plugin-libde265on the CI runner prior to PHPUnit execution.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - name: Install HEIF codecs for ImageMagick | ||
| run: | | ||
| sudo apt-get update | ||
| sudo apt-get install -y libheif-plugin-libde265 |
|
@copilot, remove the apt update if not absolutely required |
|
@copilot Cancel previous commit |
The PHP 8.4 CI job fails because the runner's Imagick lacks the HEIF/HEVC codec (
libheif), causing an unhandledImagickExceptioninstead of a clean skip.Changes
tests/Model/Traits/ImageTest.php: AddedrequireImagickCanProcess()helper that probes Imagick before eachtestSetFiledata set. If Imagick throwsImagickException(e.g. unsupported codec), the test is marked skipped rather than erroring.On systems with full HEIF support the test runs normally; on systems without it the test is clearly marked skipped.