Changeset 47122 for trunk/tests/phpunit/tests/post/nav-menu.php

Timestamp:

01/29/2020 12:43:23 AM (7 years ago)

Author:

SergeyBiryukov

Message:

Docs: Improve inline comments per the documentation standards.

Includes minor code layout fixes for better readability.

See #48303.

File:

• trunk/tests/phpunit/tests/post/nav-menu.php (modified) (9 diffs)

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

@@ -84 +84 @@
                 );
 
-                // Item without menu-item-object arg
+                // Item without menu-item-object arg.
                 $post_2_insert = wp_update_nav_menu_item(
                         $this->menu_id,
@@ -143 +143 @@
         function test_orphan_nav_menu_item() {
 
-                // Create an orphan nav menu item
+                // Create an orphan nav menu item.
                 $custom_item_id = wp_update_nav_menu_item(
                         0,
@@ -155 +155 @@
                 );
 
-                // Confirm it saved properly
+                // Confirm it saved properly.
                 $custom_item = wp_setup_nav_menu_item( get_post( $custom_item_id ) );
                 $this->assertEquals( 'Wordpress.org', $custom_item->title );
 
-                // Update the orphan with an associated nav menu
+                // Update the orphan with an associated nav menu.
                 wp_update_nav_menu_item(
                         $this->menu_id,
@@ -226 +226 @@
          */
         function test_orderby_name_by_default() {
-                // We are going to create a random number of menus (min 2, max 10)
+                // We are going to create a random number of menus (min 2, max 10).
                 $menus_no = rand( 2, 10 );
 
@@ -233 +233 @@
                 }
 
-                // This is the expected array of menu names
+                // This is the expected array of menu names.
                 $expected_nav_menus_names = wp_list_pluck(
                         get_terms(
@@ -245 +245 @@
                 );
 
-                // And this is what we got when calling wp_get_nav_menus()
+                // And this is what we got when calling wp_get_nav_menus().
                 $nav_menus_names = wp_list_pluck( wp_get_nav_menus(), 'name' );
 
@@ -312 +312 @@
 
                 $this->assertEquals( $post_type_slug, $post_type_archive_item->title );
-                $this->assertEquals( $post_type_description, $post_type_archive_item->description ); //fail!!!
+                $this->assertEquals( $post_type_description, $post_type_archive_item->description ); // Fail!
         }
 
@@ -460 +460 @@
                 );
 
-                // The markup should include whitespace between <li>s
+                // The markup should include whitespace between <li>'s.
                 $this->assertRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
                 $this->assertNotRegExp( '/<\/li><li.*>/U', $menu );
 
-                // Whitepsace suppressed.
+                // Whitespace suppressed.
                 $menu = wp_nav_menu(
                         array(
@@ -473 +473 @@
                 );
 
-                // The markup should not include whitespace around <li>s
+                // The markup should not include whitespace around <li>'s.
                 $this->assertNotRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
                 $this->assertRegExp( '/><li.*>|<\/li></U', $menu );