Changeset 48848

Timestamp:

08/23/2020 05:23:52 PM (6 years ago)

Author:

SergeyBiryukov

Message:

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.

Location:

trunk/tests/phpunit/tests

Files:

• comment/isAvatarCommentType.php (modified) (1 diff)
• functions/cleanupHeaderComment.php (modified) (1 diff)
• functions/doEnclose.php (modified) (1 diff)
• functions/wpArraySliceAssoc.php (modified) (1 diff)
• link/getThePrivacyPolicyLink.php (modified) (1 diff)
• menu/nav-menu.php (modified) (9 diffs)
• privacy/wpPrivacyCompletedRequest.php (modified) (1 diff)
• privacy/wpPrivacySendPersonalDataExportEmail.php (modified) (1 diff)
• privacy/wpPrivacySendRequestConfirmationNotification.php (modified) (1 diff)
• url/getPrivacyPolicyUrl.php (modified) (1 diff)
• user/wpSendUserRequest.php (modified) (1 diff)

trunk/tests/phpunit/tests/comment/isAvatarCommentType.php

@@ -12 +12 @@
  *
  * @group comment
- * @covers is_avatar_comment_type
+ * @covers ::is_avatar_comment_type
  *
  * @since 5.1.0

trunk/tests/phpunit/tests/functions/cleanupHeaderComment.php

@@ -11 +11 @@
          * Test cleanup header of header comment.
          *
-         * @covers _cleanup_header_comment
+         * @covers ::_cleanup_header_comment
          * @dataProvider data_cleanup_header_comment
          *

trunk/tests/phpunit/tests/functions/doEnclose.php

@@ -13 +13 @@
  * @group functions.php
  * @group post
- * @covers do_enclose
+ * @covers ::do_enclose
  *
  * @since 5.3.0

trunk/tests/phpunit/tests/functions/wpArraySliceAssoc.php

@@ -6 +6 @@
  * @since 5.3.0
  *
- * @covers wp_array_slice_assoc
+ * @covers ::wp_array_slice_assoc
  * @group functions.php
  */

trunk/tests/phpunit/tests/link/getThePrivacyPolicyLink.php

@@ -13 +13 @@
  * @group link
  * @group privacy
- * @covers get_the_privacy_policy_link
+ * @covers ::get_the_privacy_policy_link
  *
  * @since 4.9.6

trunk/tests/phpunit/tests/menu/nav-menu.php

@@ -32 +32 @@
          * 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() {
@@ -51 +51 @@
          * 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() {
@@ -73 +73 @@
          * 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() {
@@ -92 +92 @@
          * 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() {
@@ -116 +116 @@
          * Make educated guesses on theme locations.
          *
-         * @covers ::wp_map_nav_menu_locations()
+         * @covers ::wp_map_nav_menu_locations
          */
         function test_location_guessing() {
@@ -139 +139 @@
          * 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() {
@@ -160 +160 @@
          * 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() {
@@ -185 +185 @@
          * @expectedIncorrectUsage register_nav_menus
          *
-         * @covers ::wp_map_nav_menu_locations()
+         * @covers ::wp_map_nav_menu_locations
          */
         function test_numerical_locations() {
@@ -210 +210 @@
          * @expectedIncorrectUsage register_nav_menus
          *
-         * @covers wp_map_nav_menu_locations()
+         * @covers ::wp_map_nav_menu_locations
          */
         public function test_numerical_old_locations() {

trunk/tests/phpunit/tests/privacy/wpPrivacyCompletedRequest.php

@@ -12 +12 @@
  *
  * @group privacy
- * @covers _wp_privacy_completed_request
+ * @covers ::_wp_privacy_completed_request
  *
  * @since 4.9.6

trunk/tests/phpunit/tests/privacy/wpPrivacySendPersonalDataExportEmail.php

@@ -12 +12 @@
  *
  * @group privacy
- * @covers wp_privacy_send_personal_data_export_email
+ * @covers ::wp_privacy_send_personal_data_export_email
  *
  * @since 4.9.6

trunk/tests/phpunit/tests/privacy/wpPrivacySendRequestConfirmationNotification.php

@@ -13 +13 @@
  * @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 {

trunk/tests/phpunit/tests/url/getPrivacyPolicyUrl.php

@@ -13 +13 @@
  * @group url
  * @group privacy
- * @covers get_privacy_policy_url
+ * @covers ::get_privacy_policy_url
  *
  * @since 4.9.6

trunk/tests/phpunit/tests/user/wpSendUserRequest.php

@@ -14 +14 @@
  * @group privacy
  * @group user
- * @covers wp_send_user_request
+ * @covers ::wp_send_user_request
  */
 class Tests_User_WpSendUserRequest extends WP_UnitTestCase {