From a6bb0120ac6a060950290eebe6e64d08fead6ef9 Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Sun, 23 Aug 2020 17:23:52 +0000 Subject: [PATCH] Tests: Correct a number of malformed `@covers` tags. As per the documentation of the `@covers` tag, global functions being covered should be prefixed with `::`. Also, the `()` after the function name is redundant. Follow-up to [39918]. Props jrf. See #50267. git-svn-id: https://develop.svn.wordpress.org/trunk@48848 602fd350-edb4-49c9-b593-d223f7449a82 --- .../tests/comment/isAvatarCommentType.php | 2 +- .../tests/functions/cleanupHeaderComment.php | 2 +- tests/phpunit/tests/functions/doEnclose.php | 2 +- .../tests/functions/wpArraySliceAssoc.php | 2 +- .../tests/link/getThePrivacyPolicyLink.php | 2 +- tests/phpunit/tests/menu/nav-menu.php | 18 +++++++++--------- .../privacy/wpPrivacyCompletedRequest.php | 2 +- .../wpPrivacySendPersonalDataExportEmail.php | 2 +- ...vacySendRequestConfirmationNotification.php | 2 +- .../phpunit/tests/url/getPrivacyPolicyUrl.php | 2 +- tests/phpunit/tests/user/wpSendUserRequest.php | 2 +- 11 files changed, 19 insertions(+), 19 deletions(-) diff --git a/tests/phpunit/tests/comment/isAvatarCommentType.php b/tests/phpunit/tests/comment/isAvatarCommentType.php index b01e019315..fe7e7482d3 100644 --- a/tests/phpunit/tests/comment/isAvatarCommentType.php +++ b/tests/phpunit/tests/comment/isAvatarCommentType.php @@ -11,7 +11,7 @@ * Tests_Comment_IsAvatarCommentType class. * * @group comment - * @covers is_avatar_comment_type + * @covers ::is_avatar_comment_type * * @since 5.1.0 */ diff --git a/tests/phpunit/tests/functions/cleanupHeaderComment.php b/tests/phpunit/tests/functions/cleanupHeaderComment.php index bf79b65d4c..9bc0d73cd5 100644 --- a/tests/phpunit/tests/functions/cleanupHeaderComment.php +++ b/tests/phpunit/tests/functions/cleanupHeaderComment.php @@ -10,7 +10,7 @@ class Tests_Functions_CleanupHeaderComment extends WP_UnitTestCase { /** * Test cleanup header of header comment. * - * @covers _cleanup_header_comment + * @covers ::_cleanup_header_comment * @dataProvider data_cleanup_header_comment * * @param string $test_string diff --git a/tests/phpunit/tests/functions/doEnclose.php b/tests/phpunit/tests/functions/doEnclose.php index c994ea360c..720eb8c728 100644 --- a/tests/phpunit/tests/functions/doEnclose.php +++ b/tests/phpunit/tests/functions/doEnclose.php @@ -12,7 +12,7 @@ * * @group functions.php * @group post - * @covers do_enclose + * @covers ::do_enclose * * @since 5.3.0 */ diff --git a/tests/phpunit/tests/functions/wpArraySliceAssoc.php b/tests/phpunit/tests/functions/wpArraySliceAssoc.php index bedf731771..98afc785c9 100644 --- a/tests/phpunit/tests/functions/wpArraySliceAssoc.php +++ b/tests/phpunit/tests/functions/wpArraySliceAssoc.php @@ -5,7 +5,7 @@ * * @since 5.3.0 * - * @covers wp_array_slice_assoc + * @covers ::wp_array_slice_assoc * @group functions.php */ class Tests_Functions_wpArraySliceAssoc extends WP_UnitTestCase { diff --git a/tests/phpunit/tests/link/getThePrivacyPolicyLink.php b/tests/phpunit/tests/link/getThePrivacyPolicyLink.php index 3543868831..15f6e6ccbe 100644 --- a/tests/phpunit/tests/link/getThePrivacyPolicyLink.php +++ b/tests/phpunit/tests/link/getThePrivacyPolicyLink.php @@ -12,7 +12,7 @@ * * @group link * @group privacy - * @covers get_the_privacy_policy_link + * @covers ::get_the_privacy_policy_link * * @since 4.9.6 */ diff --git a/tests/phpunit/tests/menu/nav-menu.php b/tests/phpunit/tests/menu/nav-menu.php index af67f98379..39e9ed4924 100644 --- a/tests/phpunit/tests/menu/nav-menu.php +++ b/tests/phpunit/tests/menu/nav-menu.php @@ -31,7 +31,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { /** * Two themes with one location each should just map, switching to a theme not previously-active. * - * @covers ::wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ function test_one_location_each() { $this->register_nav_menu_locations( array( 'primary' ) ); @@ -50,7 +50,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { /** * Only registered locations should be mapped and returned. * - * @covers ::wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ function test_filter_registered_locations() { $this->register_nav_menu_locations( array( 'primary', 'secondary' ) ); @@ -72,7 +72,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { /** * Locations with the same name should map, switching to a theme not previously-active. * - * @covers ::wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ function test_locations_with_same_slug() { $this->register_nav_menu_locations( array( 'primary', 'secondary' ) ); @@ -91,7 +91,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { /** * If the new theme was previously active, we should honor any changes to nav menu mapping done when the other theme was active. * - * @covers ::wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ function test_new_theme_previously_active() { $this->register_nav_menu_locations( array( 'primary' ) ); @@ -115,7 +115,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { /** * Make educated guesses on theme locations. * - * @covers ::wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ function test_location_guessing() { $this->register_nav_menu_locations( array( 'primary', 'secondary' ) ); @@ -138,7 +138,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { /** * Make sure two locations that fall in the same group don't get the same menu assigned. * - * @covers ::wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ function test_location_guessing_one_menu_per_group() { $this->register_nav_menu_locations( array( 'primary' ) ); @@ -159,7 +159,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { /** * Make sure two locations that fall in the same group get menus assigned from the same group. * - * @covers ::wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ function test_location_guessing_one_menu_per_location() { $this->register_nav_menu_locations( array( 'primary', 'main' ) ); @@ -184,7 +184,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { * * @expectedIncorrectUsage register_nav_menus * - * @covers ::wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ function test_numerical_locations() { $this->register_nav_menu_locations( array( 'primary', 1 ) ); @@ -209,7 +209,7 @@ class Tests_Nav_Menu_Theme_Change extends WP_UnitTestCase { * * @expectedIncorrectUsage register_nav_menus * - * @covers wp_map_nav_menu_locations() + * @covers ::wp_map_nav_menu_locations */ public function test_numerical_old_locations() { $this->register_nav_menu_locations( array( 'primary', 1 ) ); diff --git a/tests/phpunit/tests/privacy/wpPrivacyCompletedRequest.php b/tests/phpunit/tests/privacy/wpPrivacyCompletedRequest.php index 71884dc7d8..b3d45a9c89 100644 --- a/tests/phpunit/tests/privacy/wpPrivacyCompletedRequest.php +++ b/tests/phpunit/tests/privacy/wpPrivacyCompletedRequest.php @@ -11,7 +11,7 @@ * Tests_WpPrivacyCompletedRequest class. * * @group privacy - * @covers _wp_privacy_completed_request + * @covers ::_wp_privacy_completed_request * * @since 4.9.6 */ diff --git a/tests/phpunit/tests/privacy/wpPrivacySendPersonalDataExportEmail.php b/tests/phpunit/tests/privacy/wpPrivacySendPersonalDataExportEmail.php index 010a01434a..54c775ae49 100644 --- a/tests/phpunit/tests/privacy/wpPrivacySendPersonalDataExportEmail.php +++ b/tests/phpunit/tests/privacy/wpPrivacySendPersonalDataExportEmail.php @@ -11,7 +11,7 @@ * Tests_Privacy_WpPrivacySendPersonalDataExportEmail class. * * @group privacy - * @covers wp_privacy_send_personal_data_export_email + * @covers ::wp_privacy_send_personal_data_export_email * * @since 4.9.6 */ diff --git a/tests/phpunit/tests/privacy/wpPrivacySendRequestConfirmationNotification.php b/tests/phpunit/tests/privacy/wpPrivacySendRequestConfirmationNotification.php index ce76fa2f43..5cce5cc02d 100644 --- a/tests/phpunit/tests/privacy/wpPrivacySendRequestConfirmationNotification.php +++ b/tests/phpunit/tests/privacy/wpPrivacySendRequestConfirmationNotification.php @@ -12,7 +12,7 @@ * * @group privacy * @group user - * @covers _wp_privacy_send_request_confirmation_notification() + * @covers ::_wp_privacy_send_request_confirmation_notification */ class Tests_User_WpPrivacySendRequestConfirmationNotification extends WP_UnitTestCase { /** diff --git a/tests/phpunit/tests/url/getPrivacyPolicyUrl.php b/tests/phpunit/tests/url/getPrivacyPolicyUrl.php index 69384702da..67d10a6a3d 100644 --- a/tests/phpunit/tests/url/getPrivacyPolicyUrl.php +++ b/tests/phpunit/tests/url/getPrivacyPolicyUrl.php @@ -12,7 +12,7 @@ * * @group url * @group privacy - * @covers get_privacy_policy_url + * @covers ::get_privacy_policy_url * * @since 4.9.6 */ diff --git a/tests/phpunit/tests/user/wpSendUserRequest.php b/tests/phpunit/tests/user/wpSendUserRequest.php index 68795cd93a..bbc02ea98d 100644 --- a/tests/phpunit/tests/user/wpSendUserRequest.php +++ b/tests/phpunit/tests/user/wpSendUserRequest.php @@ -13,7 +13,7 @@ * * @group privacy * @group user - * @covers wp_send_user_request + * @covers ::wp_send_user_request */ class Tests_User_WpSendUserRequest extends WP_UnitTestCase {