Changeset 60746 for branches/6.6

Timestamp:

09/15/2025 07:22:09 PM (7 months ago)

Author:

desrosj

Message:

Build/Test Tools: Fix tooling and PHPUnit test issues.

This fixes the PHPUnit test suite for the 6.6 branch by:

• Fixing issues with the env:install script when using newer PHP images.
• Temporarily disables failing tests with ImageMagick 7 is in use.

Backports [60735] and [60736] to the 6.6 branch.

See #63876, #63932.

Location:

branches/6.6

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/image/editorImagick.php (modified) (1 diff)
• tests/phpunit/tests/image/resize.php (modified) (10 diffs)
• tests/phpunit/tests/media.php (modified) (1 diff)
• tools/local-env/scripts/install.js (modified) (1 diff)

branches/6.6/tests/phpunit/tests/image/editorImagick.php

@@ -657 +657 @@
         }
 
+        $version = Imagick::getVersion();
+        if ( $version['versionNumber'] < 0x675 ) {
+            $this->markTestSkipped( 'The version of ImageMagick does not support removing alpha channels from PDFs.' );
+        }
+
         $test_file     = DIR_TESTDATA . '/images/test-alpha.pdf';
         $attachment_id = $this->factory->attachment->create_upload_object( $test_file );

branches/6.6/tests/phpunit/tests/image/resize.php

@@ -24 +24 @@
         $image = $this->resize_helper( DIR_TESTDATA . '/images/test-image.jpg', 25, 25 );
 
+        $this->assertNotWPError( $image );
+
         list( $w, $h, $type ) = getimagesize( $image );
 
@@ -79 +81 @@
         $image = $this->resize_helper( $file, 25, 25 );
 
+        $this->assertNotWPError( $image );
+
         list( $w, $h, $type ) = wp_getimagesize( $image );
 
@@ -93 +97 @@
      *
      * @ticket 51228
+     *
+     * Temporarily disabled until we can figure out why it fails on the Trixie based PHP container.
+     * See https://core.trac.wordpress.org/ticket/63932.
+     * @requires PHP < 8.3
      */
     public function test_resize_avif() {
@@ -105 +113 @@
         $image = $this->resize_helper( $file, 25, 25 );
 
+        $this->assertNotWPError( $image );
+
         list( $w, $h, $type ) = wp_getimagesize( $image );
 
@@ -126 +136 @@
         $image = $this->resize_helper( DIR_TESTDATA . '/images/2007-06-17DSC_4173.JPG', 128, 96 );
 
+        $this->assertNotWPError( $image );
+
         list( $w, $h, $type ) = getimagesize( $image );
 
@@ -139 +151 @@
         $image = $this->resize_helper( DIR_TESTDATA . '/images/2007-06-17DSC_4173.JPG', 128, 0 );
 
+        $this->assertNotWPError( $image );
+
         list( $w, $h, $type ) = getimagesize( $image );
 
@@ -152 +166 @@
         $image = $this->resize_helper( DIR_TESTDATA . '/images/2007-06-17DSC_4173.JPG', 0, 96 );
 
+        $this->assertNotWPError( $image );
+
         list( $w, $h, $type ) = getimagesize( $image );
 
@@ -165 +181 @@
         $image = $this->resize_helper( DIR_TESTDATA . '/images/2007-06-17DSC_4173.JPG', 150, 150, true );
 
+        $this->assertNotWPError( $image );
+
         list( $w, $h, $type ) = getimagesize( $image );
 
@@ -190 +208 @@
     public function test_resize_thumb_50x150_crop() {
         $image = $this->resize_helper( DIR_TESTDATA . '/images/2007-06-17DSC_4173.JPG', 50, 150, true );
+
+        $this->assertNotWPError( $image );
 
         list( $w, $h, $type ) = getimagesize( $image );
@@ -215 +235 @@
     /**
      * Function to help out the tests
+     *
+     * @return string|WP_Error The path to the resized image file or a WP_Error on failure.
      */
     protected function resize_helper( $file, $width, $height, $crop = false ) {

branches/6.6/tests/phpunit/tests/media.php

@@ -5311 +5311 @@
         // Sub-sizes: for each size, the JPEGs should be smaller than the WebP.
         $sizes_to_compare = array_intersect_key( $jpeg_sizes['sizes'], $webp_sizes['sizes'] );
+
+        $this->assertNotWPError( $sizes_to_compare );
+
         foreach ( $sizes_to_compare as $size => $size_data ) {
             $this->assertLessThan( $webp_sizes['sizes'][ $size ]['filesize'], $jpeg_sizes['sizes'][ $size ]['filesize'] );

branches/6.6/tools/local-env/scripts/install.js

@@ -38 +38 @@
 wait_on( { resources: [ `tcp:localhost:${process.env.LOCAL_PORT}`] } )
     .then( () => {
-        wp_cli( 'db reset --yes' );
+        wp_cli( 'db reset --yes --defaults' );
         const installCommand = process.env.LOCAL_MULTISITE === 'true'  ? 'multisite-install' : 'install';
         wp_cli( `core ${ installCommand } --title="WordPress Develop" --admin_user=admin --admin_password=password --admin_email=test@test.com --skip-email --url=http://localhost:${process.env.LOCAL_PORT}` );