External HTTP test failures

[SergeyBiryukov]

Some external HTTP tests ​have recently started failing on all core commits

There were 2 failures:

1) Tests_HTTP_curl::test_multiple_location_headers
Failed asserting that 'http://api.wordpress.org/core/tests/1.0/redirection.php?multiple-location-headers=1&redirected=one' is of type "array".

/var/www/tests/phpunit/tests/http/base.php:453
phpvfscomposer:///var/www/vendor/phpunit/phpunit/phpunit:60

2) Tests_HTTP_streams::test_multiple_location_headers
Failed asserting that 'http://api.wordpress.org/core/tests/1.0/redirection.php?multiple-location-headers=1&redirected=one' is of type "array".

/var/www/tests/phpunit/tests/http/base.php:453
phpvfscomposer:///var/www/vendor/phpunit/phpunit/phpunit:60

The tests send a request to ​http://api.wordpress.org/core/tests/1.0/redirection.php and try to verify that multiple redirects are handled correctly. Some history here:

• [342/tests] added the initial tests.
• [874/tests] switch to running the redirection script from api.wordpress.org.
• [1329/tests] introduced the affected test.

I don't see any recent core changes that could cause those failures. The first failure was on [54949]. It appears that something has changed on the dotorg side that could have affected this, might have to do with updating nginx to 1.23.2 around the time of the first failure.

When running the tests locally, it looks like the redirection script on api.wordpress.org no longer sends the second header it's supposed to send:

if ( ! isset( $_GET['redirected'] ) ) {
	header( "Location: $url?multiple-location-headers=1&redirected=one", false );
	header( "Location: $url?multiple-location-headers=1&redirected=two", false );
	exit;
}

This assertion fails because the result is now a string with only the first redirect, not an array with both of them:

$this->assertIsArray( wp_remote_retrieve_header( $res, 'location' ) );
...
Failed asserting that 'http://api.wordpress.org/core/tests/1.0/redirection.php?multiple-location-headers=1&redirected=one' is of type "array".

I have confirmed that the redirection script returns multiple Location headers when run locally, but only one on api.wordpress.org. I suppose we can disable the affected test for now, but it will also fail on older branches, so the disabling will need to be backported all the way to 4.1.

[costdev]

As we don't have confirmation on the exact cause yet, disabling the affected test seems appropriate so that builds/PRs pass CI.

Would it be possible to disable the affected test on trunk for now, and leave it a few days or so before backporting to the older branches in case we get a fix on the dotorg side?

[costdev]

Disable test_multiple_location_headers().

[SergeyBiryukov]

Replying to costdev:

Would it be possible to disable the affected test on trunk for now, and leave it a few days or so before backporting to the older branches in case we get a fix on the dotorg side?

Yes, that makes sense to me. Thanks for the patch!

[SergeyBiryukov]

In 54955:

Tests: Temporarily disable a WP_Http test for multiple Location headers.

The test verifies that WP_Http::handle_redirects() uses the last specified URL as the redirect location if multiple Location headers are specified in the response.

It appears that the redirection script on api.wordpress.org no longer sends the expected two Location headers, causing the test to fail.

While the exact cause is being investigated and confirmed, this commit disables the affected test for now, so that other commits and PRs are not blocked by these failures.

Follow-up to [342/tests], [874/tests], [1329/tests], [24846].

Props costdev, SergeyBiryukov.
See #57306.

[peterwilsoncc]

Is it possible to test the method directly rather than rely on wordpress.org?

It's pretty rough but I am thinking of something like this with the test moved out of the external-http group.

<?php
public function test_multiple_location_headers() {
        // Filter the response made by WP_HTTP::handle_redirects().
        add_filter(
                'pre_http_request',
                function( $response, $args, $url ) {
                        // Assert the redirect URL is correct.
                        $this->assertSame(
                                $url,
                                'http://api.wordpress.org/core/tests/1.0/redirection.php?multiple-location-headers=1&redirected=two'
                        );

                        return array(
                                'headers' => array(),
                                'body'    => 'PASS',
                                'response' => array(
                                        'code' => 200,
                                        'message' => 'OK',
                                ),
                                'cookies' => array(),
                                'filename' => null,
                        );
                },
                10,
                3
        );

        $args = array(
                'timeout'      => 30,
                '_redirection' => 3,
                'redirection'  => 2,
                'method'       => 'GET',
        );

        $headers = array(
                'server' => 'nginx',
                'date' => 'Sun, 11 Dec 2022 23:11:22 GMT',
                'content-type' => 'text/html; charset=utf-8',
                'location' => array(
                        'http://api.wordpress.org/core/tests/1.0/redirection.php?multiple-location-headers=1&redirected=one',
                        'http://api.wordpress.org/core/tests/1.0/redirection.php?multiple-location-headers=1&redirected=two',
                ),
        );

        // Test the tests: ensure multiple locations are passed to WP_HTTP::handle_redirects().
        $this->assertIsArray( $headers['location'] );
        $this->assertCount( 2, $headers['location'] );

        $response = WP_HTTP::handle_redirects(
                'http://api.wordpress.org/core/tests/1.0/redirection.php?multiple-location-headers=1',
                $args,
                array(
                        'headers'  => $headers,
                        'body'     => '',
                        'cookies'  => array (),
                        'filename' => NULL,
                        'response' => array (
                                'code' => 302,
                                'message' => 'Found',
                        ),
                )
        );
        $this->assertSame( 'PASS', wp_remote_retrieve_body( $response ) );
}

It would still need to be backported.

[dd32]

Replying to peterwilsoncc:

Is it possible to test the method directly rather than rely on wordpress.org?

This would be my preference too, even if it has to be back-ported.

As for the cause of the API starting to return differently, this was likely caused by the upgrade of nginx as noted, the ​nginx changelog lists multiple header-combining changes in 1.23.0.

https://core.trac.wordpress.org/ticket/57306

[peterwilsoncc]

In the ​linked pull request

• test moved to tests/phpunit/tests/http/http.php
• Largely the code above with a few coding standards fixes
• URLs changed to be more readable given the filter preempts the HTTP request
• Added an assertion to ensure the preempting filter runs
• It now only covers the WP_Http method, it does not cover wp_remote_verb function independently.

It looks like a few paths in the method are currently untested but I've not concerned myself with that as the purpose of the PR is to get the exiting test passing and validating the multiple location headers use-case without needing an external HTTP request.

[SergeyBiryukov]

Replying to peterwilsoncc:

Is it possible to test the method directly rather than rely on wordpress.org?

Yeah, that was my thinking too. Thanks for the PR!

Replying to dd32:

As for the cause of the API starting to return differently, this was likely caused by the upgrade of nginx as noted, the ​nginx changelog lists multiple header-combining changes in 1.23.0.

Indeed, this one appears to be the most relevant here:

Changes with nginx 1.23.0                                        21 Jun 2022
    ...
    *) Change: now nginx combines arbitrary header lines with identical
       names when sending to FastCGI, SCGI, and uwsgi backends, in the
       $r->header_in() method of the ngx_http_perl_module, and during lookup
       of the "$http_...", "$sent_http_...", "$sent_trailer_...",
       "$upstream_http_...", and "$upstream_trailer_..." variables.

[peterwilsoncc]

In 54968:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

[peterwilsoncc]

Reopening for backporting to all the branches.

Merged in https://core.trac.wordpress.org/changeset/54968 / ​https://github.com/WordPress/wordpress-develop/commit/4b4faae6cf0b1730cddc8a758c5bd281cbf3fa41

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php
# tests/phpunit/tests/http/http.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php
# tests/phpunit/tests/http/http.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php
# tests/phpunit/tests/http/http.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php
# tests/phpunit/tests/http/http.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php
# tests/phpunit/tests/http/http.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php
# tests/phpunit/tests/http/http.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php
# tests/phpunit/tests/http/http.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php

Trac ticket:

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Fixes #57306.
See #56793.

git-svn-id: ​https://develop.svn.wordpress.org/trunk@54968 602fd350-edb4-49c9-b593-d223f7449a82

# Conflicts:
# tests/phpunit/tests/http/base.php

Trac ticket:

Merged to 6.1.

[peterwilsoncc]

In 54974:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 6.1 branch.
Fixes #57306.
See #56793.

Merged to 6.0.

Merged to 5.9.

[peterwilsoncc]

In 54976:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.9 branch.
Fixes #57306.
See #56793.

Merged to 5.8.

[peterwilsoncc]

In 54977:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.8 branch.
Fixes #57306.
See #56793.

Merged to 5.7.

[peterwilsoncc]

In 54978:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.7 branch.
Fixes #57306.
See #56793.

Merged to 5.6.

[peterwilsoncc]

In 54979:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.6 branch.
Fixes #57306.
See #56793.

Merged to 5.5.

[peterwilsoncc]

In 54980:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.5 branch.
Fixes #57306.
See #56793.

Merged to 5.4.

[peterwilsoncc]

In 54981:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.4 branch.
Fixes #57306.
See #56793.

Merged to 5.3.

[peterwilsoncc]

In 54982:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.3 branch.
Fixes #57306.
See #56793.

Merged to 5.2.

[peterwilsoncc]

In 54983:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.2 branch.
Fixes #57306.
See #56793.

[peterwilsoncc]

In 54984:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.1 branch.
Fixes #57306.
See #56793.

[peterwilsoncc]

In 54985:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 5.0 branch.
Fixes #57306.
See #56793.

[peterwilsoncc]

In 54986:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.9 branch.
Fixes #57306.
See #56793.

[peterwilsoncc]

In 54987:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.8 branch.
Fixes #57306.
See #56793.

[peterwilsoncc]

In 54988:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.7 branch.
Fixes #57306.
See #56793.

[peterwilsoncc]

In 54989:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.6 branch.
Fixes #57306.
See #56793.

[peterwilsoncc]

In 54990:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.5 branch.
Fixes #57306.
See #56793.

[peterwilsoncc]

In 54991:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.4 branch.
Fixes #57306.
See #56793.

Merged to 4.3.

[peterwilsoncc]

In 54992:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.3 branch.
Fixes #57306.
See #56793.

Merged to 4.2.

[peterwilsoncc]

In 54993:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.2 branch.
Fixes #57306.
See #56793.

Merged to 4.1.

[peterwilsoncc]

In 54994:

Built/Test tools, HTTP API: Refactor test for multiple location headers.

Remove wordpress.org as an external dependency testing WP_HTTP::handle_redirects().

This refactors and reenables an existing test to call the WP_HTTP::handle_redirects() method directly with a mocked array of HTTP headers containing multiple location headers.

The test is moved from the external-http group to the http test group as it no longer makes an HTTP request.

Follow up to [54955].

Props SergeyBiryukov, dd32, peterwilsoncc.
Merges [54968] to the 4.1 branch.
Fixes #57306.
See #56793.