mirror of
https://github.com/gosticks/wordpress-develop.git
synced 2025-10-16 12:05:38 +00:00
This is a major release and contains breaking changes. Most important changes to be aware of for this release: * All code is now namespaced. Though there is a full backward compatibility layer available and the old class names are still supported, using them will generate a deprecation notice (which can be silenced by plugins if they'd need to support multiple WP versions). See the [https://requests.ryanmccue.info/docs/upgrading.html upgrade guide] for more details. * A lot of classes have been marked `final`. This should generally not affect userland code as care has been taken to not apply the `final` keyword to classes which are known to be extended in userland code. * Extensive input validation has been added to Requests. When Requests is used as documented though, this will be unnoticable. * A new `WpOrg\Requests\Requests::has_capabilities()` method has been introduced which can be used to address #37708. * A new `WpOrg\Requests\Response::decode_body()` method has been introduced which may be usable to simplify some of the WP native wrapper code. * Remaining PHP 8.0 compatibility fixed (support for named parameters). * PHP 8.1 compatibility. Release notes: https://github.com/WordPress/Requests/releases/tag/v2.0.0 For a full list of changes in this update, see the Requests GitHub: https://github.com/WordPress/Requests/compare/v1.8.1...v2.0.0 This commit also resolves 2 blocking issues which previously caused the revert of [52244]: * New Requests files are loaded into `wp-includes/Requests/src/`, matching the location of the library. In doing so, filesystems that are case-insensitive are not impacted (see #54582). * Preload: During a Core update, the old Requests files are preloaded into memory before the update deletes the files. Preloading avoids fatal errors noted in #54562. Follow-up to [50842], [51078], [52244], [52315], [52327], [52328]. Props jrf, schlessera, datagutten, wojsmol, dustinrue, soulseekah, szepeviktor. costdev, sergeybiryukov, peterwilsoncc, ironprogrammer, antonvlasenko, hellofromTonya, swissspidy, dd32, azaozz, TobiasBg, audrasjb. Fixes #54504. See #54582, #54562. git-svn-id: https://develop.svn.wordpress.org/trunk@54997 602fd350-edb4-49c9-b593-d223f7449a82
122 lines
3.3 KiB
PHP
122 lines
3.3 KiB
PHP
<?php
|
|
/**
|
|
* Unit tests for methods in `WP_SimplePie_File`.
|
|
*
|
|
* @package WordPress
|
|
* @subpackage UnitTests
|
|
* @since 5.6.1
|
|
*/
|
|
|
|
/**
|
|
* Tests the `WP_SimplePie_File` class.
|
|
*
|
|
* @group feed
|
|
* @group wp-simplepie-file
|
|
*
|
|
* @since 5.6.1
|
|
*/
|
|
class Tests_Feed_wpSimplePieFile extends WP_UnitTestCase {
|
|
public static function set_up_before_class() {
|
|
parent::set_up_before_class();
|
|
|
|
require_once ABSPATH . 'wp-includes/class-simplepie.php';
|
|
require_once ABSPATH . 'wp-includes/class-wp-simplepie-file.php';
|
|
}
|
|
|
|
/**
|
|
* Test that single- and multiple-value headers are parsed in the way that SimplePie expects.
|
|
*
|
|
* @dataProvider data_header_parsing
|
|
*
|
|
* @covers WP_SimplePie_File::__construct
|
|
*
|
|
* @since 5.6.1
|
|
*
|
|
* @ticket 51056
|
|
*/
|
|
public function test_header_parsing( $callback, $header_field, $expected ) {
|
|
add_filter( 'pre_http_request', array( $this, $callback ) );
|
|
|
|
$file = new WP_SimplePie_File( 'https://wordpress.org/news/feed/' );
|
|
|
|
$this->assertSame( $expected, $file->headers[ $header_field ] );
|
|
}
|
|
|
|
/**
|
|
* Provide test cases for `test_header_parsing()`.
|
|
*
|
|
* @return array
|
|
*/
|
|
public function data_header_parsing() {
|
|
return array(
|
|
'single content type header works' => array(
|
|
'mocked_response_single_header_values',
|
|
'content-type',
|
|
'application/rss+xml; charset=UTF-8',
|
|
),
|
|
|
|
'single generic header works' => array(
|
|
'mocked_response_single_header_values',
|
|
'link',
|
|
'<https://wordpress.org/news/wp-json/>; rel="https://api.w.org/"',
|
|
),
|
|
|
|
'only the final content-type header should be used' => array(
|
|
'mocked_response_multiple_header_values',
|
|
'content-type',
|
|
'application/rss+xml; charset=UTF-8',
|
|
),
|
|
|
|
'multiple generic header values should be merged into a comma separated string' => array(
|
|
'mocked_response_multiple_header_values',
|
|
'link',
|
|
'<https://wordpress.org/news/wp-json/>; rel="https://api.w.org/", <https://wordpress.org/news/wp/v2/categories/3>; rel="alternate"; type="application/json"',
|
|
),
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Mock a feed HTTP response where headers only have one value.
|
|
*/
|
|
public function mocked_response_single_header_values() {
|
|
$single_value_headers = array(
|
|
'content-type' => 'application/rss+xml; charset=UTF-8',
|
|
'link' => '<https://wordpress.org/news/wp-json/>; rel="https://api.w.org/"',
|
|
);
|
|
|
|
return array(
|
|
'headers' => new WpOrg\Requests\Utility\CaseInsensitiveDictionary( $single_value_headers ),
|
|
'body' => file_get_contents( DIR_TESTDATA . '/feed/wordpress-org-news.xml' ),
|
|
'response' => array(
|
|
'code' => 200,
|
|
'message' => 'OK',
|
|
),
|
|
'cookies' => array(),
|
|
'filename' => null,
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Mock a feed HTTP response where headers have multiple values.
|
|
*/
|
|
public function mocked_response_multiple_header_values() {
|
|
$response = $this->mocked_response_single_header_values();
|
|
|
|
$multiple_value_headers = array(
|
|
'content-type' => array(
|
|
'application/rss+xml; charset=ISO-8859-2',
|
|
'application/rss+xml; charset=UTF-8',
|
|
),
|
|
|
|
'link' => array(
|
|
'<https://wordpress.org/news/wp-json/>; rel="https://api.w.org/"',
|
|
'<https://wordpress.org/news/wp/v2/categories/3>; rel="alternate"; type="application/json"',
|
|
),
|
|
);
|
|
|
|
$response['headers'] = new WpOrg\Requests\Utility\CaseInsensitiveDictionary( $multiple_value_headers );
|
|
|
|
return $response;
|
|
}
|
|
}
|