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

Timestamp:

03/24/2023 05:10:44 PM (21 months ago)

Author:

SergeyBiryukov

Message:

Tests: Split the tests from post/template.php into individual test classes.

This aims to bring some consistency to the location of post template function tests, as well as to make the tests more discoverable and easier to expand.

Includes:

• Adding @covers tags.
• Renaming get_post_parent() and has_post_parent() tests to match the names of the functions.

Follow-up to [28398], [31522], [34654], [34950], [50127], [50396], [54717], [54726], [55590].

See #57841.

File:

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

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

@@ -14 +14 @@
 
         $this->menu_id = wp_create_nav_menu( 'foo' );
+    }
+
+    /**
+     * @ticket 11095
+     * @ticket 33974
+     */
+    public function test_wp_page_menu_wp_nav_menu_fallback() {
+        $pages = self::factory()->post->create_many( 3, array( 'post_type' => 'page' ) );
+
+        // No menus + wp_nav_menu() falls back to wp_page_menu().
+        $menu = wp_nav_menu( array( 'echo' => false ) );
+
+        // After falling back, the 'before' argument should be set and output as '<ul>'.
+        $this->assertMatchesRegularExpression( '/<div class="menu"><ul>/', $menu );
+
+        // After falling back, the 'after' argument should be set and output as '</ul>'.
+        $this->assertMatchesRegularExpression( '/<\/ul><\/div>/', $menu );
+
+        // After falling back, the markup should include whitespace around <li>'s.
+        $this->assertMatchesRegularExpression( '/\s<li.*>|<\/li>\s/U', $menu );
+        $this->assertDoesNotMatchRegularExpression( '/><li.*>|<\/li></U', $menu );
+
+        // No menus + wp_nav_menu() falls back to wp_page_menu(), this time without a container.
+        $menu = wp_nav_menu(
+            array(
+                'echo'      => false,
+                'container' => false,
+            )
+        );
+
+        // After falling back, the empty 'container' argument should still return a container element.
+        $this->assertMatchesRegularExpression( '/<div class="menu">/', $menu );
+
+        // No menus + wp_nav_menu() falls back to wp_page_menu(), this time without white-space.
+        $menu = wp_nav_menu(
+            array(
+                'echo'         => false,
+                'item_spacing' => 'discard',
+            )
+        );
+
+        // After falling back, the markup should not include whitespace around <li>'s.
+        $this->assertDoesNotMatchRegularExpression( '/\s<li.*>|<\/li>\s/U', $menu );
+        $this->assertMatchesRegularExpression( '/><li.*>|<\/li></U', $menu );
+
     }
 
@@ -718 +763 @@
     }
 
-
     /**
      * @ticket 35272