Unit test for wp_get_http_headers

[borgesbruno]

I've wrote some tests to improve the code coverage, this special one needs to do an external request to get the headers and check whether is a valid or not response header.

This is the second ticket regarded about this duty, the first one is in #37073, following the @rmccue recommendation I've created a new ticket only with the tests

[jipmoors]

I have created a new patch which has unit tests instead of acceptance tests, but I'm having some problems uploading it due to some OperationalError: database is locked issue.

[jipmoors]

Unit tests instead of acceptance tests

[swissspidy]

@jipmoors That's a known issue due to some changes to the trac database yesterday. You should be able to upload it soon-ish.

[jipmoors]

These acceptance and unit tests can be complementary.

I can understand internals would be changed at some point so unit tests would fail, so I am thinking about writing unit tests to cover the WP_Http class, which is used to supply the actual fetching functionality.

[borgesbruno]

This is also a good approach @jipmoors use a mock instead of a real http request, like you said can be complementary

[ocean90]

In 37907:

HTTP: Add unit tests for wp_get_http_headers() and wp_remote_retrieve_headers().

Props borgesbruno, jipmoors.
Fixes #37090.