vhad/wordpress-develop
1
0
Fork 0
mirror of https://github.com/gosticks/wordpress-develop.git synced 2026-08-11 12:20:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

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

Browse Source 
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.

git-svn-id: https://develop.svn.wordpress.org/trunk@55591 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Sergey Biryukov
2023-03-24 17:10:44 +00:00
parent 13fb9e268c
commit 561287c861
5 changed files with 328 additions and 305 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/phpunit/tests/post/getPageTemplateSlug.php
+75
View File
@@ -0,0 +1,75 @@
<?php
/**
* @group post
* @group template
*
* @covers ::get_page_template_slug
*/
class Tests_Post_GetPageTemplateSlug extends WP_UnitTestCase {
/**
* @ticket 31389
*/
public function test_get_page_template_slug_by_id() {
$page_id = self::factory()->post->create(
array(
'post_type' => 'page',
)
);
$this->assertSame( '', get_page_template_slug( $page_id ) );
update_post_meta( $page_id, '_wp_page_template', 'default' );
$this->assertSame( '', get_page_template_slug( $page_id ) );
update_post_meta( $page_id, '_wp_page_template', 'example.php' );
$this->assertSame( 'example.php', get_page_template_slug( $page_id ) );
}
/**
* @ticket 31389
*/
public function test_get_page_template_slug_from_loop() {
$page_id = self::factory()->post->create(
array(
'post_type' => 'page',
)
);
update_post_meta( $page_id, '_wp_page_template', 'example.php' );
$this->go_to( get_permalink( $page_id ) );
$this->assertSame( 'example.php', get_page_template_slug() );
}
/**
* @ticket 31389
* @ticket 18375
*/
public function test_get_page_template_slug_non_page() {
$post_id = self::factory()->post->create();
$this->assertSame( '', get_page_template_slug( $post_id ) );
update_post_meta( $post_id, '_wp_page_template', 'default' );
$this->assertSame( '', get_page_template_slug( $post_id ) );
update_post_meta( $post_id, '_wp_page_template', 'example.php' );
$this->assertSame( 'example.php', get_page_template_slug( $post_id ) );
}
/**
* @ticket 18375
*/
public function test_get_page_template_slug_non_page_from_loop() {
$post_id = self::factory()->post->create();
update_post_meta( $post_id, '_wp_page_template', 'example.php' );
$this->go_to( get_permalink( $post_id ) );
$this->assertSame( 'example.php', get_page_template_slug() );
}
}
tests/phpunit/tests/post/getPostParent.php
+72
View File
@@ -0,0 +1,72 @@
<?php
/**
* @group post
* @group template
*
* @covers ::get_post_parent
* @covers ::has_post_parent
*/
class Tests_Post_GetPostParent extends WP_UnitTestCase {
/**
* @ticket 33045
*/
public function test_get_post_parent() {
$post = array(
'post_status' => 'publish',
'post_type' => 'page',
);
// Insert two initial posts.
$parent_id = self::factory()->post->create( $post );
$child_id = self::factory()->post->create( $post );
// Test if the function returns null by default.
$parent = get_post_parent( $child_id );
$this->assertNull( $parent );
// Update child post with a parent.
wp_update_post(
array(
'ID' => $child_id,
'post_parent' => $parent_id,
)
);
// Test if the function returns the parent object.
$parent = get_post_parent( $child_id );
$this->assertNotNull( $parent );
$this->assertSame( $parent_id, $parent->ID );
}
/**
* @ticket 33045
*/
public function test_has_post_parent() {
$post = array(
'post_status' => 'publish',
'post_type' => 'page',
);
// Insert two initial posts.
$parent_id = self::factory()->post->create( $post );
$child_id = self::factory()->post->create( $post );
// Test if the function returns false by default.
$parent = has_post_parent( $child_id );
$this->assertFalse( $parent );
// Update child post with a parent.
wp_update_post(
array(
'ID' => $child_id,
'post_parent' => $parent_id,
)
);
// Test if the function returns true for a child post.
$parent = has_post_parent( $child_id );
$this->assertTrue( $parent );
}
}
tests/phpunit/tests/post/nav-menu.php
+45 -1
View File
@@ -15,6 +15,51 @@ class Tests_Post_Nav_Menu extends WP_UnitTestCase {
$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 );
}
/**
* @ticket 32464
*/
@@ -717,7 +762,6 @@ class Tests_Post_Nav_Menu extends WP_UnitTestCase {
return $ignored_1;
}
/**
* @ticket 35272
*/
tests/phpunit/tests/post/template.php
-304
View File
@@ -1,304 +0,0 @@
<?php
/**
* @group template
*/
class Tests_Post_Template extends WP_UnitTestCase {
public function test_wp_link_pages() {
$contents = array( 'One', 'Two', 'Three' );
$content = implode( '<!--nextpage-->', $contents );
$post_id = self::factory()->post->create( array( 'post_content' => $content ) );
$this->go_to( '?p=' . $post_id );
setup_postdata( get_post( $post_id ) );
$permalink = sprintf( '<a href="%s" class="post-page-numbers">', get_permalink() );
$page2 = _wp_link_page( 2 );
$page3 = _wp_link_page( 3 );
$expected = '<p class="post-nav-links">Pages: <span class="post-page-numbers current" aria-current="page">1</span> ' . $page2 . '2</a> ' . $page3 . '3</a></p>';
$output = wp_link_pages( array( 'echo' => 0 ) );
$this->assertSame( $expected, $output );
$before_after = " <span class=\"post-page-numbers current\" aria-current=\"page\">1</span> {$page2}2</a> {$page3}3</a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
)
);
$this->assertSame( $before_after, $output );
$separator = " <span class=\"post-page-numbers current\" aria-current=\"page\">1</span>{$page2}2</a>{$page3}3</a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
)
);
$this->assertSame( $separator, $output );
$link = " <span class=\"post-page-numbers current\" aria-current=\"page\"><em>1</em></span>{$page2}<em>2</em></a>{$page3}<em>3</em></a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
'link_before' => '<em>',
'link_after' => '</em>',
)
);
$this->assertSame( $link, $output );
$next = "{$page2}<em>Next page</em></a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
'link_before' => '<em>',
'link_after' => '</em>',
'next_or_number' => 'next',
)
);
$this->assertSame( $next, $output );
$GLOBALS['page'] = 2;
$next_prev = "{$permalink}<em>Previous page</em></a>{$page3}<em>Next page</em></a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
'link_before' => '<em>',
'link_after' => '</em>',
'next_or_number' => 'next',
)
);
$this->assertSame( $next_prev, $output );
$next_prev_link = "{$permalink}Woo page</a>{$page3}Hoo page</a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
'next_or_number' => 'next',
'nextpagelink' => 'Hoo page',
'previouspagelink' => 'Woo page',
)
);
$this->assertSame( $next_prev_link, $output );
$GLOBALS['page'] = 1;
$separator = "<p class=\"post-nav-links\">Pages: <span class=\"post-page-numbers current\" aria-current=\"page\">1</span> | {$page2}2</a> | {$page3}3</a></p>";
$output = wp_link_pages(
array(
'echo' => 0,
'separator' => ' | ',
)
);
$this->assertSame( $separator, $output );
$pagelink = " <span class=\"post-page-numbers current\" aria-current=\"page\">Page 1</span> | {$page2}Page 2</a> | {$page3}Page 3</a>";
$output = wp_link_pages(
array(
'echo' => 0,
'separator' => ' | ',
'before' => '',
'after' => '',
'pagelink' => 'Page %',
)
);
$this->assertSame( $pagelink, $output );
}
/**
* @ticket 31389
*/
public function test_get_page_template_slug_by_id() {
$page_id = self::factory()->post->create(
array(
'post_type' => 'page',
)
);
$this->assertSame( '', get_page_template_slug( $page_id ) );
update_post_meta( $page_id, '_wp_page_template', 'default' );
$this->assertSame( '', get_page_template_slug( $page_id ) );
update_post_meta( $page_id, '_wp_page_template', 'example.php' );
$this->assertSame( 'example.php', get_page_template_slug( $page_id ) );
}
/**
* @ticket 31389
*/
public function test_get_page_template_slug_from_loop() {
$page_id = self::factory()->post->create(
array(
'post_type' => 'page',
)
);
update_post_meta( $page_id, '_wp_page_template', 'example.php' );
$this->go_to( get_permalink( $page_id ) );
$this->assertSame( 'example.php', get_page_template_slug() );
}
/**
* @ticket 31389
* @ticket 18375
*/
public function test_get_page_template_slug_non_page() {
$post_id = self::factory()->post->create();
$this->assertSame( '', get_page_template_slug( $post_id ) );
update_post_meta( $post_id, '_wp_page_template', 'default' );
$this->assertSame( '', get_page_template_slug( $post_id ) );
update_post_meta( $post_id, '_wp_page_template', 'example.php' );
$this->assertSame( 'example.php', get_page_template_slug( $post_id ) );
}
/**
* @ticket 18375
*/
public function test_get_page_template_slug_non_page_from_loop() {
$post_id = self::factory()->post->create();
update_post_meta( $post_id, '_wp_page_template', 'example.php' );
$this->go_to( get_permalink( $post_id ) );
$this->assertSame( 'example.php', get_page_template_slug() );
}
/**
* @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 );
}
/**
* @ticket 33045
*/
public function test_get_parent_post() {
$post = array(
'post_status' => 'publish',
'post_type' => 'page',
);
// Insert two initial posts.
$parent_id = self::factory()->post->create( $post );
$child_id = self::factory()->post->create( $post );
// Test if child get_parent_post() post returns Null by default.
$parent = get_post_parent( $child_id );
$this->assertNull( $parent );
// Update child post with a parent.
wp_update_post(
array(
'ID' => $child_id,
'post_parent' => $parent_id,
)
);
// Test if child get_parent_post() post returns the parent object.
$parent = get_post_parent( $child_id );
$this->assertNotNull( $parent );
$this->assertSame( $parent_id, $parent->ID );
}
/**
* @ticket 33045
*/
public function test_has_parent_post() {
$post = array(
'post_status' => 'publish',
'post_type' => 'page',
);
// Insert two initial posts.
$parent_id = self::factory()->post->create( $post );
$child_id = self::factory()->post->create( $post );
// Test if child has_parent_post() post returns False by default.
$parent = has_post_parent( $child_id );
$this->assertFalse( $parent );
// Update child post with a parent.
wp_update_post(
array(
'ID' => $child_id,
'post_parent' => $parent_id,
)
);
// Test if child has_parent_post() returns True.
$parent = has_post_parent( $child_id );
$this->assertTrue( $parent );
}
}
tests/phpunit/tests/post/wpLinkPages.php
+136
View File
@@ -0,0 +1,136 @@
<?php
/**
* @group post
* @group template
*
* @covers ::wp_link_pages
*/
class Tests_Post_wpLinkPages extends WP_UnitTestCase {
public function test_wp_link_pages() {
$contents = array( 'One', 'Two', 'Three' );
$content = implode( '<!--nextpage-->', $contents );
$post_id = self::factory()->post->create( array( 'post_content' => $content ) );
$this->go_to( '?p=' . $post_id );
setup_postdata( get_post( $post_id ) );
$permalink = sprintf( '<a href="%s" class="post-page-numbers">', get_permalink() );
$page2 = _wp_link_page( 2 );
$page3 = _wp_link_page( 3 );
$expected = '<p class="post-nav-links">Pages: <span class="post-page-numbers current" aria-current="page">1</span> ' . $page2 . '2</a> ' . $page3 . '3</a></p>';
$output = wp_link_pages( array( 'echo' => 0 ) );
$this->assertSame( $expected, $output );
$before_after = " <span class=\"post-page-numbers current\" aria-current=\"page\">1</span> {$page2}2</a> {$page3}3</a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
)
);
$this->assertSame( $before_after, $output );
$separator = " <span class=\"post-page-numbers current\" aria-current=\"page\">1</span>{$page2}2</a>{$page3}3</a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
)
);
$this->assertSame( $separator, $output );
$link = " <span class=\"post-page-numbers current\" aria-current=\"page\"><em>1</em></span>{$page2}<em>2</em></a>{$page3}<em>3</em></a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
'link_before' => '<em>',
'link_after' => '</em>',
)
);
$this->assertSame( $link, $output );
$next = "{$page2}<em>Next page</em></a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
'link_before' => '<em>',
'link_after' => '</em>',
'next_or_number' => 'next',
)
);
$this->assertSame( $next, $output );
$GLOBALS['page'] = 2;
$next_prev = "{$permalink}<em>Previous page</em></a>{$page3}<em>Next page</em></a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
'link_before' => '<em>',
'link_after' => '</em>',
'next_or_number' => 'next',
)
);
$this->assertSame( $next_prev, $output );
$next_prev_link = "{$permalink}Woo page</a>{$page3}Hoo page</a>";
$output = wp_link_pages(
array(
'echo' => 0,
'before' => '',
'after' => '',
'separator' => '',
'next_or_number' => 'next',
'nextpagelink' => 'Hoo page',
'previouspagelink' => 'Woo page',
)
);
$this->assertSame( $next_prev_link, $output );
$GLOBALS['page'] = 1;
$separator = "<p class=\"post-nav-links\">Pages: <span class=\"post-page-numbers current\" aria-current=\"page\">1</span> | {$page2}2</a> | {$page3}3</a></p>";
$output = wp_link_pages(
array(
'echo' => 0,
'separator' => ' | ',
)
);
$this->assertSame( $separator, $output );
$pagelink = " <span class=\"post-page-numbers current\" aria-current=\"page\">Page 1</span> | {$page2}Page 2</a> | {$page3}Page 3</a>";
$output = wp_link_pages(
array(
'echo' => 0,
'separator' => ' | ',
'before' => '',
'after' => '',
'pagelink' => 'Page %',
)
);
$this->assertSame( $pagelink, $output );
}
}