Index: tests/phpunit/tests/actions.php =================================================================== --- tests/phpunit/tests/actions.php (revision 48816) +++ tests/phpunit/tests/actions.php (working copy) @@ -7,6 +7,10 @@ */ class Tests_Actions extends WP_UnitTestCase { + /** + * + * @covers ::do_action + */ function test_simple_action() { $a = new MockAction(); $tag = __FUNCTION__; @@ -24,6 +28,9 @@ $this->assertEquals( array( '' ), $args ); } + /** + * @covers ::remove_action + */ function test_remove_action() { $a = new MockAction(); $tag = __FUNCTION__; @@ -43,6 +50,9 @@ } + /** + * @covers ::has_action + */ function test_has_action() { $tag = __FUNCTION__; $func = __FUNCTION__ . '_func'; @@ -57,7 +67,11 @@ $this->assertFalse( has_action( $tag ) ); } - // One tag with multiple actions. + /** + * One tag with multiple actions. + * + * @covers ::add_action + */ function test_multiple_actions() { $a1 = new MockAction(); $a2 = new MockAction(); @@ -74,6 +88,11 @@ $this->assertEquals( 1, $a2->get_call_count() ); } + /** + * One tag with multiple actions. + * + * @covers ::do_action + */ function test_action_args_1() { $a = new MockAction(); $tag = __FUNCTION__; @@ -89,6 +108,11 @@ $this->assertEquals( array( $val ), array_pop( $argsvar ) ); } + /** + * One tag with multiple actions. + * + * @covers ::do_action + */ function test_action_args_2() { $a1 = new MockAction(); $a2 = new MockAction(); @@ -120,6 +144,8 @@ * * @see https://core.trac.wordpress.org/ticket/17817#comment:72 * @ticket 17817 + * + * @covers ::do_action */ function test_action_args_3() { $a1 = new MockAction(); @@ -157,6 +183,8 @@ * Tests PHP 4 notation for calling actions while passing in an object by reference. * * @ticket 48312 + * + * @covers ::do_action */ function test_action_args_with_php4_syntax() { $a = new MockAction(); @@ -201,6 +229,9 @@ $this->assertEquals( $expected, $a->get_events() ); } + /** + * @covers ::did_action + */ function test_did_action() { $tag1 = 'action1'; $tag2 = 'action2'; @@ -222,6 +253,9 @@ } + /** + * @covers ::add_action + */ function test_all_action() { $a = new MockAction(); $tag1 = __FUNCTION__ . '_1'; @@ -246,6 +280,9 @@ } + /** + * @covers ::remove_action + */ function test_remove_all_action() { $a = new MockAction(); $tag = __FUNCTION__; @@ -266,6 +303,9 @@ $this->assertEquals( array( $tag ), $a->get_tags() ); } + /** + * @covers ::do_action_ref_array + */ function test_action_ref_array() { $obj = new stdClass(); $a = new MockAction(); @@ -284,6 +324,8 @@ /** * @ticket 11241 + * + * @covers ::add_action */ function test_action_keyed_array() { $a = new MockAction(); @@ -309,6 +351,9 @@ } + /** + * @covers ::do_action + */ function test_action_self_removal() { add_action( 'test_action_self_removal', array( $this, 'action_self_removal' ) ); do_action( 'test_action_self_removal' ); @@ -321,6 +366,8 @@ /** * @ticket 17817 + * + * @covers ::do_action */ function test_action_recursion() { $tag = __FUNCTION__; @@ -336,6 +383,9 @@ $this->assertEquals( 2, $b->get_call_count(), 'recursive actions should call callbacks with later priority' ); } + /** + * @covers ::do_action + */ function action_that_causes_recursion( $tag ) { static $recursing = false; if ( ! $recursing ) { @@ -348,6 +398,8 @@ /** * @ticket 9968 * @ticket 17817 + * + * @covers ::add_action */ function test_action_callback_manipulation_while_running() { $tag = __FUNCTION__; @@ -383,6 +435,8 @@ * * This specificaly addresses the concern raised at * https://core.trac.wordpress.org/ticket/17817#comment:52 + * + * @covers ::remove_filter */ function test_remove_anonymous_callback() { $tag = __FUNCTION__; @@ -416,6 +470,8 @@ * Test the ArrayAccess methods of WP_Hook * * @ticket 17817 + * + * @covers ::has_action */ function test_array_access_of_wp_filter_global() { global $wp_filter; @@ -442,6 +498,8 @@ * Make sure current_action() behaves as current_filter() * * @ticket 14994 + * + * @covers ::current_action */ function test_current_action() { global $wp_current_filter; @@ -453,6 +511,8 @@ /** * @ticket 14994 + * + * @covers ::doing_filter */ function test_doing_filter() { global $wp_current_filter; @@ -472,6 +532,8 @@ /** * @ticket 14994 + * + * @covers ::doing_filter */ function test_doing_action() { global $wp_current_filter; @@ -491,6 +553,8 @@ /** * @ticket 14994 + * + * @covers ::doing_filter */ function test_doing_filter_real() { $this->assertFalse( doing_filter() ); // No filter is passed in, and no filter is being processed. @@ -509,6 +573,9 @@ $this->assertFalse( doing_filter( 'testing' ) ); // No longer doing this filter. } + /** + * @covers ::doing_filter + */ function apply_testing_filter() { $this->apply_testing_filter = true; @@ -530,6 +597,9 @@ $this->assertFalse( doing_filter( 'testing_nested' ) ); } + /** + * @covers ::doing_filter + */ function apply_testing_nested_filter() { $this->apply_testing_nested_filter = true; $this->assertTrue( doing_filter() ); @@ -541,6 +611,8 @@ /** * @ticket 10441 * @expectedDeprecated tests_do_action_deprecated + * + * @covers ::do_action_deprecated */ public function test_do_action_deprecated() { $p = new WP_Post( (object) array( 'post_title' => 'Foo' ) ); @@ -559,6 +631,8 @@ /** * @ticket 10441 * @expectedDeprecated tests_do_action_deprecated + * + * @covers ::do_action_deprecated */ public function test_do_action_deprecated_with_multiple_params() { $p1 = new WP_Post( (object) array( 'post_title' => 'Foo1' ) ); Index: tests/phpunit/tests/adminbar.php =================================================================== --- tests/phpunit/tests/adminbar.php (revision 48816) +++ tests/phpunit/tests/adminbar.php (working copy) @@ -31,6 +31,8 @@ /** * @ticket 21117 + * + * @covers WP_Admin_Bar::get_nodes */ function test_content_post_type() { wp_set_current_user( self::$editor_id ); @@ -50,6 +52,8 @@ /** * @ticket 21117 + * + * @covers WP_Admin_Bar::add_node */ function test_merging_existing_meta_values() { wp_set_current_user( self::$editor_id ); @@ -86,6 +90,8 @@ /** * @ticket 25162 * @group ms-excluded + * + * @covers ::_wp_admin_bar_init */ public function test_admin_bar_contains_correct_links_for_users_with_no_role() { $this->assertFalse( user_can( self::$no_role_id, 'read' ) ); @@ -111,6 +117,8 @@ /** * @ticket 25162 * @group ms-excluded + * + * @covers ::_wp_admin_bar_init */ public function test_admin_bar_contains_correct_links_for_users_with_role() { $this->assertTrue( user_can( self::$editor_id, 'read' ) ); @@ -139,6 +147,8 @@ * @ticket 25162 * @group multisite * @group ms-required + * + * @covers ::_wp_admin_bar_init */ public function test_admin_bar_contains_correct_links_for_users_with_no_role_on_blog() { $blog_id = self::factory()->blog->create( @@ -188,6 +198,8 @@ * @ticket 25162 * @group multisite * @group ms-required + * + * @covers ::_wp_admin_bar_init */ public function test_admin_bar_contains_correct_links_for_users_with_no_role_on_network() { $this->assertTrue( user_can( self::$admin_id, 'read' ) ); @@ -239,6 +251,9 @@ restore_current_blog(); } + /** + * @return mixed + */ protected function get_standard_admin_bar() { global $wp_admin_bar; @@ -259,6 +274,8 @@ * * @param array $node_data The data for a node, passed to `WP_Admin_Bar::add_node()`. * @param string $expected_html The expected HTML when admin menu is rendered. + * + * @covers WP_Admin_Bar::add_node */ public function test_admin_bar_with_tabindex_meta( $node_data, $expected_html ) { $admin_bar = new WP_Admin_Bar(); @@ -347,6 +364,9 @@ /** * @ticket 22247 + * + * @covers ::_wp_admin_bar_init + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_has_edit_link_for_existing_posts() { wp_set_current_user( self::$editor_id ); @@ -374,6 +394,8 @@ /** * @ticket 22247 + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_has_no_edit_link_for_non_existing_posts() { wp_set_current_user( self::$editor_id ); @@ -393,6 +415,8 @@ /** * @ticket 34113 + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_has_no_archives_link_if_no_static_front_page() { set_current_screen( 'edit-post' ); @@ -407,6 +431,8 @@ /** * @ticket 34113 + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_contains_view_archive_link_if_static_front_page() { update_option( 'show_on_front', 'page' ); @@ -422,6 +448,8 @@ /** * @ticket 34113 + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_has_no_archives_link_for_pages() { set_current_screen( 'edit-page' ); @@ -437,6 +465,8 @@ /** * @ticket 37949 * @group ms-excluded + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_contains_correct_about_link_for_users_with_role() { wp_set_current_user( self::$editor_id ); @@ -454,6 +484,8 @@ /** * @ticket 37949 * @group ms-excluded + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_contains_correct_about_link_for_users_with_no_role() { wp_set_current_user( self::$no_role_id ); @@ -473,6 +505,8 @@ * @ticket 37949 * @group multisite * @group ms-required + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_contains_correct_about_link_for_users_with_no_role_in_multisite() { // User is not a member of a site. @@ -492,6 +526,8 @@ /** * @ticket 34113 + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_has_no_archives_link_for_non_public_cpt() { register_post_type( @@ -516,6 +552,8 @@ /** * @ticket 34113 + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_has_no_archives_link_for_cpt_without_archive() { register_post_type( @@ -540,6 +578,8 @@ /** * @ticket 34113 + * + * @covers WP_Admin_Bar::get_node */ public function test_admin_bar_has_no_archives_link_for_cpt_not_shown_in_admin_bar() { register_post_type( @@ -596,6 +636,8 @@ /** * @ticket 39252 + * + * @covers WP_Admin_Bar::get_node */ public function test_new_user_link_exists_for_user_with_create_users() { wp_set_current_user( self::$admin_id ); @@ -615,6 +657,8 @@ /** * @ticket 39252 + * + * @covers WP_Admin_Bar::get_node */ public function test_new_user_link_existence_for_user_with_promote_users() { wp_set_current_user( self::$admin_id ); @@ -638,6 +682,8 @@ /** * @ticket 39252 + * + * @covers WP_Admin_Bar::get_node */ public function test_new_user_link_does_not_exist_for_user_without_create_or_promote_users() { wp_set_current_user( self::$admin_id ); @@ -657,6 +703,8 @@ /** * @ticket 30937 * @covers ::wp_admin_bar_customize_menu + * + * @covers WP_Admin_Bar::get_node */ public function test_customize_link() { global $wp_customize; @@ -694,6 +742,8 @@ /** * @ticket 39082 * @group ms-required + * + * @covers WP_Admin_Bar::get_nodes */ public function test_my_sites_network_menu_for_regular_user() { wp_set_current_user( self::$editor_id ); @@ -709,6 +759,8 @@ /** * @ticket 39082 * @group ms-required + * + * @covers WP_Admin_Bar::get_nodes */ public function test_my_sites_network_menu_for_super_admin() { wp_set_current_user( self::$editor_id ); @@ -726,6 +778,8 @@ /** * @ticket 39082 * @group ms-required + * + * @covers WP_Admin_Bar::get_nodes */ public function test_my_sites_network_menu_for_regular_user_with_network_caps() { global $current_user; Index: tests/phpunit/tests/auth.php =================================================================== --- tests/phpunit/tests/auth.php (revision 48816) +++ tests/phpunit/tests/auth.php (working copy) @@ -35,11 +35,17 @@ wp_set_current_user( self::$user_id ); } + /** + * @covers ::wp_validate_auth_cookie + */ function test_auth_cookie_valid() { $cookie = wp_generate_auth_cookie( self::$user_id, time() + 3600, 'auth' ); $this->assertEquals( self::$user_id, wp_validate_auth_cookie( $cookie, 'auth' ) ); } + /** + * @covers ::wp_validate_auth_cookie + */ function test_auth_cookie_invalid() { // 3600 or less and +3600 may occur in wp_validate_auth_cookie(), // as an ajax test may have defined DOING_AJAX, failing the test. @@ -56,6 +62,9 @@ $this->assertEquals( false, wp_validate_auth_cookie( self::$user_id, 'auth' ), 'altered cookie' ); } + /** + * @covers ::wp_validate_auth_cookie + */ function test_auth_cookie_scheme() { // Arbitrary scheme name. $cookie = wp_generate_auth_cookie( self::$user_id, time() + 3600, 'foo' ); @@ -68,6 +77,8 @@ /** * @ticket 23494 + * + * @covers ::wp_authenticate */ function test_password_trimming() { $passwords_to_test = array( @@ -93,11 +104,13 @@ * wp_hash_password function * * @ticket 24973 + * + * @covers ::wp_hash_password */ function test_wp_hash_password_trimming() { $password = ' pass with leading whitespace'; - $this->assertTrue( wp_check_password( 'pass with leading whitespace', wp_hash_password( $password ) ) ); + $this->assertTrue( wp_check_password( 'pass with leading whitespace', ( $password ) ) ); $password = 'pass with trailing whitespace '; $this->assertTrue( wp_check_password( 'pass with trailing whitespace', wp_hash_password( $password ) ) ); @@ -114,6 +127,8 @@ /** * @ticket 29217 + * + * @covers ::wp_verify_nonce */ function test_wp_verify_nonce_with_empty_arg() { $this->assertFalse( wp_verify_nonce( '' ) ); @@ -122,6 +137,8 @@ /** * @ticket 29542 + * + * @covers ::wp_verify_nonce */ function test_wp_verify_nonce_with_integer_arg() { $this->assertFalse( wp_verify_nonce( 1 ) ); @@ -129,6 +146,8 @@ /** * @ticket 24030 + * + * @covers ::wp_verify_nonce */ function test_wp_nonce_verify_failed() { $nonce = substr( md5( uniqid() ), 0, 10 ); @@ -141,6 +160,8 @@ /** * @ticket 24030 + * + * @covers ::wp_verify_nonce */ function test_wp_nonce_verify_success() { $nonce = wp_create_nonce( 'nonce_test_action' ); @@ -153,6 +174,8 @@ /** * @ticket 36361 + * + * @covers ::check_admin_referer */ public function test_check_admin_referer_with_no_action_triggers_doing_it_wrong() { $this->setExpectedIncorrectUsage( 'check_admin_referer' ); @@ -165,6 +188,9 @@ unset( $_REQUEST['_wpnonce'] ); } + /** + * @covers ::check_admin_referer + */ public function test_check_admin_referer_with_default_action_as_string_not_doing_it_wrong() { // A valid nonce needs to be set so the check doesn't die(). $_REQUEST['_wpnonce'] = wp_create_nonce( '-1' ); @@ -176,6 +202,10 @@ /** * @ticket 36361 + /** + * + * @c * + * @covers ::check_ajax_referer */ public function test_check_ajax_referer_with_no_action_triggers_doing_it_wrong() { $this->setExpectedIncorrectUsage( 'check_ajax_referer' ); @@ -188,6 +218,9 @@ unset( $_REQUEST['_wpnonce'] ); } + /** + * @covers ::wp_authenticate + */ function test_password_length_limit() { $limit = str_repeat( 'a', 4096 ); @@ -237,6 +270,8 @@ /** * @ticket 45746 + * + * @covers ::check_password_reset_key */ function test_user_activation_key_is_saved() { $user = get_userdata( $this->user->ID ); @@ -251,6 +286,8 @@ /** * @ticket 32429 + * + * @covers ::check_password_reset_key */ function test_user_activation_key_is_checked() { global $wpdb; @@ -289,6 +326,8 @@ /** * @ticket 32429 + * + * @covers ::check_password_reset_key */ function test_expired_user_activation_key_is_rejected() { global $wpdb; @@ -312,6 +351,8 @@ /** * @ticket 32429 + * + * @covers ::check_password_reset_key */ function test_empty_user_activation_key_fails_key_check() { // An empty user_activation_key should not allow any key to be accepted. @@ -325,6 +366,8 @@ /** * @ticket 32429 + * + * @covers ::check_password_reset_key */ function test_legacy_user_activation_key_is_rejected() { global $wpdb; @@ -355,6 +398,8 @@ /** * @ticket 32429 * @ticket 24783 + * + * @covers ::check_password_reset_key */ function test_plaintext_user_activation_key_is_rejected() { global $wpdb; @@ -386,6 +431,8 @@ * Ensure users can log in using both their username and their email address. * * @ticket 9568 + * + * @covers ::wp_authenticate */ public function test_log_in_using_email() { $user_args = array( @@ -401,6 +448,8 @@ /** * @ticket 38744 + * + * @covers ::wp_signon */ public function test_wp_signon_using_email_with_an_apostrophe() { $user_args = array( Index: tests/phpunit/tests/avatar.php =================================================================== --- tests/phpunit/tests/avatar.php (revision 48816) +++ tests/phpunit/tests/avatar.php (working copy) @@ -8,6 +8,8 @@ class Tests_Avatar extends WP_UnitTestCase { /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_get_avatar_url_gravatar_url() { $url = get_avatar_url( 1 ); @@ -16,6 +18,8 @@ /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_get_avatar_url_size() { $url = get_avatar_url( 1 ); @@ -28,6 +32,8 @@ /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_get_avatar_url_default() { $url = get_avatar_url( 1 ); @@ -45,6 +51,8 @@ /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_get_avatar_url_rating() { $url = get_avatar_url( 1 ); @@ -57,6 +65,8 @@ /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_get_avatar_url_scheme() { $url = get_avatar_url( 1 ); @@ -73,6 +83,8 @@ /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_get_avatar_url_user() { $url = get_avatar_url( 1 ); @@ -106,6 +118,8 @@ protected $fake_url; /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_pre_get_avatar_url_filter() { $this->fake_url = 'haha wat'; @@ -116,6 +130,7 @@ $this->assertEquals( $url, $this->fake_url ); } + public function pre_get_avatar_url_filter( $args ) { $args['url'] = $this->fake_url; return $args; @@ -123,6 +138,8 @@ /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_get_avatar_url_filter() { $this->fake_url = 'omg lol'; @@ -139,6 +156,8 @@ /** * @ticket 21195 + * + * @covers ::get_avatar_url */ public function test_get_avatar_comment_types_filter() { $url = get_avatar_url( 1 ); @@ -167,11 +186,17 @@ return $comment_types; } + /** + * @covers ::get_avatar + */ public function test_get_avatar() { $img = get_avatar( 1 ); $this->assertEquals( preg_match( "|^[^$|", $img ), 1 ); } + /** + * @covers ::get_avatar + */ public function test_get_avatar_size() { $size = '100'; $img = get_avatar( 1, $size ); @@ -178,6 +203,9 @@ $this->assertEquals( preg_match( "|^assertEquals( preg_match( "|^$alt $class ) ); @@ -190,11 +221,17 @@ $this->assertEquals( preg_match( "|^ true ) ); $this->assertEquals( preg_match( "|^fake_img = 'YOU TOO?!'; @@ -226,6 +265,8 @@ /** * @ticket 21195 + * + * @covers ::get_avatar */ public function test_get_avatar_filter() { $this->fake_url = 'YA RLY'; @@ -244,6 +285,9 @@ * The `get_avatar_data()` function should return gravatar url when comment type allowed to retrieve avatars. * * @ticket 44033 + * + * @covers ::is_avatar_comment_type + * @covers ::get_avatar */ public function test_get_avatar_data_should_return_gravatar_url_when_input_avatar_comment_type() { $comment_type = 'comment'; @@ -264,6 +308,9 @@ * The `get_avatar_data()` function should return invalid url when comment type not allowed to retrieve avatars. * * @ticket 44033 + * + * @covers ::is_avatar_comment_type + * @covers ::get_avatar */ public function test_get_avatar_data_should_return_invalid_url_when_input_not_avatar_comment_type() { $comment_type = 'review'; Index: tests/phpunit/tests/basic.php =================================================================== --- tests/phpunit/tests/basic.php (revision 48816) +++ tests/phpunit/tests/basic.php (working copy) @@ -3,10 +3,15 @@ /** * just make sure the test framework is working * + * No Covers as this checks for content in files + * * @group testsuite */ class Tests_Basic extends WP_UnitTestCase { + /** + * @coversNothing + */ function test_license() { // This test is designed to only run on trunk/master. $this->skipOnAutomatedBranches(); @@ -17,6 +22,9 @@ $this->assertEquals( $this_year, trim( $matches[1] ), "license.txt's year needs to be updated to $this_year." ); } + /** + * @coversNothing + */ function test_security_md() { // This test is designed to only run on trunk/master. $this->skipOnAutomatedBranches(); @@ -28,6 +36,9 @@ $this->assertEquals( $latest_stable, trim( $matches[0] ), "SECURITY.md's version needs to be updated to $latest_stable." ); } + /** + * @coversNothing + */ function test_package_json() { $package_json = file_get_contents( dirname( ABSPATH ) . '/package.json' ); $package_json = json_decode( $package_json, true ); @@ -42,6 +53,8 @@ /** * @depends test_package_json + * + * @coversNothing */ function test_package_json_node_engine( $package_json ) { $this->assertArrayHasKey( 'engines', $package_json ); @@ -50,8 +63,11 @@ $this->assertRegExp( '~^=?\d+\.\d+\.\d+$~', $node, "package.json's node version cannot be a range." ); } - // Test some helper utility functions. - + /** + * Test some helper utility functions. + * + * @coversNothing + */ function test_strip_ws() { $this->assertEquals( '', strip_ws( '' ) ); $this->assertEquals( 'foo', strip_ws( 'foo' ) ); @@ -79,6 +95,9 @@ } + /** + * @coversNothing + */ function test_mask_input_value() { $in = <<Assign Authors Index: tests/phpunit/tests/cache.php =================================================================== --- tests/phpunit/tests/cache.php (revision 48816) +++ tests/phpunit/tests/cache.php (working copy) @@ -26,10 +26,18 @@ return $cache; } + /** + * + * @covers WP_Object_Cache::test_miss + */ function test_miss() { $this->assertEquals( null, $this->cache->get( 'test_miss' ) ); } + /** + * + * @covers WP_Object_Cache::add + */ function test_add_get() { $key = __FUNCTION__; $val = 'val'; @@ -38,6 +46,10 @@ $this->assertEquals( $val, $this->cache->get( $key ) ); } + /** + * + * @covers WP_Object_Cache::add + */ function test_add_get_0() { $key = __FUNCTION__; $val = 0; @@ -47,6 +59,10 @@ $this->assertEquals( $val, $this->cache->get( $key ) ); } + /** + * + * @covers WP_Object_Cache::add + */ function test_add_get_null() { $key = __FUNCTION__; $val = null; @@ -56,6 +72,10 @@ $this->assertEquals( '', $this->cache->get( $key ) ); } + /** + * + * @covers WP_Object_Cache::add + */ function test_add() { $key = __FUNCTION__; $val1 = 'val1'; @@ -69,6 +89,10 @@ $this->assertEquals( $val1, $this->cache->get( $key ) ); } + /** + * + * @covers WP_Object_Cache::replace + */ function test_replace() { $key = __FUNCTION__; $val = 'val1'; @@ -83,6 +107,10 @@ $this->assertEquals( $val2, $this->cache->get( $key ) ); } + /** + * + * @covers WP_Object_Cache::set + */ function test_set() { $key = __FUNCTION__; $val1 = 'val1'; @@ -96,6 +124,10 @@ $this->assertEquals( $val2, $this->cache->get( $key ) ); } + /** + * + * @covers WP_Object_Cache::flush + */ function test_flush() { global $_wp_using_ext_object_cache; @@ -114,7 +146,12 @@ $this->assertFalse( $this->cache->get( $key ) ); } - // Make sure objects are cloned going to and from the cache. + /** + * Make sure objects are cloned going to and from the cache. + * + * @covers WP_Object_Cache::set + * @covers WP_Object_Cache::get + */ function test_object_refs() { $key = __FUNCTION__ . '_1'; $object_a = new stdClass; @@ -137,6 +174,10 @@ $this->assertEquals( 'bravo', $object_a->foo ); } + /** + * + * @covers WP_Object_Cache::incr + */ function test_incr() { $key = __FUNCTION__; @@ -150,6 +191,10 @@ $this->assertEquals( 3, $this->cache->get( $key ) ); } + /** + * + * @covers ::wp_cache_incr + */ function test_wp_cache_incr() { $key = __FUNCTION__; @@ -163,6 +208,10 @@ $this->assertEquals( 3, wp_cache_get( $key ) ); } + /** + * + * @covers WP_Object_Cache::decr + */ function test_decr() { $key = __FUNCTION__; @@ -182,6 +231,8 @@ /** * @ticket 21327 + * + * @covers ::wp_cache_decr */ function test_wp_cache_decr() { $key = __FUNCTION__; @@ -200,6 +251,10 @@ $this->assertEquals( 0, wp_cache_get( $key ) ); } + /** + * + * @covers WP_Object_Cache::delete + */ function test_delete() { $key = __FUNCTION__; $val = 'val'; @@ -215,6 +270,12 @@ $this->assertFalse( $this->cache->delete( $key, 'default' ) ); } + /** + * + * @covers ::wp_cache_delete + * @covers ::wp_cache_get + * @covers ::wp_cache_set + */ function test_wp_cache_delete() { $key = __FUNCTION__; $val = 'val'; @@ -234,6 +295,10 @@ $this->assertFalse( wp_cache_delete( $key, 'default' ) ); } + /** + * + * @covers WP_Object_Cache::switch_to_blog + */ function test_switch_to_blog() { if ( ! method_exists( $this->cache, 'switch_to_blog' ) ) { return; @@ -280,6 +345,10 @@ $this->assertEquals( $val2, $this->cache->get( $key, 'global-cache-test' ) ); } + /** + * + * @covers ::wp_cache_init + */ function test_wp_cache_init() { $new_blank_cache_object = new WP_Object_Cache(); wp_cache_init(); @@ -294,6 +363,10 @@ } } + /** + * + * @covers ::wp_cache_replace + */ function test_wp_cache_replace() { $key = 'my-key'; $val1 = 'first-val'; @@ -318,6 +391,8 @@ /** * @ticket 20875 + * + * @covers ::wp_cache_get_multiple */ public function test_get_multiple() { wp_cache_set( 'foo1', 'bar', 'group1' ); Index: tests/phpunit/tests/canonical.php =================================================================== --- tests/phpunit/tests/canonical.php (revision 48816) +++ tests/phpunit/tests/canonical.php (working copy) @@ -17,6 +17,8 @@ /** * @dataProvider data_canonical + * + * @covers ::redirect_canonical */ function test_canonical( $test_url, $expected, $ticket = 0, $expected_doing_it_wrong = array() ) { @@ -232,6 +234,8 @@ /** * @ticket 16557 + * + * @covers ::redirect_guess_404_permalink */ public function test_do_redirect_guess_404_permalink() { // Test disable do_redirect_guess_404_permalink(). @@ -242,6 +246,8 @@ /** * @ticket 16557 + * + * @covers ::redirect_guess_404_permalink */ public function test_pre_redirect_guess_404_permalink() { // Test short-circuit filter. @@ -257,6 +263,8 @@ /** * @ticket 16557 + * + * @covers ::redirect_guess_404_permalink */ public function test_strict_redirect_guess_404_permalink() { $post = self::factory()->post->create( @@ -277,6 +285,8 @@ /** * @ticket 43745 + * + * @covers ::redirect_canonical */ public function test_utf8_query_keys_canonical() { $p = self::factory()->post->create( Index: tests/phpunit/tests/category.php =================================================================== --- tests/phpunit/tests/category.php (revision 48816) +++ tests/phpunit/tests/category.php (working copy) @@ -18,6 +18,8 @@ * Validate get_all_category_ids * * @expectedDeprecated get_all_category_ids + * + * @covers ::get_all_category_ids */ function test_get_all_category_ids() { // Ccreate categories. @@ -34,6 +36,8 @@ /** * Validate get_category_by_slug function + * + * @covers ::get_category_by_slug */ function test_get_category_by_slug() { @@ -64,6 +68,8 @@ /** * Validate _make_cat_compat function + * + * @covers ::_make_cat_compat */ function test__make_cat_compat() { @@ -141,6 +147,8 @@ /** * Validate get_cat_name function + * + * @covers ::get_cat_name */ function test_get_cat_name() { @@ -161,6 +169,8 @@ /** * Validate get_cat_name function + * + * @covers ::get_cat_ID */ function test_get_cat_ID() { @@ -181,6 +191,8 @@ /** * Validate get_category_by_path function + * + * @covers ::get_category_by_path */ function test_get_category_by_path() { Index: tests/phpunit/tests/comment-submission.php =================================================================== --- tests/phpunit/tests/comment-submission.php (revision 48816) +++ tests/phpunit/tests/comment-submission.php (working copy) @@ -39,6 +39,9 @@ require_once ABSPATH . WPINC . '/class-phpass.php'; } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_invalid_post_returns_error() { $error = 'comment_id_not_found'; @@ -55,6 +58,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_post_with_closed_comments_returns_error() { $error = 'comment_closed'; @@ -78,6 +84,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_trashed_post_returns_error() { $error = 'comment_on_trash'; @@ -99,6 +108,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_draft_post_returns_error() { $error = 'comment_on_draft'; @@ -124,6 +136,8 @@ /** * @ticket 39650 + * + * @covers ::wp_handle_comment_submission */ public function test_submitting_comment_to_draft_post_returns_error_message_for_user_with_correct_caps() { $error = 'comment_on_draft'; @@ -150,6 +164,9 @@ $this->assertNotEmpty( $comment->get_error_message() ); } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_scheduled_post_returns_error() { // Same error as commenting on a draft. @@ -176,6 +193,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_password_required_post_returns_error() { $error = 'comment_on_password_protected'; @@ -199,6 +219,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_password_protected_post_succeeds() { $password = 'password'; @@ -227,6 +250,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_valid_comment_as_logged_in_user_succeeds() { $user = self::factory()->user->create_and_get( @@ -254,6 +280,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_valid_comment_anonymously_succeeds() { $data = array( @@ -280,6 +309,8 @@ * wp_handle_comment_submission() expects un-slashed data. * * @group slashes + * + * @covers ::wp_handle_comment_submission */ public function test_submitting_comment_handles_slashes_correctly_handles_slashes() { @@ -300,6 +331,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_anonymously_to_private_post_returns_error() { $error = 'comment_id_not_found'; @@ -321,6 +355,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_as_logged_in_user_to_inaccessible_private_post_returns_error() { $error = 'comment_id_not_found'; @@ -351,6 +388,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_private_post_with_closed_comments_returns_correct_error() { $error = 'comment_id_not_found'; @@ -382,6 +422,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_own_private_post_succeeds() { wp_set_current_user( self::$author_id ); @@ -405,6 +448,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_to_accessible_private_post_succeeds() { wp_set_current_user( self::$editor_id ); @@ -428,6 +474,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_anonymous_user_cannot_comment_unfiltered_html() { $data = array( @@ -444,6 +493,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_unprivileged_user_cannot_comment_unfiltered_html() { wp_set_current_user( self::$author_id ); @@ -462,6 +514,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_unprivileged_user_cannot_comment_unfiltered_html_even_with_valid_nonce() { wp_set_current_user( self::$author_id ); @@ -486,6 +541,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_privileged_user_can_comment_unfiltered_html_with_valid_nonce() { $this->assertFalse( defined( 'DISALLOW_UNFILTERED_HTML' ) ); @@ -518,6 +576,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_privileged_user_cannot_comment_unfiltered_html_without_valid_nonce() { if ( is_multisite() ) { @@ -542,6 +603,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_as_anonymous_user_when_registration_required_returns_error() { $error = 'not_logged_in'; @@ -561,6 +625,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_with_no_name_when_name_email_required_returns_error() { $error = 'require_name_email'; @@ -582,6 +649,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_with_no_email_when_name_email_required_returns_error() { $error = 'require_name_email'; @@ -603,6 +673,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_with_invalid_email_when_name_email_required_returns_error() { $error = 'require_valid_email'; @@ -625,6 +698,9 @@ } + /** + * @covers ::wp_handle_comment_submission + */ public function test_submitting_comment_with_no_comment_content_returns_error() { $error = 'require_valid_comment'; @@ -644,6 +720,8 @@ /** * @ticket 10377 + * + * @covers ::wp_handle_comment_submission */ public function test_submitting_comment_with_content_too_long_returns_error() { $error = 'comment_content_column_length'; @@ -662,6 +740,8 @@ /** * @ticket 10377 + * + * @covers ::wp_handle_comment_submission */ public function test_submitting_comment_with_author_too_long_returns_error() { $error = 'comment_author_column_length'; @@ -680,6 +760,8 @@ /** * @ticket 10377 + * + * @covers ::wp_handle_comment_submission */ public function test_submitting_comment_with_email_too_long_returns_error() { $error = 'comment_author_email_column_length'; @@ -698,6 +780,8 @@ /** * @ticket 10377 + * + * @covers ::wp_handle_comment_submission */ public function test_submitting_comment_with_url_too_long_returns_error() { $error = 'comment_author_url_column_length'; @@ -717,6 +801,8 @@ /** * @ticket 49236 + * + * @covers ::wp_handle_comment_submission */ public function test_submitting_comment_with_empty_type_results_in_correct_type() { $data = array( @@ -736,6 +822,8 @@ /** * @ticket 49236 + * + * @covers ::wp_handle_comment_submission */ public function test_inserting_comment_with_empty_type_results_in_correct_type() { $data = array( @@ -756,6 +844,8 @@ /** * @ticket 34997 + * + * @covers ::wp_handle_comment_submission */ public function test_comment_submission_sends_all_expected_parameters_to_preprocess_comment_filter() { @@ -798,6 +888,8 @@ /** * @ticket 36901 + * + * @covers ::wp_handle_comment_submission */ public function test_submitting_duplicate_comments() { $data = array( @@ -814,6 +906,8 @@ /** * @ticket 36901 + * + * @covers ::wp_handle_comment_submission */ public function test_comments_flood() { $data = array( @@ -833,6 +927,8 @@ /** * @ticket 36901 + * + * @covers ::wp_handle_comment_submission */ public function test_comments_flood_user_is_admin() { $user = self::factory()->user->create_and_get( Index: tests/phpunit/tests/comment.php =================================================================== --- tests/phpunit/tests/comment.php (revision 48816) +++ tests/phpunit/tests/comment.php (working copy) @@ -30,6 +30,11 @@ ); } + /** + * + * + * @covers ::wp_update_comment + */ public function test_wp_update_comment() { $post = self::factory()->post->create_and_get( array( @@ -78,6 +83,8 @@ /** * @ticket 30627 + * + * @covers ::wp_update_comment */ public function test_wp_update_comment_updates_comment_type() { $comment_id = self::factory()->comment->create( array( 'comment_post_ID' => self::$post_id ) ); @@ -95,6 +102,8 @@ /** * @ticket 36784 + * + * @covers ::wp_update_comment */ public function test_wp_update_comment_updates_comment_meta() { $comment_id = self::factory()->comment->create( array( 'comment_post_ID' => self::$post_id ) ); @@ -114,6 +123,8 @@ /** * @ticket 30307 + * + * @covers ::wp_update_comment */ public function test_wp_update_comment_updates_user_id() { $comment_id = self::factory()->comment->create( array( 'comment_post_ID' => self::$post_id ) ); @@ -131,6 +142,8 @@ /** * @ticket 34954 + * + * @covers ::wp_update_comment */ public function test_wp_update_comment_with_no_post_id() { $comment_id = self::factory()->comment->create( array( 'comment_post_ID' => 0 ) ); @@ -152,6 +165,8 @@ /** * @ticket 39732 + * + * @covers ::wp_update_comment */ public function test_wp_update_comment_returns_false_for_invalid_comment_or_post_id() { $comment_id = self::factory()->comment->create( array( 'comment_post_ID' => self::$post_id ) ); @@ -175,6 +190,8 @@ /** * @ticket 39732 + * + * @covers ::wp_update_comment */ public function test_wp_update_comment_is_wp_error() { $comment_id = self::factory()->comment->create( array( 'comment_post_ID' => self::$post_id ) ); @@ -201,6 +218,11 @@ return new WP_Error( 'comment_wrong', 'wp_update_comment_data filter fails for this comment.', 500 ); } + /** + * + * + * @covers ::get_approved_comments + */ public function test_get_approved_comments() { $ca1 = self::factory()->comment->create( array( @@ -257,6 +279,8 @@ /** * @ticket 30412 + * + * @covers ::get_approved_comments */ public function test_get_approved_comments_with_post_id_0_should_return_empty_array() { $ca1 = self::factory()->comment->create( @@ -273,6 +297,8 @@ /** * @ticket 14279 + * + * @covers ::wp_new_comment */ public function test_wp_new_comment_respects_dates() { $data = array( @@ -296,6 +322,8 @@ /** * @ticket 14601 + * + * @covers ::wp_new_comment */ public function test_wp_new_comment_respects_author_ip() { $data = array( @@ -317,6 +345,8 @@ /** * @ticket 14601 + * + * @covers ::wp_new_comment */ public function test_wp_new_comment_respects_author_ip_empty_string() { $data = array( @@ -338,6 +368,8 @@ /** * @ticket 14601 + * + * @covers ::wp_new_comment */ public function test_wp_new_comment_respects_comment_agent() { $data = array( @@ -360,6 +392,8 @@ /** * @ticket 14601 + * + * @covers ::wp_new_comment */ public function test_wp_new_comment_should_trim_provided_comment_agent_to_254_chars() { $data = array( @@ -382,6 +416,8 @@ /** * @ticket 14601 + * + * @covers ::wp_new_comment */ public function test_wp_new_comment_respects_comment_agent_empty_string() { $data = array( @@ -402,7 +438,9 @@ $this->assertEquals( $data['comment_agent'], $comment->comment_agent ); } - + /** + * @covers ::wp_new_comment + */ public function test_comment_field_lengths() { $data = array( 'comment_post_ID' => self::$post_id, @@ -424,6 +462,8 @@ /** * @ticket 32566 + * + * @covers ::wp_notify_moderator */ public function test_wp_notify_moderator_should_not_throw_notice_when_post_author_is_0() { $p = self::factory()->post->create( @@ -441,6 +481,11 @@ $this->assertTrue( wp_notify_moderator( $c ) ); } + /** + * + * + * @covers ::wp_new_comment_notify_postauthor + */ public function test_wp_new_comment_notify_postauthor_should_send_email_when_comment_is_approved() { $c = self::factory()->comment->create( array( @@ -452,6 +497,11 @@ $this->assertTrue( $sent ); } + /** + * + * + * @covers ::wp_new_comment_notify_postauthor + */ public function test_wp_new_comment_notify_postauthor_should_not_send_email_when_comment_is_unapproved() { $c = self::factory()->comment->create( array( @@ -466,6 +516,8 @@ /** * @ticket 33587 + * + * @covers ::wp_new_comment_notify_postauthor */ public function test_wp_new_comment_notify_postauthor_should_not_send_email_when_comment_has_been_marked_as_spam() { $c = self::factory()->comment->create( @@ -481,6 +533,8 @@ /** * @ticket 35006 + * + * @covers ::wp_new_comment_notify_postauthor */ public function test_wp_new_comment_notify_postauthor_should_not_send_email_when_comment_has_been_trashed() { $c = self::factory()->comment->create( @@ -496,6 +550,8 @@ /** * @ticket 43805 + * + * @covers ::wp_new_comment_notify_postauthor */ public function test_wp_new_comment_notify_postauthor_content_should_include_link_to_parent() { $c1 = self::factory()->comment->create( @@ -520,6 +576,8 @@ /** * @ticket 43805 + * + * @covers ::wp_new_comment_notify_moderator */ public function test_wp_new_comment_notify_moderator_content_should_include_link_to_parent() { $c1 = self::factory()->comment->create( @@ -556,6 +614,8 @@ /** * @ticket 12431 + * + * @covers ::get_comment_meta */ public function test_wp_new_comment_with_meta() { $c = self::factory()->comment->create( @@ -573,6 +633,8 @@ /** * @ticket 8071 + * + * @covers WP_Comment::get_children */ public function test_wp_comment_get_children_should_fill_children() { $c1 = self::factory()->comment->create( @@ -633,6 +695,8 @@ /** * @ticket 27571 + * + * @covers ::get_comment */ public function test_post_properties_should_be_lazyloaded() { $c = self::factory()->comment->create( array( 'comment_post_ID' => self::$post_id ) ); @@ -684,6 +748,8 @@ /** * @ticket 761 + * + * @covers ::wp_new_comment */ public function test_wp_notify_moderator_filter_moderation_notify_option_true_filter_false() { $comment_data = $this->setup_notify_comment(); @@ -705,6 +771,8 @@ /** * @ticket 761 + * + * @covers ::wp_new_comment */ public function test_wp_notify_moderator_filter_moderation_notify_option_false_filter_true() { $comment_data = $this->setup_notify_comment(); @@ -726,6 +794,8 @@ /** * @ticket 761 + * + * @covers ::wp_new_comment */ public function test_wp_notify_post_author_filter_comments_notify_option_true_filter_false() { @@ -748,6 +818,8 @@ /** * @ticket 761 + * + * @covers ::wp_new_comment */ public function test_wp_notify_post_author_filter_comments_notify_option_false_filter_true() { $comment_data = $this->setup_notify_comment(); @@ -852,6 +924,11 @@ return $email_sent_when_comment_approved || $email_sent_when_comment_added; } + /** + * + * + * @covers ::_close_comments_for_old_post + */ public function test_close_comments_for_old_post() { update_option( 'close_comments_for_old_posts', true ); // Close comments more than one day old. @@ -867,6 +944,11 @@ $this->assertTrue( $new_post_comment_status ); } + /** + * + * + * @covers ::_close_comments_for_old_post + */ public function test_close_comments_for_old_post_undated_draft() { $draft_id = self::factory()->post->create( array( @@ -881,6 +963,8 @@ /** * @ticket 35276 + * + * @covers ::wp_update_comment */ public function test_wp_update_comment_author_id_and_agent() { @@ -917,6 +1001,9 @@ $this->assertSame( 'SHIELD_AGENT', $updated->comment_agent ); } + /** + * @covers ::wp_get_comment_fields_max_lengths + */ public function test_wp_get_comment_fields_max_lengths() { $expected = array( 'comment_author' => 245, @@ -937,6 +1024,8 @@ * * @group privacy * @ticket 43442 + * + * @covers ::wp_comments_personal_data_eraser */ public function test_wp_comments_personal_data_eraser() { @@ -994,6 +1083,8 @@ * * @group privacy * @ticket 43442 + * + * @covers ::wp_comments_personal_data_eraser */ public function test_wp_comments_personal_data_eraser_empty_first_page_output() { @@ -1013,6 +1104,8 @@ * * @group privacy * @ticket 43442 + * + * @covers ::wp_comments_personal_data_eraser */ public function test_wp_comments_personal_data_eraser_non_empty_first_page_output() { @@ -1045,6 +1138,8 @@ * * @group privacy * @ticket 43442 + * + * @covers ::wp_comments_personal_data_eraser */ public function test_wp_comments_personal_data_eraser_empty_second_page_output() { @@ -1077,6 +1172,8 @@ * * @group privacy * @ticket 43442 + * + * @covers ::wp_comments_personal_data_eraser */ public function test_wp_anonymize_comment_filter_to_prevent_comment_anonymization() { @@ -1114,6 +1211,8 @@ * * @group privacy * @ticket 43442 + * + * @covers ::wp_comments_personal_data_eraser */ public function test_wp_anonymize_comment_filter_to_prevent_comment_anonymization_with_custom_message() { @@ -1179,6 +1278,11 @@ $this->assertSame( 'Bar', $comment->comment_author ); } + /** + * + * + * @covers ::wp_trash_comment + */ public function test_trash_should_invalidate_comment_cache() { global $wpdb; @@ -1193,6 +1297,11 @@ $this->assertSame( 'trash', $comment->comment_approved ); } + /** + * + * + * @covers ::wp_untrash_comment + */ public function test_untrash_should_invalidate_comment_cache() { global $wpdb; @@ -1209,6 +1318,11 @@ $this->assertSame( '1', $comment->comment_approved ); } + /** + * + * + * @covers ::wp_spam_comment + */ public function test_spam_should_invalidate_comment_cache() { global $wpdb; @@ -1223,6 +1337,11 @@ $this->assertSame( 'spam', $comment->comment_approved ); } + /** + * + * + * @covers ::wp_unspam_comment + */ public function test_unspam_should_invalidate_comment_cache() { global $wpdb; @@ -1244,6 +1363,8 @@ * * @group privacy * @ticket 43440 + * + * @covers ::wp_comments_personal_data_exporter */ public function test_wp_comments_personal_data_exporter() { $args = array( @@ -1290,6 +1411,8 @@ * * @group privacy * @ticket 43440 + * + * @covers ::wp_comments_personal_data_exporter */ public function test_wp_comments_personal_data_exporter_no_comments_found() { @@ -1308,6 +1431,8 @@ * * @group privacy * @ticket 43440 + * + * @covers ::wp_comments_personal_data_exporter */ public function test_wp_comments_personal_data_exporter_empty_comment_prop() { $args = array( @@ -1339,6 +1464,8 @@ * * @group privacy * @ticket 43440 + * + * @covers ::wp_comments_personal_data_exporter */ public function test_wp_comments_personal_data_exporter_empty_second_page() { $args = array( Index: tests/phpunit/tests/compat.php =================================================================== --- tests/phpunit/tests/compat.php (revision 48816) +++ tests/phpunit/tests/compat.php (working copy) @@ -34,6 +34,8 @@ /** * @dataProvider utf8_string_lengths + * + * @covers ::_mb_strlen */ function test_mb_strlen( $string, $expected_character_length ) { $this->assertEquals( $expected_character_length, _mb_strlen( $string, 'UTF-8' ) ); @@ -41,6 +43,8 @@ /** * @dataProvider utf8_string_lengths + * + * @covers ::_mb_strlen */ function test_mb_strlen_via_regex( $string, $expected_character_length ) { _wp_can_use_pcre_u( false ); @@ -50,6 +54,8 @@ /** * @dataProvider utf8_string_lengths + * + * @covers ::_mb_strlen */ function test_8bit_mb_strlen( $string, $expected_character_length, $expected_byte_length ) { $this->assertEquals( $expected_byte_length, _mb_strlen( $string, '8bit' ) ); @@ -57,6 +63,8 @@ /** * @dataProvider utf8_substrings + * + * @covers ::_mb_substr */ function test_mb_substr( $string, $start, $length, $expected_character_substring ) { $this->assertEquals( $expected_character_substring, _mb_substr( $string, $start, $length, 'UTF-8' ) ); @@ -64,6 +72,8 @@ /** * @dataProvider utf8_substrings + * + * @covers ::_mb_substr */ function test_mb_substr_via_regex( $string, $start, $length, $expected_character_substring ) { _wp_can_use_pcre_u( false ); @@ -73,11 +83,17 @@ /** * @dataProvider utf8_substrings + * + * @covers ::_mb_substr */ function test_8bit_mb_substr( $string, $start, $length, $expected_character_substring, $expected_byte_substring ) { $this->assertEquals( $expected_byte_substring, _mb_substr( $string, $start, $length, '8bit' ) ); } + /** + * + * @covers ::_mb_substr + */ function test_mb_substr_phpcore() { /* https://github.com/php/php-src/blob/php-5.6.8/ext/mbstring/tests/mb_substr_basic.phpt */ $string_ascii = 'ABCDEF'; @@ -164,16 +180,28 @@ } + /** + * + * @covers ::_hash_hmac + */ function test_hash_hmac_simple() { $this->assertEquals( '140d1cb79fa12e2a31f32d35ad0a2723', _hash_hmac( 'md5', 'simple', 'key' ) ); $this->assertEquals( '993003b95758e0ac2eba451a4c5877eb1bb7b92a', _hash_hmac( 'sha1', 'simple', 'key' ) ); } + /** + * + * @covers ::_hash_hmac + */ function test_hash_hmac_padding() { $this->assertEquals( '3c1399103807cf12ec38228614416a8c', _hash_hmac( 'md5', 'simple', '65 character key 65 character key 65 character key 65 character k' ) ); $this->assertEquals( '4428826d20003e309d6c2a6515891370daf184ea', _hash_hmac( 'sha1', 'simple', '65 character key 65 character key 65 character key 65 character k' ) ); } + /** + * + * @covers ::_hash_hmac + */ function test_hash_hmac_output() { $this->assertEquals( array( 1 => '140d1cb79fa12e2a31f32d35ad0a2723' ), unpack( 'H32', _hash_hmac( 'md5', 'simple', 'key', true ) ) ); $this->assertEquals( array( 1 => '993003b95758e0ac2eba451a4c5877eb1bb7b92a' ), unpack( 'H40', _hash_hmac( 'sha1', 'simple', 'key', true ) ) ); @@ -181,6 +209,9 @@ /** * @expectedException PHPUnit_Framework_Error_Deprecated + * + * @covers Services_JSON::encodeUnsafe + * @covers Services_JSON::decode */ function test_json_encode_decode() { require_once ABSPATH . WPINC . '/class-json.php'; @@ -194,6 +225,8 @@ * Test that is_countable() is always available (either from PHP or WP). * * @ticket 43583 + * + * @covers ::is_countable */ function test_is_countable_availability() { $this->assertTrue( function_exists( 'is_countable' ) ); @@ -208,6 +241,8 @@ * * @param mixed $variable Variable to check. * @param bool $is_countable The expected return value of PHP 7.3 is_countable() function. + * + * @covers ::is_countable */ function test_is_countable_functionality( $variable, $is_countable ) { $this->assertSame( is_countable( $variable ), $is_countable ); @@ -243,6 +278,8 @@ * Test is_countable() polyfill for ResourceBundle. * * @ticket 43583 + * + * @covers ::is_countable */ function test_is_countable_ResourceBundle() { if ( ! class_exists( 'ResourceBundle' ) ) { @@ -256,6 +293,8 @@ * Test is_countable() polyfill for SimpleXMLElement. * * @ticket 43583 + * + * @covers ::is_countable */ function test_is_countable_SimpleXMLElement() { if ( ! class_exists( 'SimpleXMLElement' ) ) { @@ -269,6 +308,8 @@ * Test that is_iterable() is always available (either from PHP or WP). * * @ticket 43619 + * + * @covers ::is_iterable */ function test_is_iterable_availability() { $this->assertTrue( function_exists( 'is_iterable' ) ); @@ -283,6 +324,8 @@ * * @param mixed $variable Variable to check. * @param bool $is_iterable The expected return value of PHP 7.1 is_iterable() function. + * + * @covers ::is_iterable */ function test_is_iterable_functionality( $variable, $is_iterable ) { $this->assertSame( is_iterable( $variable ), $is_iterable ); Index: tests/phpunit/tests/cron.php =================================================================== --- tests/phpunit/tests/cron.php (revision 48816) +++ tests/phpunit/tests/cron.php (working copy) @@ -30,6 +30,10 @@ parent::tearDown(); } + /** + * + * @covers ::wp_get_schedule + */ function test_wp_get_schedule_empty() { // Nothing scheduled. $hook = __FUNCTION__; @@ -36,6 +40,10 @@ $this->assertFalse( wp_get_schedule( $hook ) ); } + /** + * + * @covers ::wp_schedule_single_event + */ function test_schedule_event_single() { // Schedule an event and make sure it's returned by wp_next_scheduled(). $hook = __FUNCTION__; @@ -50,6 +58,10 @@ } + /** + * + * @covers ::wp_schedule_single_event + */ function test_schedule_event_single_args() { // Schedule an event with arguments and make sure it's returned by wp_next_scheduled(). $hook = 'event'; @@ -68,6 +80,10 @@ $this->assertEquals( '', wp_get_schedule( $hook, $args ) ); } + /** + * + * @covers ::wp_schedule_event + */ function test_schedule_event() { // Schedule an event and make sure it's returned by wp_next_scheduled(). $hook = __FUNCTION__; @@ -82,6 +98,10 @@ $this->assertEquals( $recur, wp_get_schedule( $hook ) ); } + /** + * + * @covers ::wp_schedule_event + */ function test_schedule_event_args() { // Schedule an event and make sure it's returned by wp_next_scheduled(). $hook = 'event'; @@ -101,6 +121,10 @@ } + /** + * + * @covers ::wp_unschedule_event + */ function test_unschedule_event() { // Schedule an event and make sure it's returned by wp_next_scheduled(). $hook = __FUNCTION__; @@ -115,6 +139,10 @@ $this->assertEquals( false, wp_next_scheduled( $hook ) ); } + /** + * + * @covers ::wp_clear_scheduled_hook + */ function test_clear_schedule() { $hook = __FUNCTION__; $args = array( 'arg1' ); @@ -142,6 +170,10 @@ $this->assertFalse( wp_next_scheduled( $hook, $args ) ); } + /** + * + * @covers ::wp_clear_scheduled_hook + */ function test_clear_undefined_schedule() { $hook = __FUNCTION__; $args = array( 'arg1' ); @@ -154,6 +186,10 @@ $this->assertSame( 0, $hook_unscheduled ); } + /** + * + * @covers ::wp_clear_scheduled_hook + */ function test_clear_schedule_multiple_args() { $hook = __FUNCTION__; $args = array( 'arg1', 'arg2' ); @@ -182,6 +218,8 @@ /** * @ticket 10468 + * + * @covers ::wp_clear_scheduled_hook */ function test_clear_schedule_new_args() { $hook = __FUNCTION__; @@ -220,6 +258,8 @@ /** * @ticket 18997 + * + * @covers ::wp_clear_scheduled_hook */ function test_unschedule_hook() { $hook = __FUNCTION__; @@ -240,7 +280,10 @@ $this->assertSame( 4, $unschedule_hook ); $this->assertFalse( wp_next_scheduled( $hook ) ); } - + /** + * + * @covers ::wp_clear_scheduled_hook + */ function test_unschedule_undefined_hook() { $hook = __FUNCTION__; $unrelated_hook = __FUNCTION__ . '_two'; @@ -261,6 +304,8 @@ /** * @ticket 6966 + * + * @covers ::wp_schedule_single_event */ function test_duplicate_event() { // Duplicate events close together should be skipped. @@ -280,6 +325,8 @@ /** * @ticket 6966 + * + * @covers ::wp_schedule_single_event */ function test_not_duplicate_event() { // Duplicate events far apart should work normally. @@ -300,6 +347,10 @@ $this->assertEquals( $ts1, wp_next_scheduled( $hook, $args ) ); } + /** + * + * @covers ::wp_schedule_single_event + */ function test_not_duplicate_event_reversed() { // Duplicate events far apart should work normally regardless of order. $hook = __FUNCTION__; @@ -324,6 +375,8 @@ * modification of the cron_array_option. * * @ticket 32656 + * + * @covers ::wp_schedule_single_event */ function test_pre_schedule_event_filter() { $hook = __FUNCTION__; @@ -376,6 +429,8 @@ * modification of the cron_array_option. * * @ticket 32656 + * + * @covers ::wp_schedule_event */ function test_pre_reschedule_event_filter() { $hook = __FUNCTION__; @@ -400,6 +455,8 @@ * modification of the cron_array_option. * * @ticket 32656 + * + * @covers ::wp_unschedule_event */ function test_pre_unschedule_event_filter() { $hook = __FUNCTION__; @@ -424,6 +481,8 @@ * modification of the cron_array_option. * * @ticket 32656 + * + * @covers ::wp_clear_scheduled_hook */ function test_pre_clear_scheduled_hook_filters() { $hook = __FUNCTION__; @@ -455,6 +514,8 @@ * return a filtered value as expected. * * @ticket 32656 + * + * @covers ::wp_get_scheduled_event */ function test_pre_scheduled_event_hooks() { add_filter( 'pre_get_scheduled_event', array( $this, 'filter_pre_scheduled_event_hooks' ) ); @@ -489,6 +550,8 @@ * When a timestamp is specified, a particular event should be returned. * * @ticket 45976. + * + * @covers ::wp_get_scheduled_event */ function test_get_scheduled_event_singles() { $hook = __FUNCTION__; @@ -532,6 +595,8 @@ * When a timestamp is specified, a particular event should be returned. * * @ticket 45976. + * + * @covers ::wp_get_scheduled_event */ function test_get_scheduled_event_recurring() { $hook = __FUNCTION__; @@ -576,6 +641,8 @@ * Ensure wp_get_scheduled_event() returns false when expected. * * @ticket 45976. + * + * @covers ::wp_get_scheduled_event */ function test_get_scheduled_event_false() { $hook = __FUNCTION__; @@ -601,6 +668,8 @@ * Ensure any past event counts as a duplicate. * * @ticket 44818 + * + * @covers ::wp_get_scheduled_event */ function test_duplicate_past_event() { $hook = __FUNCTION__; @@ -623,6 +692,8 @@ * Ensure any near future event counts as a duplicate. * * @ticket 44818 + * + * @covers ::wp_get_scheduled_event */ function test_duplicate_near_future_event() { $hook = __FUNCTION__; @@ -645,6 +716,8 @@ * Duplicate future events are disallowed. * * @ticket 44818 + * + * @covers ::wp_get_scheduled_event */ function test_duplicate_future_event() { $hook = __FUNCTION__; @@ -665,6 +738,8 @@ * Future events are allowed. * * @ticket 44818 + * + * @covers ::wp_get_scheduled_event */ function test_not_duplicate_future_event() { $hook = __FUNCTION__; Index: tests/phpunit/tests/db.php =================================================================== --- tests/phpunit/tests/db.php (revision 48816) +++ tests/phpunit/tests/db.php (working copy) @@ -58,6 +58,8 @@ * Test that WPDB will reconnect when the DB link dies * * @ticket 5932 + * + * @covers wpdb::db_connect */ public function test_db_reconnect() { global $wpdb; @@ -82,6 +84,8 @@ * @global mixed $wpdb * * @ticket 19861 + * + * @covers wpdb::update */ public function test_locale_floats() { global $wpdb; @@ -128,6 +132,8 @@ /** * @ticket 10041 + * + * @covers wpdb::esc_like */ function test_esc_like() { global $wpdb; @@ -164,6 +170,8 @@ * @param $data string The haystack, raw. * @param $like string The like phrase, raw. * @param $result string The expected comparison result; '1' = true, '0' = false + * + * @covers wpdb::esc_like */ function test_like_query( $data, $like, $result ) { global $wpdb; @@ -227,6 +235,8 @@ /** * @ticket 18510 + * + * @coversNothing */ function test_wpdb_supposedly_protected_properties() { global $wpdb; @@ -243,6 +253,8 @@ /** * @ticket 21212 + * + * @coversNothing */ function test_wpdb_actually_protected_properties() { global $wpdb; @@ -274,6 +286,9 @@ * Test that an escaped %%f is not altered * * @ticket 19861 + * + * @covers wpdb::placeholder_escape + * @covers wpdb::remove_placeholder_escape */ public function test_double_escaped_placeholders() { global $wpdb; @@ -289,6 +304,8 @@ * Test that SQL modes are set correctly * * @ticket 26847 + * + * @covers wpdb::set_sql_mode */ function test_set_sql_mode() { global $wpdb; @@ -309,6 +326,8 @@ * Test that incompatible SQL modes are blocked * * @ticket 26847 + * + * @covers wpdb::set_sql_mode */ function test_set_incompatible_sql_mode() { global $wpdb; @@ -327,6 +346,8 @@ * Test that incompatible SQL modes can be changed * * @ticket 26847 + * + * @covers wpdb::set_sql_mode */ function test_set_allowed_incompatible_sql_mode() { global $wpdb; @@ -360,6 +381,8 @@ /** * @ticket 25604 * @expectedIncorrectUsage wpdb::prepare + * + * @covers wpdb::prepare */ function test_prepare_without_arguments() { global $wpdb; @@ -370,6 +393,10 @@ $this->assertEquals( "SELECT * FROM $wpdb->users WHERE id = 0", $prepared ); } + /** + * + * @covers wpdb::prepare + */ function test_prepare_sprintf() { global $wpdb; @@ -379,6 +406,8 @@ /** * @expectedIncorrectUsage wpdb::prepare + * + * @covers wpdb::prepare */ function test_prepare_sprintf_invalid_args() { global $wpdb; @@ -392,6 +421,10 @@ $this->assertEquals( "SELECT * FROM $wpdb->users WHERE id = 0 AND user_login = 'admin'", $prepared ); } + /** + * + * @covers wpdb::prepare + */ function test_prepare_vsprintf() { global $wpdb; @@ -401,6 +434,8 @@ /** * @expectedIncorrectUsage wpdb::prepare + * + * @covers wpdb::prepare */ function test_prepare_vsprintf_invalid_args() { global $wpdb; @@ -418,6 +453,8 @@ * @ticket 42040 * @dataProvider data_prepare_incorrect_arg_count * @expectedIncorrectUsage wpdb::prepare + * + * @covers wpdb::prepare */ public function test_prepare_incorrect_arg_count( $query, $args, $expected ) { global $wpdb; @@ -479,12 +516,19 @@ ); } + /** + * + * @covers wpdb::db_version + */ function test_db_version() { global $wpdb; $this->assertTrue( version_compare( $wpdb->db_version(), '5.0', '>=' ) ); } - + /** + * + * @covers wpdb::get_caller + */ function test_get_caller() { global $wpdb; $str = $wpdb->get_caller(); @@ -493,6 +537,10 @@ $this->assertEquals( $called, end( $calls ) ); } + /** + * + * @covers wpdb::has_cap + */ function test_has_cap() { global $wpdb; $this->assertTrue( $wpdb->has_cap( 'collation' ) ); @@ -513,6 +561,8 @@ /** * @expectedDeprecated supports_collation + * + * @covers wpdb::supports_collation */ function test_supports_collation() { global $wpdb; @@ -519,6 +569,10 @@ $this->assertTrue( $wpdb->supports_collation() ); } + /** + * + * @covers wpdb::check_database_version + */ function test_check_database_version() { global $wpdb; $this->assertEmpty( $wpdb->check_database_version() ); @@ -526,12 +580,18 @@ /** * @expectedException WPDieException + * + * @covers wpdb::bail */ function test_bail() { global $wpdb; $wpdb->bail( 'Database is dead.' ); } - + /** + * + * @covers wpdb::timer_start + * @covers wpdb::timer_stop + */ function test_timers() { global $wpdb; @@ -542,7 +602,10 @@ $this->assertNotEquals( $wpdb->time_start, $stop ); $this->assertGreaterThan( $stop, $wpdb->time_start ); } - + /** + * + * @covers wpdb::get_col_info + */ function test_get_col_info() { global $wpdb; @@ -552,7 +615,11 @@ $this->assertEquals( array( $wpdb->users ), $wpdb->get_col_info( 'table' ) ); $this->assertEquals( $wpdb->users, $wpdb->get_col_info( 'table', 0 ) ); } - + /** + * + * @covers wpdb::query + * @covers wpdb::delete + */ function test_query_and_delete() { global $wpdb; $rows = $wpdb->query( "INSERT INTO $wpdb->users (display_name) VALUES ('Walter Sobchak')" ); @@ -561,7 +628,11 @@ $d_rows = $wpdb->delete( $wpdb->users, array( 'ID' => $wpdb->insert_id ) ); $this->assertEquals( 1, $d_rows ); } - + /** + * + * @covers wpdb::query + * @covers wpdb::get_row + */ function test_get_row() { global $wpdb; $rows = $wpdb->query( "INSERT INTO $wpdb->users (display_name) VALUES ('Walter Sobchak')" ); @@ -584,6 +655,8 @@ * @dataProvider data_test_get_col * * @ticket 45299 + * + * @covers wpdb::get_col */ function test_get_col( $query, $expected, $last_result, $column ) { global $wpdb; @@ -664,6 +737,10 @@ ); } + /** + * + * @covers wpdb::replace + */ function test_replace() { global $wpdb; $rows1 = $wpdb->insert( $wpdb->users, array( 'display_name' => 'Walter Sobchak' ) ); @@ -691,6 +768,8 @@ * wpdb::update() requires a WHERE condition. * * @ticket 26106 + * + * @covers wpdb::update */ function test_empty_where_on_update() { global $wpdb; @@ -713,6 +792,8 @@ * mysqli_ incorrect flush and further sync issues. * * @ticket 28155 + * + * @coversNothing */ function test_mysqli_flush_sync() { global $wpdb; @@ -873,6 +954,8 @@ /** * @dataProvider data_get_table_from_query * @ticket 21212 + * + * @covers wpdb::get_table_from_query */ function test_get_table_from_query( $query, $table ) { $this->assertEquals( $table, self::$_wpdb->get_table_from_query( $query ) ); @@ -888,6 +971,8 @@ /** * @dataProvider data_get_table_from_query_false * @ticket 21212 + * + * @covers wpdb::get_table_from_query */ function test_get_table_from_query_false( $query ) { $this->assertFalse( self::$_wpdb->get_table_from_query( $query ) ); @@ -915,6 +1000,8 @@ /** * @dataProvider data_get_escaped_table_from_show_query * @ticket 38751 + * + * @covers wpdb::get_table_from_query */ function test_get_escaped_table_from_show_query( $query, $table ) { $this->assertEquals( $table, self::$_wpdb->get_table_from_query( $query ) ); @@ -1030,6 +1117,8 @@ /** * @dataProvider data_process_field_formats * @ticket 21212 + * + * @covers wpdb::process_field_formats */ function test_process_field_formats( $data, $format, $expected, $message ) { $actual = self::$_wpdb->process_field_formats( $data, $format ); @@ -1038,6 +1127,8 @@ /** * @ticket 21212 + * + * @covers wpdb::process_fields */ function test_process_fields() { global $wpdb; @@ -1068,6 +1159,8 @@ /** * @ticket 21212 * @depends test_process_fields + * + * @covers wpdb::process_fields */ function test_process_fields_on_nonexistent_table( $data ) { self::$_wpdb->suppress_errors( true ); @@ -1078,6 +1171,8 @@ /** * @ticket 21212 + * + * @covers wpdb::get_table_charset */ function test_pre_get_table_charset_filter() { add_filter( 'pre_get_table_charset', array( $this, 'filter_pre_get_table_charset' ), 10, 2 ); @@ -1092,6 +1187,8 @@ /** * @ticket 21212 + * + * @covers wpdb::get_col_charset */ function test_pre_get_col_charset_filter() { add_filter( 'pre_get_col_charset', array( $this, 'filter_pre_get_col_charset' ), 10, 3 ); @@ -1106,6 +1203,8 @@ /** * @ticket 15158 + * + * @covers wpdb::insert */ function test_null_insert() { global $wpdb; @@ -1128,6 +1227,8 @@ /** * @ticket 15158 + * + * @covers wpdb::insert */ function test_null_update_value() { global $wpdb; @@ -1166,6 +1267,8 @@ /** * @ticket 15158 + * + * @covers wpdb::update */ function test_null_update_where() { global $wpdb; @@ -1204,6 +1307,8 @@ /** * @ticket 15158 + * + * @covers wpdb::delete */ function test_null_delete() { global $wpdb; @@ -1240,6 +1345,8 @@ /** * @ticket 34903 + * + * @covers wpdb::close */ function test_close() { global $wpdb; @@ -1259,6 +1366,8 @@ /** * @ticket 36917 + * + * @covers wpdb::determine_charset */ function test_charset_not_determined_when_disconnected() { global $wpdb; @@ -1277,6 +1386,8 @@ /** * @ticket 36917 + * + * @covers wpdb::determine_charset */ function test_charset_switched_to_utf8mb4() { global $wpdb; @@ -1296,6 +1407,8 @@ /** * @ticket 32105 * @ticket 36917 + * + * @covers wpdb::determine_charset */ function test_collate_switched_to_utf8mb4_520() { global $wpdb; @@ -1315,6 +1428,8 @@ /** * @ticket 32405 * @ticket 36917 + * + * @covers wpdb::determine_charset */ function test_non_unicode_collations() { global $wpdb; @@ -1333,6 +1448,8 @@ /** * @ticket 37982 + * + * @covers wpdb::determine_charset */ function test_charset_switched_to_utf8() { global $wpdb; @@ -1352,6 +1469,8 @@ /** * @dataProvider data_prepare_with_placeholders + * + * @covers wpdb::prepare */ function test_prepare_with_placeholders_and_individual_args( $sql, $values, $incorrect_usage, $expected ) { global $wpdb; @@ -1371,6 +1490,8 @@ /** * @dataProvider data_prepare_with_placeholders + * + * @covers wpdb::prepare */ function test_prepare_with_placeholders_and_array_args( $sql, $values, $incorrect_usage, $expected ) { global $wpdb; @@ -1565,7 +1686,12 @@ /** * @dataProvider data_escape_and_prepare + * + * @covers ::esc_sql + /** + * @coversNothing */ + */ function test_escape_and_prepare( $escape, $sql, $values, $incorrect_usage, $expected ) { global $wpdb; @@ -1612,6 +1738,8 @@ /** * @expectedIncorrectUsage wpdb::prepare + * + * @covers wpdb::prepare */ function test_double_prepare() { global $wpdb; @@ -1623,6 +1751,10 @@ $this->assertNull( $query ); } + /** + * + * @covers wpdb::prepare + */ function test_prepare_numeric_placeholders_float_args() { global $wpdb; @@ -1638,6 +1770,10 @@ $this->assertContains( ' second=2.2', $actual ); } + /** + * + * @covers wpdb::prepare + */ function test_prepare_numeric_placeholders_float_array() { global $wpdb; @@ -1652,6 +1788,10 @@ $this->assertContains( ' second=2.2', $actual ); } + /** + * + * @covers wpdb::prepare + */ function test_query_unescapes_placeholders() { global $wpdb; @@ -1671,6 +1811,10 @@ $this->assertEquals( $value, $actual ); } + /** + * + * @covers wpdb::placeholder_escape + */ function test_esc_sql_with_unsupported_placeholder_type() { global $wpdb; @@ -1684,6 +1828,8 @@ /** * @dataProvider parse_db_host_data_provider * @ticket 41722 + * + * @covers wpdb::parse_db_host */ public function test_parse_db_host( $host_string, $expect_bail, $host, $port, $socket, $is_ipv6 ) { global $wpdb; Index: tests/phpunit/tests/dbdelta.php =================================================================== --- tests/phpunit/tests/dbdelta.php (revision 48816) +++ tests/phpunit/tests/dbdelta.php (working copy) @@ -96,6 +96,8 @@ /** * Test table creation. + * + * @covers ::dbDelta */ public function test_creating_a_table() { @@ -133,6 +135,8 @@ /** * Test that it does nothing for an existing table. + * + * @covers ::dbDelta */ public function test_existing_table() { @@ -155,6 +159,8 @@ /** * Test the column type is updated. + * + * @covers ::dbDelta */ public function test_column_type_change() { @@ -184,6 +190,8 @@ /** * Test new column added. + * + * @covers ::dbDelta */ public function test_column_added() { @@ -218,6 +226,8 @@ * Test that it does nothing when a column is removed. * * @ticket 26801 + * + * @covers ::dbDelta */ public function test_columns_arent_removed() { @@ -242,6 +252,8 @@ /** * Test that nothing happens with $execute is false. + * + * @covers ::dbDelta */ public function test_no_execution() { @@ -275,6 +287,8 @@ /** * Test inserting into the database + * + * @covers ::dbDelta */ public function test_insert_into_table() { global $wpdb; @@ -296,6 +310,8 @@ * Test that FULLTEXT indexes are detected. * * @ticket 14445 + * + * @covers ::dbDelta */ public function test_fulltext_index() { global $wpdb; @@ -401,6 +417,8 @@ /** * @ticket 31869 + * + * @covers ::dbDelta */ function test_truncated_index() { global $wpdb; @@ -441,6 +459,8 @@ /** * @ticket 36748 + * + * @covers ::dbDelta */ function test_dont_downsize_text_fields() { global $wpdb; @@ -466,6 +486,8 @@ /** * @ticket 36748 + * + * @covers ::dbDelta */ function test_dont_downsize_blob_fields() { global $wpdb; @@ -491,6 +513,8 @@ /** * @ticket 36748 + * + * @covers ::dbDelta */ function test_upsize_text_fields() { global $wpdb; @@ -522,6 +546,8 @@ /** * @ticket 36748 + * + * @covers ::dbDelta */ function test_upsize_blob_fields() { global $wpdb; @@ -553,6 +579,8 @@ /** * @ticket 20263 + * + * @covers ::dbDelta */ function test_query_with_backticks_does_not_throw_an_undefined_index_warning() { global $wpdb; @@ -578,6 +606,8 @@ /** * @ticket 36948 + * + * @covers ::dbDelta */ function test_spatial_indices() { global $wpdb; @@ -639,6 +669,8 @@ /** * @ticket 20263 + * + * @covers ::dbDelta */ function test_query_with_backticks_does_not_cause_a_query_to_alter_all_columns_and_indices_to_run_even_if_none_have_changed() { global $wpdb; @@ -668,6 +700,8 @@ /** * @ticket 20263 + * + * @covers ::dbDelta */ function test_index_with_a_reserved_keyword_can_be_created() { global $wpdb; @@ -704,6 +738,8 @@ /** * @ticket 20263 + * + * @covers ::dbDelta */ function test_wp_get_db_schema_does_no_alter_queries_on_existing_install() { $updates = dbDelta( wp_get_db_schema() ); @@ -713,6 +749,8 @@ /** * @ticket 20263 + * + * @covers ::dbDelta */ function test_key_and_index_and_fulltext_key_and_fulltext_index_and_unique_key_and_unique_index_indicies() { global $wpdb; @@ -751,6 +789,8 @@ /** * @ticket 20263 + * + * @covers ::dbDelta */ function test_index_and_key_are_synonyms_and_do_not_recreate_indices() { global $wpdb; @@ -775,6 +815,8 @@ /** * @ticket 20263 + * + * @covers ::dbDelta */ function test_indices_with_prefix_limits_are_created_and_do_not_recreate_indices() { global $wpdb; @@ -809,6 +851,8 @@ /** * @ticket 34959 + * + * @covers ::dbDelta */ function test_index_col_names_with_order_do_not_recreate_indices() { global $wpdb; @@ -833,6 +877,8 @@ /** * @ticket 34873 + * + * @covers ::dbDelta */ function test_primary_key_with_single_space_does_not_recreate_index() { global $wpdb; @@ -857,6 +903,8 @@ /** * @ticket 34869 + * + * @covers ::dbDelta */ function test_index_definitions_with_spaces_do_not_recreate_indices() { global $wpdb; @@ -881,6 +929,8 @@ /** * @ticket 34871 + * + * @covers ::dbDelta */ function test_index_types_are_not_case_sensitive_and_do_not_recreate_indices() { global $wpdb; @@ -905,6 +955,8 @@ /** * @ticket 34874 + * + * @covers ::dbDelta */ function test_key_names_are_not_case_sensitive_and_do_not_recreate_indices() { global $wpdb; @@ -930,6 +982,8 @@ /** * @ticket 34870 + * + * @covers ::dbDelta */ function test_unchanged_key_lengths_do_not_recreate_index() { global $wpdb; @@ -955,6 +1009,8 @@ /** * @ticket 34870 + * + * @covers ::dbDelta */ function test_changed_key_lengths_do_not_recreate_index() { global $wpdb; @@ -1039,6 +1095,8 @@ /** * @ticket 31679 + * + * @covers ::dbDelta */ function test_column_type_change_with_hyphens_in_name() { global $wpdb; Index: tests/phpunit/tests/dependencies.php =================================================================== --- tests/phpunit/tests/dependencies.php (revision 48816) +++ tests/phpunit/tests/dependencies.php (working copy) @@ -4,6 +4,11 @@ * @group scripts */ class Tests_Dependencies extends WP_UnitTestCase { + + /** + * + * @covers WP_Dependencies::add + */ function test_add() { $dep = new WP_Dependencies; @@ -16,7 +21,10 @@ // Cannot reuse names. $this->assertFalse( $dep->add( 'one', '' ) ); } - + /** + * + * @covers WP_Dependencies::remove + */ function test_remove() { $dep = new WP_Dependencies; @@ -29,7 +37,10 @@ $this->assertInstanceOf( '_WP_Dependency', $dep->query( 'two' ) ); } - + /** + * + * @covers WP_Dependencies::enqueue + */ function test_enqueue() { $dep = new WP_Dependencies; @@ -45,7 +56,10 @@ $this->assertTrue( $dep->query( 'one', 'queue' ) ); $this->assertTrue( $dep->query( 'two', 'queue' ) ); } - + /** + * + * @covers WP_Dependencies::dequeue + */ function test_dequeue() { $dep = new WP_Dependencies; @@ -65,7 +79,10 @@ $this->assertFalse( $dep->query( 'one', 'queue' ) ); $this->assertFalse( $dep->query( 'two', 'queue' ) ); } - + /** + * + * @covers WP_Dependencies::enqueue + */ function test_enqueue_args() { $dep = new WP_Dependencies; @@ -83,7 +100,10 @@ $this->assertTrue( $dep->query( 'two', 'queue' ) ); $this->assertEquals( 'arg', $dep->args['two'] ); } - + /** + * + * @covers WP_Dependencies::dequeue + */ function test_dequeue_args() { $dep = new WP_Dependencies; @@ -110,6 +130,8 @@ /** * @ticket 21741 + * + * @covers WP_Dependencies::query */ function test_query_and_registered_enqueued() { $dep = new WP_Dependencies; Index: tests/phpunit/tests/dependencies/scripts.php =================================================================== --- tests/phpunit/tests/dependencies/scripts.php (revision 48816) +++ tests/phpunit/tests/dependencies/scripts.php (working copy) @@ -1045,7 +1045,7 @@ * Testing `wp_enqueue_code_editor` with file path. * * @ticket 41871 - * @covers ::wp_enqueue_code_editor() + * @covers ::wp_enqueue_code_editor */ public function test_wp_enqueue_code_editor_when_php_file_will_be_passed() { $real_file = WP_PLUGIN_DIR . '/hello.php'; @@ -1132,7 +1132,7 @@ * Testing `wp_enqueue_code_editor` with `compact`. * * @ticket 41871 - * @covers ::wp_enqueue_code_editor() + * @covers ::wp_enqueue_code_editor */ public function test_wp_enqueue_code_editor_when_generated_array_by_compact_will_be_passed() { $file = ''; @@ -1215,7 +1215,7 @@ * Testing `wp_enqueue_code_editor` with `array_merge`. * * @ticket 41871 - * @covers ::wp_enqueue_code_editor() + * @covers ::wp_enqueue_code_editor */ public function test_wp_enqueue_code_editor_when_generated_array_by_array_merge_will_be_passed() { $wp_enqueue_code_editor = wp_enqueue_code_editor( @@ -1312,7 +1312,7 @@ * Testing `wp_enqueue_code_editor` with `array`. * * @ticket 41871 - * @covers ::wp_enqueue_code_editor() + * @covers ::wp_enqueue_code_editor */ public function test_wp_enqueue_code_editor_when_simple_array_will_be_passed() { $wp_enqueue_code_editor = wp_enqueue_code_editor( Index: tests/phpunit/tests/file.php =================================================================== --- tests/phpunit/tests/file.php (revision 48816) +++ tests/phpunit/tests/file.php (working copy) @@ -16,6 +16,8 @@ /** * @group plugins * @group themes + * + * @cover ::get_file_data */ function test_get_file_data() { $theme_headers = array( @@ -46,6 +48,8 @@ /** * @group plugins * @group themes + * + * @cover ::get_file_data */ function test_get_file_data_cr_line_endings() { $headers = array( @@ -88,6 +92,10 @@ return $result; } + /** + * + * @cover ::wp_unique_filename + */ function test_unique_filename_is_valid() { // Make sure it produces a valid, writable, unique filename. $filename = wp_unique_filename( $this->dir, __FUNCTION__ . '.txt' ); @@ -97,6 +105,10 @@ unlink( $this->dir . DIRECTORY_SEPARATOR . $filename ); } + /** + * + * @cover ::wp_unique_filename + */ function test_unique_filename_is_unique() { // Make sure it produces two unique filenames. $name = __FUNCTION__; @@ -113,6 +125,10 @@ unlink( $this->dir . DIRECTORY_SEPARATOR . $filename2 ); } + /** + * + * @cover ::wp_unique_filename + */ function test_unique_filename_is_sanitized() { $name = __FUNCTION__; $filename = wp_unique_filename( $this->dir, $name . $this->badchars . '.txt' ); @@ -125,6 +141,10 @@ unlink( $this->dir . DIRECTORY_SEPARATOR . $filename ); } + /** + * + * @cover ::wp_unique_filename + */ function test_unique_filename_with_slashes() { $name = __FUNCTION__; // "foo/foo.txt" @@ -138,6 +158,10 @@ unlink( $this->dir . DIRECTORY_SEPARATOR . $filename ); } + /** + * + * @cover ::wp_unique_filename + */ function test_unique_filename_multiple_ext() { $name = __FUNCTION__; $filename = wp_unique_filename( $this->dir, $name . '.php.txt' ); @@ -150,6 +174,10 @@ unlink( $this->dir . DIRECTORY_SEPARATOR . $filename ); } + /** + * + * @cover ::wp_unique_filename + */ function test_unique_filename_no_ext() { $name = __FUNCTION__; $filename = wp_unique_filename( $this->dir, $name ); @@ -163,6 +191,8 @@ /** * @dataProvider data_wp_tempnam_filenames + * + * @cover ::wp_tempnam */ function test_wp_tempnam( $case ) { $file = wp_tempnam( $case ); @@ -183,6 +213,8 @@ /** * @ticket 47186 + * + * @cover ::verify_file_signature */ function test_file_signature_functions_as_expected() { $file = wp_tempnam(); @@ -216,6 +248,8 @@ /** * @ticket 47186 + * + * @cover ::verify_file_signature */ function test_file_signature_expected_failure() { $file = wp_tempnam(); Index: tests/phpunit/tests/filters.php =================================================================== --- tests/phpunit/tests/filters.php (revision 48816) +++ tests/phpunit/tests/filters.php (working copy) @@ -7,6 +7,9 @@ */ class Tests_Filters extends WP_UnitTestCase { + /** + * @covers ::add_filter + */ function test_simple_filter() { $a = new MockAction(); $tag = __FUNCTION__; @@ -25,6 +28,9 @@ $this->assertEquals( array( $val ), $args ); } + /** + * @covers ::remove_filter + */ function test_remove_filter() { $a = new MockAction(); $tag = __FUNCTION__; @@ -45,6 +51,9 @@ } + /** + * @covers ::has_filter + */ function test_has_filter() { $tag = __FUNCTION__; $func = __FUNCTION__ . '_func'; @@ -59,7 +68,11 @@ $this->assertFalse( has_filter( $tag ) ); } - // One tag with multiple filters. + /** + * One tag with multiple filters. + * + * @covers ::add_filter + */ function test_multiple_filters() { $a1 = new MockAction(); $a2 = new MockAction(); @@ -77,6 +90,11 @@ $this->assertEquals( 1, $a2->get_call_count() ); } + /** + * One tag with multiple filters. + * + * @covers ::add_filter + */ function test_filter_args_1() { $a = new MockAction(); $tag = __FUNCTION__; @@ -92,6 +110,11 @@ $this->assertEquals( array( $val, $arg1 ), array_pop( $argsvar ) ); } + /** + * One tag with multiple filters. + * + * @covers ::add_filter + */ function test_filter_args_2() { $a1 = new MockAction(); $a2 = new MockAction(); @@ -117,6 +140,11 @@ $this->assertEquals( array( $val ), array_pop( $argsvar2 ) ); } + /** + * One tag with multiple filters. + * + * @covers ::add_filter + */ function test_filter_priority() { $a = new MockAction(); $tag = __FUNCTION__; @@ -148,6 +176,11 @@ $this->assertEquals( $expected, $a->get_events() ); } + /** + * One tag with multiple filters. + * + * @covers ::add_filter + */ function test_all_filter() { $a = new MockAction(); $tag1 = __FUNCTION__ . '_1'; @@ -172,6 +205,11 @@ } + /** + * One tag with multiple filters. + * + * @covers ::add_filter + */ function test_remove_all_filter() { $a = new MockAction(); $tag = __FUNCTION__; @@ -198,6 +236,8 @@ /** * @ticket 20920 + * + * @covers ::add_filter */ function test_remove_all_filters_should_respect_the_priority_argument() { $a = new MockAction(); @@ -217,6 +257,8 @@ /** * @ticket 9886 + * + * @covers ::add_filter */ function test_filter_ref_array() { $obj = new stdClass(); @@ -236,6 +278,8 @@ /** * @ticket 12723 + * + * @covers ::add_filter */ function test_filter_ref_array_result() { $obj = new stdClass(); @@ -271,6 +315,8 @@ /** * @ticket 29070 + * + * @covers ::has_filter */ function test_has_filter_after_remove_all_filters() { $a = new MockAction(); @@ -298,6 +344,8 @@ /** * @ticket 10441 * @expectedDeprecated tests_apply_filters_deprecated + * + * @covers ::apply_filters_deprecated */ public function test_apply_filters_deprecated() { $p = 'Foo'; @@ -317,6 +365,8 @@ /** * @ticket 10441 * @expectedDeprecated tests_apply_filters_deprecated + * + * @covers ::apply_filters_deprecated */ public function test_apply_filters_deprecated_with_multiple_params() { $p1 = 'Foo1'; @@ -341,6 +391,8 @@ /** * @ticket 10441 + * + * @covers ::apply_filters_deprecated */ public function test_apply_filters_deprecated_without_filter() { $val = 'Foobar'; @@ -351,6 +403,8 @@ private $current_priority; /** * @ticket 39007 + * + * @covers ::add_action */ public function test_current_priority() { add_action( 'test_current_priority', array( $this, '_current_priority_action' ), 99 ); @@ -360,6 +414,9 @@ $this->assertSame( 99, $this->current_priority ); } + /** + * One tag with multiple filters. + */ public function _current_priority_action() { global $wp_filter; $this->current_priority = $wp_filter[ current_filter() ]->current_priority(); @@ -367,6 +424,8 @@ /** * @ticket 39007 + * + * @covers ::add_action */ public function test_other_priority() { add_action( 'test_current_priority', array( $this, '_other_priority_action' ), 99 ); Index: tests/phpunit/tests/functions.php =================================================================== --- tests/phpunit/tests/functions.php (revision 48816) +++ tests/phpunit/tests/functions.php (working copy) @@ -4,6 +4,11 @@ * @group functions.php */ class Tests_Functions extends WP_UnitTestCase { + + /** + * + * @covers ::wp_parse_args + */ function test_wp_parse_args_object() { $x = new MockClass; $x->_baba = 5; @@ -21,6 +26,11 @@ $this->assertEquals( array(), wp_parse_args( $y ) ); } + + /** + * + * @covers ::wp_parse_args + */ function test_wp_parse_args_array() { // Arrays. $a = array(); @@ -40,6 +50,11 @@ ); } + + /** + * + * @covers ::wp_parse_args + */ function test_wp_parse_args_defaults() { $x = new MockClass; $x->_baba = 5; @@ -66,6 +81,11 @@ ); } + + /** + * + * @covers ::wp_parse_args + */ function test_wp_parse_args_other() { $b = true; wp_parse_str( $b, $s ); @@ -77,6 +97,8 @@ /** * @ticket 30753 + * + * @covers ::wp_parse_args */ function test_wp_parse_args_boolean_strings() { $args = wp_parse_args( 'foo=false&bar=true' ); @@ -86,6 +108,8 @@ /** * @ticket 35972 + * + * @covers ::bool_from_yn */ function test_bool_from_yn() { $this->assertTrue( bool_from_yn( 'Y' ) ); @@ -93,6 +117,10 @@ $this->assertFalse( bool_from_yn( 'n' ) ); } + /** + * + * @covers ::path_is_absolute + */ function test_path_is_absolute() { $absolute_paths = array( '/', @@ -111,6 +139,10 @@ } } + /** + * + * @covers ::path_is_absolute + */ function test_path_is_not_absolute() { $relative_paths = array( '', @@ -134,6 +166,7 @@ * @ticket 35996 * * @dataProvider data_wp_normalize_path + * @covers ::wp_normalize_path */ function test_wp_normalize_path( $path, $expected ) { $this->assertEquals( $expected, wp_normalize_path( $path ) ); @@ -163,6 +196,11 @@ ); } + + /** + * + * @covers ::wp_unique_filename + */ function test_wp_unique_filename() { $testdir = DIR_TESTDATA . '/images/'; @@ -198,6 +236,8 @@ /** * @ticket 42437 + * + * @covers ::wp_unique_filename */ function test_unique_filename_with_dimension_like_filename() { $testdir = DIR_TESTDATA . '/images/'; @@ -223,6 +263,8 @@ /** * @dataProvider data_is_not_serialized + * + * @covers ::serialize */ function test_maybe_serialize( $value ) { if ( is_array( $value ) || is_object( $value ) ) { @@ -236,6 +278,8 @@ /** * @dataProvider data_is_serialized + * + * @covers ::serialize */ function test_maybe_serialize_with_double_serialization( $value ) { $expected = serialize( $value ); @@ -246,6 +290,8 @@ /** * @dataProvider data_is_serialized * @dataProvider data_is_not_serialized + * + * @covers ::maybe_unserialize */ function test_maybe_unserialize( $value, $is_serialized ) { if ( $is_serialized ) { @@ -264,6 +310,8 @@ /** * @dataProvider data_is_serialized * @dataProvider data_is_not_serialized + * + * @covers ::is_serialized */ function test_is_serialized( $value, $expected ) { $this->assertSame( $expected, is_serialized( $value ) ); @@ -324,6 +372,8 @@ /** * @ticket 46570 * @dataProvider data_is_serialized_should_return_true_for_large_floats + * + * @covers ::is_serialized */ function test_is_serialized_should_return_true_for_large_floats( $value ) { $this->assertTrue( is_serialized( $value ) ); @@ -338,6 +388,8 @@ /** * @ticket 17375 + * + * @covers ::is_serialized */ function test_no_new_serializable_types() { $this->assertFalse( is_serialized( 'C:16:"Serialized_Class":6:{a:0:{}}' ) ); @@ -345,6 +397,8 @@ /** * @group add_query_arg + * + * @covers ::add_query_arg */ function test_add_query_arg() { $old_req_uri = $_SERVER['REQUEST_URI']; @@ -533,6 +587,8 @@ /** * @ticket 31306 + * + * @covers ::add_query_arg */ function test_add_query_arg_numeric_keys() { $url = add_query_arg( array( 'foo' => 'bar' ), '1=1' ); @@ -553,6 +609,8 @@ /** * @ticket 21594 + * + * @covers ::get_allowed_mime_types */ function test_get_allowed_mime_types() { $mimes = get_allowed_mime_types(); @@ -573,6 +631,8 @@ /** * @ticket 21594 + * + * @covers ::wp_get_mime_types */ function test_wp_get_mime_types() { $mimes = wp_get_mime_types(); @@ -605,6 +665,9 @@ /** * @ticket 23688 + * + * @covers ::get_option + * @covers ::_canonical_charset */ function test_canonical_charset() { $orig_blog_charset = get_option( 'blog_charset' ); @@ -643,6 +706,8 @@ /** * @ticket 43977 * @dataProvider data_wp_parse_list + * + * @covers ::wp_parse_list */ function test_wp_parse_list( $expected, $actual ) { $this->assertSame( $expected, array_values( wp_parse_list( $actual ) ) ); @@ -666,6 +731,8 @@ /** * @dataProvider data_wp_parse_id_list + * + * @covers ::wp_parse_id_list */ function test_wp_parse_id_list( $expected, $actual ) { $this->assertSame( $expected, array_values( wp_parse_id_list( $actual ) ) ); @@ -701,6 +768,8 @@ /** * @dataProvider data_device_can_upload + * + * @covers ::_device_can_upload */ function test_device_can_upload( $user_agent, $expected ) { $_SERVER['HTTP_USER_AGENT'] = $user_agent; @@ -761,6 +830,8 @@ /** * @ticket 9064 + * + * @covers ::wp_extract_urls */ function test_wp_extract_urls() { $original_urls = array( @@ -928,6 +999,8 @@ /** * @ticket 28786 + * + * @covers ::wp_json_encode */ function test_wp_json_encode() { $this->assertEquals( wp_json_encode( 'a' ), '"a"' ); @@ -935,6 +1008,8 @@ /** * @ticket 28786 + * + * @covers ::wp_json_encode */ function test_wp_json_encode_utf8() { $this->assertEquals( wp_json_encode( 'è¿™' ), '"\u8fd9"' ); @@ -942,6 +1017,8 @@ /** * @ticket 28786 + * + * @covers ::wp_json_encode */ function test_wp_json_encode_non_utf8() { if ( ! function_exists( 'mb_detect_order' ) ) { @@ -967,6 +1044,8 @@ /** * @ticket 28786 + * + * @covers ::wp_json_encode */ function test_wp_json_encode_non_utf8_in_array() { if ( ! function_exists( 'mb_detect_order' ) ) { @@ -992,6 +1071,8 @@ /** * @ticket 28786 + * + * @covers ::wp_json_encode */ function test_wp_json_encode_array() { $this->assertEquals( wp_json_encode( array( 'a' ) ), '["a"]' ); @@ -999,6 +1080,8 @@ /** * @ticket 28786 + * + * @covers ::wp_json_encode */ function test_wp_json_encode_object() { $object = new stdClass; @@ -1008,6 +1091,8 @@ /** * @ticket 28786 + * + * @covers ::wp_json_encode */ function test_wp_json_encode_depth() { $data = array( array( array( 1, 2, 3 ) ) ); @@ -1022,6 +1107,8 @@ /** * @ticket 36054 * @dataProvider datetime_provider + * + * @covers ::mysql_to_rfc3339 */ function test_mysql_to_rfc3339( $expected, $actual ) { $date_return = mysql_to_rfc3339( $actual ); @@ -1044,6 +1131,8 @@ /** * @ticket 35987 + * + * @covers ::wp_get_ext_types */ public function test_wp_get_ext_types() { $extensions = wp_get_ext_types(); @@ -1063,6 +1152,8 @@ /** * @ticket 35987 + * + * @covers ::wp_get_ext_types */ public function test_wp_ext2type() { $extensions = wp_get_ext_types(); @@ -1084,6 +1175,8 @@ * test suite is -1, we can not test the memory limit negotiations. * * @ticket 32075 + * + * @covers ::wp_raise_memory_limit */ function test_wp_raise_memory_limit() { if ( -1 !== WP_MAX_MEMORY_LIMIT ) { @@ -1102,8 +1195,9 @@ /** * Tests wp_generate_uuid4(). * + * @ticket 38164 + * * @covers ::wp_generate_uuid4 - * @ticket 38164 */ function test_wp_generate_uuid4() { $uuids = array(); @@ -1120,8 +1214,9 @@ /** * Tests wp_is_uuid(). * + * @ticket 39778 + * * @covers ::wp_is_uuid - * @ticket 39778 */ function test_wp_is_valid_uuid() { $uuids_v4 = array( @@ -1170,8 +1265,9 @@ /** * Tests wp_unique_id(). * + * @ticket 44883 + * * @covers ::wp_unique_id - * @ticket 44883 */ function test_wp_unique_id() { @@ -1198,6 +1294,8 @@ /** * @ticket 40017 * @dataProvider _wp_get_image_mime + * + * @covers ::wp_get_image_mime */ public function test_wp_get_image_mime( $file, $expected ) { if ( ! is_callable( 'exif_imagetype' ) && ! function_exists( 'getimagesize' ) ) { @@ -1210,6 +1308,8 @@ /** * @ticket 39550 * @dataProvider _wp_check_filetype_and_ext_data + * + * @covers ::wp_check_filetype_and_ext */ function test_wp_check_filetype_and_ext( $file, $filename, $expected ) { if ( ! extension_loaded( 'fileinfo' ) ) { @@ -1222,6 +1322,8 @@ /** * @ticket 39550 * @group ms-excluded + * + * @covers ::wp_check_filetype_and_ext */ function test_wp_check_filetype_and_ext_with_filtered_svg() { if ( ! extension_loaded( 'fileinfo' ) ) { @@ -1247,6 +1349,8 @@ /** * @ticket 39550 * @group ms-excluded + * + * @covers ::wp_check_filetype_and_ext */ function test_wp_check_filetype_and_ext_with_filtered_woff() { if ( ! extension_loaded( 'fileinfo' ) ) { @@ -1488,6 +1592,8 @@ * @param string $file File path. * @param array $allowed_files List of allowed files. * @param int $expected Expected result. + * + * @covers ::validate_file */ public function test_validate_file( $file, $allowed_files, $expected ) { $this->assertSame( $expected, validate_file( $file, $allowed_files ) ); @@ -1630,6 +1736,8 @@ * * @param string $path The resource path or URL. * @param bool $expected Expected result. + * + * @covers ::wp_is_stream */ public function test_wp_is_stream( $path, $expected ) { if ( ! extension_loaded( 'openssl' ) && false !== strpos( $path, 'https://' ) ) { @@ -1674,6 +1782,8 @@ * * @param string $input Duration. * @param string $expected Expected human readable duration. + * + * @covers ::human_readable_duration */ public function test_human_readable_duration( $input, $expected ) { $this->assertSame( $expected, human_readable_duration( $input ) ); Index: tests/phpunit/tests/kses.php =================================================================== --- tests/phpunit/tests/kses.php (revision 48816) +++ tests/phpunit/tests/kses.php (working copy) @@ -9,6 +9,8 @@ /** * @ticket 20210 + * + * @covers ::wp_kses */ function test_wp_filter_post_kses_address() { global $allowedposttags; @@ -36,6 +38,8 @@ /** * @ticket 20210 + * + * @covers ::wp_kses */ function test_wp_filter_post_kses_a() { global $allowedposttags; @@ -77,6 +81,8 @@ * @param string $source Source HTML. * @param string $context Context to use for parsing source. * @param string $expected Expected output following KSES parsing. + * + * @covers ::wp_kses */ function test_wp_kses_video( $source, $context, $expected ) { $actual = wp_kses( $source, $context ); @@ -123,6 +129,8 @@ /** * @ticket 20210 + * + * @covers ::wp_kses */ function test_wp_filter_post_kses_abbr() { global $allowedposttags; @@ -141,6 +149,10 @@ } } + /** + * + * @covers ::wp_kses + */ function test_feed_links() { global $allowedposttags; @@ -175,6 +187,10 @@ $this->assertEquals( $expected, wp_kses( $content, $allowedposttags ) ); } + /** + * + * @covers ::wp_kses_bad_protocol + */ function test_wp_kses_bad_protocol() { $bad = array( 'dummy:alert(1)', @@ -277,6 +293,10 @@ } } + /** + * + * @covers ::wp_kses_data + */ public function test_hackers_attacks() { $xss = simplexml_load_file( DIR_TESTDATA . '/formatting/xssAttacks.xml' ); foreach ( $xss->attack as $attack ) { @@ -421,6 +441,8 @@ /** * @ticket 20210 + * + * @covers ::wp_kses_allowed_html */ public function test_wp_kses_allowed_html() { global $allowedposttags, $allowedtags, $allowedentitynames; @@ -470,6 +492,10 @@ $this->assertEquals( $allowedtags, wp_kses_allowed_html( 'data' ) ); } + /** + * + * @covers ::wp_kses_post + */ function test_hyphenated_tag() { $string = 'Alot of hyphens.'; $custom_tags = array( @@ -486,6 +512,8 @@ /** * @ticket 26290 + * + * @covers ::wp_kses_normalize_entities */ public function test_wp_kses_normalize_entities() { $this->assertEquals( '♠', wp_kses_normalize_entities( '♠' ) ); @@ -504,6 +532,8 @@ * * @ticket 28506 * @dataProvider data_ctrl_removal + * + * @covers ::wp_kses */ function test_ctrl_removal( $input, $output ) { global $allowedposttags; @@ -541,6 +571,8 @@ * * @ticket 28699 * @dataProvider data_slash_zero_removal + * + * @covers ::wp_kses */ function test_slash_zero_removal( $input, $output ) { global $allowedposttags; @@ -593,6 +625,8 @@ * Test new function wp_kses_hair_parse(). * * @dataProvider data_hair_parse + * + * @covers ::wp_kses_hair_parse */ function test_hair_parse( $input, $output ) { return $this->assertEquals( $output, wp_kses_hair_parse( $input ) ); @@ -659,6 +693,8 @@ * Test new function wp_kses_attr_parse(). * * @dataProvider data_attr_parse + * + * @covers ::wp_kses_attr_parse */ function test_attr_parse( $input, $output ) { return $this->assertEquals( $output, wp_kses_attr_parse( $input ) ); @@ -717,6 +753,8 @@ * Test new function wp_kses_one_attr(). * * @dataProvider data_one_attr + * + * @covers ::wp_kses_one_attr */ function test_one_attr( $element, $input, $output ) { return $this->assertEquals( $output, wp_kses_one_attr( $input, $element ) ); @@ -794,6 +832,8 @@ /** * @ticket 34063 + * + * @covers ::wp_kses */ function test_bdo() { global $allowedposttags; @@ -805,6 +845,8 @@ /** * @ticket 35079 + * + * @covers ::wp_kses */ function test_ol_reversed() { global $allowedposttags; @@ -816,6 +858,8 @@ /** * @ticket 40680 + * + * @covers ::wp_kses_attr */ function test_wp_kses_attr_no_attributes_allowed_with_empty_array() { $element = 'foo'; @@ -826,6 +870,8 @@ /** * @ticket 40680 + * + * @covers ::wp_kses_attr */ function test_wp_kses_attr_no_attributes_allowed_with_true() { $element = 'foo'; @@ -836,6 +882,8 @@ /** * @ticket 40680 + * + * @covers ::wp_kses_attr */ function test_wp_kses_attr_single_attribute_is_allowed() { $element = 'foo'; @@ -846,6 +894,8 @@ /** * @ticket 43312 + * + * @covers ::wp_kses_attr */ function test_wp_kses_attr_no_attributes_allowed_with_false() { $element = 'foo'; @@ -864,6 +914,8 @@ * * @param string $css A string of CSS rules. * @param string $expected Expected string of CSS rules. + * + * @covers ::safecss_filter_attr */ public function test_safecss_filter_attr( $css, $expected ) { $this->assertSame( $expected, safecss_filter_attr( $css ) ); @@ -1052,6 +1104,8 @@ * Data attributes are globally accepted. * * @ticket 33121 + * + * @covers ::wp_kses_post */ function test_wp_kses_attr_data_attribute_is_allowed() { $test = '
Pens and pencils
'; @@ -1064,6 +1118,8 @@ * Ensure wildcard attributes block unprefixed wildcard uses. * * @ticket 33121 + * + * @covers ::wp_kses */ function test_wildcard_requires_hyphen_after_prefix() { $allowed_html = array( @@ -1085,6 +1141,8 @@ * Ensure wildcard allows two hyphen. * * @ticket 33121 + * + * @covers ::wp_kses */ function test_wildcard_allows_two_hyphens() { $allowed_html = array( @@ -1107,6 +1165,8 @@ * @dataProvider data_wildcard_attribute_prefixes * * @ticket 33121 + * + * @covers ::wp_kses_attr_check */ function test_wildcard_attribute_prefixes( $wildcard_attribute, $expected ) { $allowed_html = array( @@ -1152,6 +1212,8 @@ * * @param $input string The style attribute saved in the editor. * @param $expected string The sanitized style attribute. + * + * @covers ::safecss_filter_attr */ function test_kses_style_attr_with_url( $input, $expected ) { $actual = safecss_filter_attr( $input ); @@ -1299,6 +1361,8 @@ * * @param string $css A string of CSS rules. * @param string $expected Expected string of CSS rules. + * + * @covers ::safecss_filter_attr */ public function test_safecss_filter_attr_filtered( $css, $expected ) { add_filter( 'safecss_filter_attr_allow_css', '__return_true' ); @@ -1366,6 +1430,8 @@ * Test filtering a standard img tag. * * @ticket 50731 + * + * @covers ::wp_kses_post */ function test_wp_kses_img_tag_standard_attributes() { $html = array( Index: tests/phpunit/tests/l10n.php =================================================================== --- tests/phpunit/tests/l10n.php (revision 48816) +++ tests/phpunit/tests/l10n.php (working copy) @@ -17,6 +17,8 @@ /** * @ticket 35961 + * + * @covers ::_n_noop */ function test_n_noop() { $text_domain = 'text-domain'; @@ -30,6 +32,8 @@ /** * @ticket 35961 + * + * @covers ::_nx_noop */ function test_nx_noop() { $text_domain = 'text-domain'; @@ -44,6 +48,8 @@ /** * @ticket 35073 + * + * @covers ::before_last_bar */ function test_before_last_bar() { $this->assertEquals( 'no-bar-at-all', before_last_bar( 'no-bar-at-all' ) ); @@ -53,6 +59,8 @@ /** * @ticket 35950 + * + * @covers ::get_available_languages */ function test_get_available_languages() { $array = get_available_languages(); @@ -67,6 +75,8 @@ /** * @ticket 35284 + * + * @covers ::wp_get_installed_translations */ function test_wp_get_installed_translations_for_core() { $installed_translations = wp_get_installed_translations( 'core' ); @@ -89,6 +99,8 @@ /** * @ticket 35294 + * + * @covers ::wp_dropdown_languages */ function test_wp_dropdown_languages() { $args = array( @@ -111,6 +123,8 @@ /** * @ticket 38632 + * + * @covers ::wp_dropdown_languages */ function test_wp_dropdown_languages_site_default() { $args = array( @@ -135,6 +149,8 @@ /** * @ticket 44494 + * + * @covers ::wp_dropdown_languages */ function test_wp_dropdown_languages_exclude_en_us() { $args = array( @@ -153,6 +169,8 @@ /** * @ticket 38632 + * + * @covers ::wp_dropdown_languages */ function test_wp_dropdown_languages_en_US_selected() { $args = array( @@ -175,6 +193,8 @@ /** * Add site default language to ja_JP in dropdown + * + * @covers ::wp_dropdown_languages */ function test_wp_dropdown_languages_site_default_ja_JP() { $args = array( @@ -199,6 +219,8 @@ /** * Select dropdown language from de_DE to ja_JP + * + * @covers ::wp_dropdown_languages */ function test_wp_dropdown_languages_ja_JP_selected() { $args = array( @@ -246,6 +268,8 @@ /** * @ticket 35284 + * + * @covers ::wp_get_pomo_file_data */ function test_wp_get_pomo_file_data() { $file = DIR_TESTDATA . '/pomo/empty.po'; @@ -272,6 +296,8 @@ /** * @ticket 44541 + * + * @covers ::the_excerpt */ function test_length_of_excerpt_should_be_counted_by_words() { global $post; @@ -296,6 +322,8 @@ /** * @ticket 44541 + * + * @covers ::the_excerpt */ function test_length_of_excerpt_should_be_counted_by_chars() { global $post; @@ -320,6 +348,8 @@ /** * @ticket 44541 + * + * @covers ::the_excerpt */ function test_length_of_excerpt_should_be_counted_by_chars_in_japanese() { global $post; @@ -344,6 +374,8 @@ /** * @ticket 44541 + * + * @covers ::the_excerpt_rss */ function test_length_of_excerpt_rss_should_be_counted_by_words() { global $post; @@ -368,6 +400,8 @@ /** * @ticket 44541 + * + * @covers ::the_excerpt_rss */ function test_length_of_excerpt_rss_should_be_counted_by_chars() { global $post; @@ -393,6 +427,8 @@ /** * @ticket 44541 + * + * @covers ::wp_dashboard_recent_drafts */ function test_length_of_draft_should_be_counted_by_words() { require_once ABSPATH . 'wp-admin/includes/dashboard.php'; @@ -417,6 +453,8 @@ /** * @ticket 44541 + * + * @covers ::wp_dashboard_recent_drafts */ function test_length_of_draft_should_be_counted_by_chars() { require_once ABSPATH . 'wp-admin/includes/dashboard.php'; @@ -441,6 +479,8 @@ /** * @ticket 44541 + * + * @covers ::wp_dashboard_recent_drafts */ function test_length_of_draft_should_be_counted_by_chars_in_japanese() { require_once ABSPATH . 'wp-admin/includes/dashboard.php'; @@ -465,6 +505,8 @@ /** * @ticket 44541 + * + * @covers ::get_comment_excerpt */ function test_length_of_comment_excerpt_should_be_counted_by_words() { switch_to_locale( 'en_US' ); @@ -483,6 +525,8 @@ /** * @ticket 44541 + * + * @covers ::get_comment_excerpt */ function test_length_of_comment_excerpt_should_be_counted_by_chars() { switch_to_locale( 'ja_JP' ); @@ -501,6 +545,8 @@ /** * @ticket 44541 + * + * @covers ::get_comment_excerpt */ function test_length_of_comment_excerpt_should_be_counted_by_chars_in_Japanese() { switch_to_locale( 'ja_JP' ); Index: tests/phpunit/tests/link.php =================================================================== --- tests/phpunit/tests/link.php (revision 48816) +++ tests/phpunit/tests/link.php (working copy) @@ -10,6 +10,8 @@ /** * @ticket 8847 + * + * @covers ::get_pagenum_link */ function test_get_pagenum_link_case_insensitivity() { $old_req_uri = $_SERVER['REQUEST_URI']; @@ -26,6 +28,10 @@ $_SERVER['REQUEST_URI'] = $old_req_uri; } + /** + * + * @covers ::wp_get_shortlink + */ function test_wp_get_shortlink() { $post_id = self::factory()->post->create(); $post_id2 = self::factory()->post->create(); @@ -70,6 +76,10 @@ $this->assertEquals( home_url( '?p=' . $post_id ), wp_get_shortlink() ); } + /** + * + * @covers ::wp_get_shortlink + */ function test_wp_get_shortlink_with_page() { $post_id = self::factory()->post->create( array( 'post_type' => 'page' ) ); @@ -84,6 +94,8 @@ /** * @ticket 26871 + * + * @covers ::wp_get_shortlink */ function test_wp_get_shortlink_with_home_page() { $post_id = self::factory()->post->create( array( 'post_type' => 'page' ) ); @@ -99,6 +111,8 @@ /** * @ticket 30910 + * + * @covers ::get_permalink */ public function test_get_permalink_should_not_reveal_post_name_for_post_with_post_status_future() { update_option( 'permalink_structure', '/%year%/%monthnum%/%day%/%postname%/' ); @@ -114,11 +128,13 @@ $non_pretty_permalink = add_query_arg( 'p', $p, trailingslashit( home_url() ) ); - $this->assertEquals( $non_pretty_permalink, get_permalink( $p ) ); + $this->assertSame( $non_pretty_permalink, get_permalink( $p ) ); } /** * @ticket 30910 + * + * @covers ::get_permalink */ public function test_get_permalink_should_not_reveal_post_name_for_cpt_with_post_status_future() { update_option( 'permalink_structure', '/%year%/%monthnum%/%day%/%postname%/' ); @@ -148,6 +164,8 @@ /** * @ticket 1914 + * + * @covers ::get_permalink */ public function test_unattached_attachment_has_a_pretty_permalink() { $this->set_permalink_structure( '/%year%/%monthnum%/%day%/%postname%/' ); @@ -170,6 +188,8 @@ /** * @ticket 1914 + * + * @covers ::get_permalink */ public function test_attachment_attached_to_non_existent_post_type_has_a_pretty_permalink() { global $wp_post_types; Index: tests/phpunit/tests/locale.php =================================================================== --- tests/phpunit/tests/locale.php (revision 48816) +++ tests/phpunit/tests/locale.php (working copy) @@ -15,6 +15,10 @@ $this->locale = new WP_Locale(); } + /** + * + * @covers WP_Locale::get_weekday + */ public function test_get_weekday() { $this->assertEquals( __( 'Sunday' ), $this->locale->get_weekday( 0 ) ); $this->assertEquals( __( 'Monday' ), $this->locale->get_weekday( 1 ) ); @@ -27,11 +31,17 @@ /** * @expectedException PHPUnit_Framework_Error_Notice + * + * @covers WP_Locale::get_weekday */ public function test_get_weekday_undefined_index() { $this->locale->get_weekday( 7 ); } + /** + * + * @covers WP_Locale::get_weekday_initial + */ public function test_get_weekday_initial() { $this->assertEquals( __( 'S' ), $this->locale->get_weekday_initial( __( 'Sunday' ) ) ); $this->assertEquals( __( 'M' ), $this->locale->get_weekday_initial( __( 'Monday' ) ) ); @@ -42,6 +52,10 @@ $this->assertEquals( __( 'S' ), $this->locale->get_weekday_initial( __( 'Saturday' ) ) ); } + /** + * + * @covers WP_Locale::get_weekday_abbrev + */ public function test_get_weekday_abbrev() { $this->assertEquals( __( 'Sun' ), $this->locale->get_weekday_abbrev( __( 'Sunday' ) ) ); $this->assertEquals( __( 'Mon' ), $this->locale->get_weekday_abbrev( __( 'Monday' ) ) ); @@ -52,6 +66,10 @@ $this->assertEquals( __( 'Sat' ), $this->locale->get_weekday_abbrev( __( 'Saturday' ) ) ); } + /** + * + * @covers WP_Locale::get_month + */ public function test_get_month() { $this->assertEquals( __( 'January' ), $this->locale->get_month( 1 ) ); $this->assertEquals( __( 'February' ), $this->locale->get_month( 2 ) ); @@ -67,6 +85,10 @@ $this->assertEquals( __( 'December' ), $this->locale->get_month( 12 ) ); } + /** + * + * @covers WP_Locale::get_month + */ public function test_get_month_leading_zero() { $this->assertEquals( __( 'January' ), $this->locale->get_month( '01' ) ); $this->assertEquals( __( 'February' ), $this->locale->get_month( '02' ) ); @@ -79,6 +101,10 @@ $this->assertEquals( __( 'September' ), $this->locale->get_month( '09' ) ); } + /** + * + * @covers WP_Locale::get_month_abbrev + */ public function test_get_month_abbrev() { $this->assertEquals( __( 'Jan' ), $this->locale->get_month_abbrev( __( 'January' ) ) ); $this->assertEquals( __( 'Feb' ), $this->locale->get_month_abbrev( __( 'February' ) ) ); @@ -94,6 +120,10 @@ $this->assertEquals( __( 'Dec' ), $this->locale->get_month_abbrev( __( 'December' ) ) ); } + /** + * + * @covers WP_Locale::get_meridiem + */ public function test_get_meridiem() { $this->assertEquals( __( 'am' ), $this->locale->get_meridiem( 'am' ) ); $this->assertEquals( __( 'AM' ), $this->locale->get_meridiem( 'AM' ) ); @@ -101,6 +131,10 @@ $this->assertEquals( __( 'PM' ), $this->locale->get_meridiem( 'PM' ) ); } + /** + * + * @covers WP_Locale::is_rtl + */ public function test_is_rtl() { $this->assertFalse( $this->locale->is_rtl() ); $this->locale->text_direction = 'foo'; Index: tests/phpunit/tests/mail.php =================================================================== --- tests/phpunit/tests/mail.php (revision 48816) +++ tests/phpunit/tests/mail.php (working copy) @@ -21,6 +21,8 @@ * when it encounters a line longer than 999 characters. But PHPMailer doesn't clean up after itself / presets * all variables, which means that following tests would fail. To solve this issue we set `$this->Encoding` * back to 8bit in `MockPHPMailer::preSend`. + * + * @covers ::wp_mail */ function test_wp_mail_break_it() { $content = str_repeat( 'A', 1000 ); @@ -27,6 +29,10 @@ $this->assertTrue( wp_mail( WP_TESTS_EMAIL, 'Looong line testing', $content ) ); } + /** + * + * @covers ::wp_mail + */ function test_wp_mail_custom_boundaries() { $to = 'user@example.com'; $subject = 'Test email with custom boundaries'; @@ -87,6 +93,8 @@ /** * @ticket 17305 + * + * @covers ::wp_mail */ function test_wp_mail_rfc2822_addresses() { $to = 'Name '; @@ -117,6 +125,8 @@ /** * @ticket 17305 + * + * @covers ::wp_mail */ function test_wp_mail_multiple_rfc2822_to_addresses() { $to = 'Name , Another Name '; @@ -135,6 +145,10 @@ $this->assertEqualsIgnoreEOL( $message . "\n", $mailer->get_sent()->body ); } + /** + * + * @covers ::wp_mail + */ function test_wp_mail_multiple_to_addresses() { $to = 'address@tld.com, another_address@different-tld.com'; $subject = 'RFC2822 Testing'; @@ -150,6 +164,8 @@ /** * @ticket 18463 + * + * @covers ::wp_mail */ function test_wp_mail_to_address_no_name() { $to = ''; @@ -165,6 +181,8 @@ /** * @ticket 23642 + * + * @covers ::wp_mail */ function test_wp_mail_return_value() { // No errors. @@ -179,6 +197,8 @@ /** * @ticket 30266 + * + * @covers ::wp_mail */ public function test_wp_mail_with_valid_from_header() { $to = 'address@tld.com'; @@ -195,6 +215,8 @@ /** * @ticket 30266 + * + * @covers ::wp_mail */ public function test_wp_mail_with_empty_from_header() { $to = 'address@tld.com'; @@ -211,6 +233,8 @@ /** * @ticket 30266 + * + * @covers ::wp_mail */ public function test_wp_mail_with_empty_from_name_for_the_from_header() { $to = 'address@tld.com'; @@ -227,6 +251,8 @@ /** * @ticket 30266 + * + * @covers ::wp_mail */ public function test_wp_mail_with_valid_content_type_header() { $to = 'address@tld.com'; @@ -243,6 +269,8 @@ /** * @ticket 30266 + * + * @covers ::wp_mail */ public function test_wp_mail_with_empty_content_type_header() { $to = 'address@tld.com'; @@ -259,6 +287,8 @@ /** * @ticket 30266 + * + * @covers ::wp_mail */ public function test_wp_mail_with_empty_charset_for_the_content_type_header() { $to = 'address@tld.com'; @@ -275,6 +305,8 @@ /** * @ticket 43542 + * + * @covers ::wp_mail */ public function test_wp_mail_does_not_duplicate_mime_version_header() { $to = 'user@example.com'; @@ -308,6 +340,8 @@ * https://tools.ietf.org/html/rfc2045#section-6.1 * * @ticket 28039 + * + * @covers ::wp_mail */ function test_wp_mail_content_transfer_encoding_in_quoted_printable_multipart() { add_action( 'phpmailer_init', array( $this, 'wp_mail_quoted_printable' ) ); @@ -325,6 +359,8 @@ /** * @ticket 21659 + * + * @covers ::wp_mail */ public function test_wp_mail_addresses_arent_encoded() { $to = 'Lukáš To '; @@ -367,6 +403,8 @@ * set it correctly. * * @ticket 37736 + * + * @covers ::wp_mail */ public function test_wp_mail_sender_not_set() { wp_mail( 'user@example.org', 'Testing the Sender field', 'The Sender field should not have been set.' ); @@ -378,6 +416,8 @@ /** * @ticket 35598 + * + * @covers ::wp_mail */ public function test_phpmailer_exception_thrown() { $to = 'an_invalid_address'; @@ -410,6 +450,8 @@ /** * @ticket 50720 + * + * @coversNothing */ function test_phpmailer_validator() { $phpmailer = $GLOBALS['phpmailer']; Index: tests/phpunit/tests/media.php =================================================================== --- tests/phpunit/tests/media.php (revision 48816) +++ tests/phpunit/tests/media.php (working copy) @@ -46,6 +46,9 @@ ); } + /** + * @covers ::add_shortcode + */ function test_img_caption_shortcode_added() { global $shortcode_tags; $this->assertEquals( 'img_caption_shortcode', $shortcode_tags['caption'] ); @@ -52,6 +55,9 @@ $this->assertEquals( 'img_caption_shortcode', $shortcode_tags['wp_caption'] ); } + /** + * @covers ::img_caption_shortcode + */ function test_img_caption_shortcode_with_empty_params() { $result = img_caption_shortcode( array() ); $this->assertNull( $result ); @@ -59,6 +65,8 @@ /** * @ticket 33981 + * + * @covers ::img_caption_shortcode */ function test_img_caption_shortcode_with_empty_params_but_content() { $result = img_caption_shortcode( array(), $this->caption ); @@ -67,6 +75,8 @@ /** * @ticket 33981 + * + * @covers ::img_caption_shortcode */ function test_img_caption_shortcode_short_circuit_filter() { add_filter( 'img_caption_shortcode', array( $this, '_return_alt_caption' ) ); @@ -84,6 +94,8 @@ /** * @ticket 33981 + * + * @covers ::img_caption_shortcode */ function test_img_caption_shortcode_empty_width() { $result = img_caption_shortcode( @@ -97,6 +109,8 @@ /** * @ticket 33981 + * + * @covers ::img_caption_shortcode */ function test_img_caption_shortcode_empty_caption() { $result = img_caption_shortcode( @@ -109,6 +123,8 @@ /** * @ticket 33981 + * + * @covers ::img_caption_shortcode */ function test_img_caption_shortcode_empty_caption_and_content() { $result = img_caption_shortcode( @@ -120,6 +136,9 @@ $this->assertEquals( $this->caption, $result ); } + /** + * @covers ::img_caption_shortcode + */ function test_img_caption_shortcode_with_old_format() { $result = img_caption_shortcode( array( @@ -139,6 +158,9 @@ } } + /** + * @covers ::img_caption_shortcode + */ function test_img_caption_shortcode_with_old_format_id_and_align() { $result = img_caption_shortcode( array( @@ -153,6 +175,9 @@ $this->assertEquals( 1, preg_match_all( "/{$this->caption}/", $result, $_r ) ); } + /** + * @covers ::img_caption_shortcode + */ function test_img_caption_shortcode_with_old_format_and_class() { $result = img_caption_shortcode( array( @@ -165,6 +190,9 @@ } + /** + * @covers ::img_caption_shortcode + */ function test_new_img_caption_shortcode_with_html_caption() { $result = img_caption_shortcode( array( @@ -177,6 +205,9 @@ $this->assertEquals( 1, preg_match_all( "~{$our_preg}~", $result, $_r ) ); } + /** + * @covers ::img_caption_shortcode + */ function test_new_img_caption_shortcode_new_format() { $result = img_caption_shortcode( array( 'width' => 20 ), @@ -189,6 +220,9 @@ $this->assertEquals( 1, preg_match_all( "~wp-caption-text.*{$content_preg}~", $result, $_r ) ); } + /** + * @covers ::img_caption_shortcode + */ function test_new_img_caption_shortcode_new_format_and_linked_image() { $linked_image = "{$this->img_content}"; $result = img_caption_shortcode( @@ -202,6 +236,9 @@ $this->assertEquals( 1, preg_match_all( "~wp-caption-text.*{$content_preg}~", $result, $_r ) ); } + /** + * @covers ::img_caption_shortcode + */ function test_new_img_caption_shortcode_new_format_and_linked_image_with_newline() { $linked_image = "{$this->img_content}"; $result = img_caption_shortcode( @@ -217,6 +254,8 @@ /** * @ticket 34595 + * + * @covers ::img_caption_shortcode */ function test_img_caption_shortcode_has_aria_describedby() { $result = img_caption_shortcode( @@ -230,6 +269,9 @@ $this->assertEquals( 1, preg_match_all( '/aria-describedby="caption-myId"/', $result, $_r ) ); } + /** + * @covers ::wp_oembed_remove_provider + */ function test_add_remove_oembed_provider() { wp_oembed_add_provider( 'http://foo.bar/*', 'http://foo.bar/oembed' ); $this->assertTrue( wp_oembed_remove_provider( 'http://foo.bar/*' ) ); @@ -238,6 +280,8 @@ /** * @ticket 23776 + * + * @covers WP_Embed::autoembed */ function test_autoembed_empty() { global $wp_embed; @@ -250,6 +294,8 @@ /** * @ticket 23776 + * + * @covers WP_Embed::autoembed */ function test_autoembed_no_paragraphs_around_urls() { global $wp_embed; @@ -329,6 +375,8 @@ /** * @dataProvider data_autoembed + * + * @covers WP_Embed::autoembed */ function test_autoembed( $content, $result = null ) { $wp_embed = new Test_Autoembed; @@ -336,6 +384,9 @@ $this->assertEquals( $wp_embed->autoembed( $content ), $result ? $result : $content ); } + /** + * @covers ::wp_prepare_attachment_for_js + */ function test_wp_prepare_attachment_for_js() { // Attachment without media. $id = wp_insert_attachment( @@ -389,6 +440,8 @@ /** * @ticket 38965 + * + * @covers ::wp_prepare_attachment_for_js */ function test_wp_prepare_attachment_for_js_without_image_sizes() { // Create the attachement post. @@ -420,6 +473,9 @@ /** * @ticket 19067 * @expectedDeprecated wp_convert_bytes_to_hr + * + * @covers ::wp_convert_bytes_to_hr + * @covers ::size_format */ function test_wp_convert_bytes_to_hr() { $kb = 1024; @@ -458,6 +514,8 @@ /** * @ticket 22960 + * + * @covers ::get_attached_media */ function test_get_attached_images() { $post_id = self::factory()->post->create(); @@ -476,6 +534,8 @@ /** * @ticket 22960 + * + * @covers ::wp_update_attachment_metadata */ function test_post_galleries_images() { $ids1 = array(); @@ -527,6 +587,8 @@ /** * @ticket 39304 + * + * @covers ::get_post_galleries */ function test_post_galleries_images_without_global_post() { // Set up an unattached image. @@ -552,6 +614,8 @@ /** * @ticket 39304 + * + * @covers ::get_post_galleries */ function test_post_galleries_ignores_global_post() { $global_post_id = $this->factory->post->create( @@ -587,6 +651,8 @@ /** * @ticket 39304 + * + * @covers ::get_post_galleries */ function test_post_galleries_respects_id_attrs() { $post_id = $this->factory->post->create( @@ -626,6 +692,8 @@ /** * @ticket 22960 + * + * @covers ::get_post_gallery_images */ function test_post_gallery_images() { $ids1 = array(); @@ -675,6 +743,9 @@ $this->assertEquals( $srcs, $ids1_srcs ); } + /** + * @covers ::get_media_embedded_in_content + */ function test_get_media_embedded_in_content() { $object = << @@ -739,6 +810,9 @@ $this->assertEquals( $contents, $matches ); } + /** + * @covers ::get_media_embedded_in_content + */ function test_get_media_embedded_in_content_order() { $audio = <<