Changeset 57930

Timestamp:

04/04/2024 09:10:39 PM (6 months ago)

Author:

jorbin

Message:

Tests: Update expectations in wp_remote_head() and wp_remote_get() tests.

It appears that something has changed on the WP.com side to compress the requested images on the fly, which interfered with the previous expectations in these tests.

This commit uses a direct file URL and updates the expected image size to match the currently returned response.

Follow-up to [139/tests], [31258], [47142].

Reviewed by jorbin.
Merges [57903] to the 6.4 branch.

Props dextorlobo, swissspidy, davidbaumwald, SergeyBiryukov.
See #60865.

Location:

branches/6.4

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/functions/wpRemoteFopen.php (modified) (1 diff)
• tests/phpunit/tests/http/functions.php (modified) (5 diffs)

branches/6.4/tests/phpunit/tests/functions/wpRemoteFopen.php

@@ -27 +27 @@
     public function test_wp_remote_fopen() {
         // This URL gives a direct 200 response.
-        $url      = 'https://asdftestblog1.files.wordpress.com/2007/09/2007-06-30-dsc_4700-1.jpg';
+        $url      = 'https://asdftestblog1.wordpress.com/wp-content/uploads/2007/09/2007-06-30-dsc_4700-1.jpg';
         $response = wp_remote_fopen( $url );
 
         $this->assertIsString( $response );
-        $this->assertSame( 40148, strlen( $response ) );
+        $this->assertSame( 31325, strlen( $response ) );
     }
 }

branches/6.4/tests/phpunit/tests/http/functions.php

@@ -12 +12 @@
     public function test_head_request() {
         // This URL gives a direct 200 response.
-        $url      = 'https://asdftestblog1.files.wordpress.com/2007/09/2007-06-30-dsc_4700-1.jpg';
+        $url      = 'https://asdftestblog1.wordpress.com/wp-content/uploads/2007/09/2007-06-30-dsc_4700-1.jpg';
         $response = wp_remote_head( $url );
 
@@ -22 +22 @@
 
         $this->assertSame( 'image/jpeg', $headers['Content-Type'] );
-        $this->assertSame( '40148', $headers['Content-Length'] );
+        $this->assertSame( '31325', $headers['Content-Length'] );
         $this->assertSame( 200, wp_remote_retrieve_response_code( $response ) );
     }
@@ -42 +42 @@
      */
     public function test_head_404() {
-        $url      = 'https://asdftestblog1.files.wordpress.com/2007/09/awefasdfawef.jpg';
+        $url      = 'https://asdftestblog1.wordpress.com/wp-content/uploads/2007/09/awefasdfawef.jpg';
         $response = wp_remote_head( $url );
 
@@ -67 +67 @@
         // Should return the same headers as a HEAD request.
         $this->assertSame( 'image/jpeg', $headers['Content-Type'] );
-        $this->assertSame( '40148', $headers['Content-Length'] );
+        $this->assertSame( '31325', $headers['Content-Length'] );
         $this->assertSame( 200, wp_remote_retrieve_response_code( $response ) );
     }
@@ -88 +88 @@
         // Should return the same headers as a HEAD request.
         $this->assertSame( 'image/jpeg', $headers['Content-Type'] );
-        $this->assertSame( '40148', $headers['Content-Length'] );
+        $this->assertSame( '31325', $headers['Content-Length'] );
         $this->assertSame( 200, wp_remote_retrieve_response_code( $response ) );
     }