Move wp_delete_user()
tests to their own file.
See #33800. git-svn-id: https://develop.svn.wordpress.org/trunk@34033 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
parent
32b8ff1b14
commit
40a0d1f3cb
@ -334,62 +334,6 @@ class Tests_User extends WP_UnitTestCase {
|
|||||||
$this->assertEquals( $value, $user->get( $key ), $key );
|
$this->assertEquals( $value, $user->get( $key ), $key );
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Test that usermeta cache is cleared after user deletion.
|
|
||||||
*
|
|
||||||
* @ticket 19500
|
|
||||||
*/
|
|
||||||
function test_get_blogs_of_user() {
|
|
||||||
// Logged out users don't have blogs.
|
|
||||||
$this->assertEquals( array(), get_blogs_of_user( 0 ) );
|
|
||||||
|
|
||||||
$user_id = $this->factory->user->create( array( 'role' => 'subscriber' ) );
|
|
||||||
$blogs = get_blogs_of_user( $user_id );
|
|
||||||
$this->assertEquals( array( 1 ), array_keys( $blogs ) );
|
|
||||||
|
|
||||||
// Non-existent users don't have blogs.
|
|
||||||
if ( is_multisite() )
|
|
||||||
wpmu_delete_user( $user_id );
|
|
||||||
else
|
|
||||||
wp_delete_user( $user_id );
|
|
||||||
|
|
||||||
$user = new WP_User( $user_id );
|
|
||||||
$this->assertFalse( $user->exists(), 'WP_User->exists' );
|
|
||||||
$this->assertEquals( array(), get_blogs_of_user( $user_id ) );
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Test that usermeta cache is cleared after user deletion.
|
|
||||||
*
|
|
||||||
* @ticket 19500
|
|
||||||
*/
|
|
||||||
function test_is_user_member_of_blog() {
|
|
||||||
$old_current = get_current_user_id();
|
|
||||||
|
|
||||||
$this->assertSame( 0, $old_current );
|
|
||||||
|
|
||||||
// test for "get current user" when not logged in
|
|
||||||
$this->assertFalse( is_user_member_of_blog() );
|
|
||||||
|
|
||||||
$user_id = $this->factory->user->create( array( 'role' => 'subscriber' ) );
|
|
||||||
wp_set_current_user( $user_id );
|
|
||||||
|
|
||||||
$this->assertTrue( is_user_member_of_blog() );
|
|
||||||
$this->assertTrue( is_user_member_of_blog( 0, 0 ) );
|
|
||||||
$this->assertTrue( is_user_member_of_blog( 0, get_current_blog_id() ) );
|
|
||||||
$this->assertTrue( is_user_member_of_blog( $user_id ) );
|
|
||||||
$this->assertTrue( is_user_member_of_blog( $user_id, get_current_blog_id() ) );
|
|
||||||
|
|
||||||
// Will only remove the user from the current site in multisite; this is desired
|
|
||||||
// and will achieve the desired effect with is_user_member_of_blog().
|
|
||||||
wp_delete_user( $user_id );
|
|
||||||
|
|
||||||
$this->assertFalse( is_user_member_of_blog( $user_id ) );
|
|
||||||
$this->assertFalse( is_user_member_of_blog( $user_id, get_current_blog_id() ) );
|
|
||||||
|
|
||||||
wp_set_current_user( $old_current );
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* ticket 19595
|
* ticket 19595
|
||||||
*/
|
*/
|
||||||
@ -461,60 +405,6 @@ class Tests_User extends WP_UnitTestCase {
|
|||||||
setup_postdata( get_post( $old_post_id ) );
|
setup_postdata( get_post( $old_post_id ) );
|
||||||
}
|
}
|
||||||
|
|
||||||
function test_delete_user() {
|
|
||||||
$user_id = $this->factory->user->create( array( 'role' => 'author' ) );
|
|
||||||
$user = new WP_User( $user_id );
|
|
||||||
|
|
||||||
$post = array(
|
|
||||||
'post_author' => $user_id,
|
|
||||||
'post_status' => 'publish',
|
|
||||||
'post_content' => rand_str(),
|
|
||||||
'post_title' => rand_str(),
|
|
||||||
'post_type' => 'post',
|
|
||||||
);
|
|
||||||
|
|
||||||
// insert a post and make sure the ID is ok
|
|
||||||
$post_id = wp_insert_post($post);
|
|
||||||
$this->assertTrue(is_numeric($post_id));
|
|
||||||
$this->assertTrue($post_id > 0);
|
|
||||||
|
|
||||||
$post = get_post( $post_id );
|
|
||||||
$this->assertEquals( $post_id, $post->ID );
|
|
||||||
|
|
||||||
$post = array(
|
|
||||||
'post_author' => $user_id,
|
|
||||||
'post_status' => 'publish',
|
|
||||||
'post_content' => rand_str(),
|
|
||||||
'post_title' => rand_str(),
|
|
||||||
'post_type' => 'nav_menu_item',
|
|
||||||
);
|
|
||||||
|
|
||||||
// insert a post and make sure the ID is ok
|
|
||||||
$nav_id = wp_insert_post($post);
|
|
||||||
$this->assertTrue(is_numeric($nav_id));
|
|
||||||
$this->assertTrue($nav_id > 0);
|
|
||||||
|
|
||||||
$post = get_post( $nav_id );
|
|
||||||
$this->assertEquals( $nav_id, $post->ID );
|
|
||||||
|
|
||||||
wp_delete_user( $user_id );
|
|
||||||
$user = new WP_User( $user_id );
|
|
||||||
if ( is_multisite() )
|
|
||||||
$this->assertTrue( $user->exists() );
|
|
||||||
else
|
|
||||||
$this->assertFalse( $user->exists() );
|
|
||||||
|
|
||||||
$this->assertNotNull( get_post( $post_id ) );
|
|
||||||
$this->assertEquals( 'trash', get_post( $post_id )->post_status );
|
|
||||||
// nav_menu_item is delete_with_user = false so the nav post should remain published.
|
|
||||||
$this->assertNotNull( get_post( $nav_id ) );
|
|
||||||
$this->assertEquals( 'publish', get_post( $nav_id )->post_status );
|
|
||||||
wp_delete_post( $nav_id, true );
|
|
||||||
$this->assertNull( get_post( $nav_id ) );
|
|
||||||
wp_delete_post( $post_id, true );
|
|
||||||
$this->assertNull( get_post( $post_id ) );
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @ticket 13317
|
* @ticket 13317
|
||||||
*/
|
*/
|
||||||
@ -555,22 +445,6 @@ class Tests_User extends WP_UnitTestCase {
|
|||||||
$this->assertEquals( false, $user );
|
$this->assertEquals( false, $user );
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* @ticket 20447
|
|
||||||
*/
|
|
||||||
function test_wp_delete_user_reassignment_clears_post_caches() {
|
|
||||||
$user_id = $this->factory->user->create();
|
|
||||||
$reassign = $this->factory->user->create();
|
|
||||||
$post_id = $this->factory->post->create( array( 'post_author' => $user_id ) );
|
|
||||||
|
|
||||||
get_post( $post_id ); // Ensure this post is in the cache.
|
|
||||||
|
|
||||||
wp_delete_user( $user_id, $reassign );
|
|
||||||
|
|
||||||
$post = get_post( $post_id );
|
|
||||||
$this->assertEquals( $reassign, $post->post_author );
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @ticket 21431
|
* @ticket 21431
|
||||||
*/
|
*/
|
||||||
|
128
tests/phpunit/tests/user/wpDeleteUser.php
Normal file
128
tests/phpunit/tests/user/wpDeleteUser.php
Normal file
@ -0,0 +1,128 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @group user
|
||||||
|
*/
|
||||||
|
class Tests_User_WpDeleteUser extends WP_UnitTestCase {
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Test that usermeta cache is cleared after user deletion.
|
||||||
|
*
|
||||||
|
* @ticket 19500
|
||||||
|
*/
|
||||||
|
function test_get_blogs_of_user() {
|
||||||
|
// Logged out users don't have blogs.
|
||||||
|
$this->assertEquals( array(), get_blogs_of_user( 0 ) );
|
||||||
|
|
||||||
|
$user_id = $this->factory->user->create( array( 'role' => 'subscriber' ) );
|
||||||
|
$blogs = get_blogs_of_user( $user_id );
|
||||||
|
$this->assertEquals( array( 1 ), array_keys( $blogs ) );
|
||||||
|
|
||||||
|
// Non-existent users don't have blogs.
|
||||||
|
if ( is_multisite() )
|
||||||
|
wpmu_delete_user( $user_id );
|
||||||
|
else
|
||||||
|
wp_delete_user( $user_id );
|
||||||
|
|
||||||
|
$user = new WP_User( $user_id );
|
||||||
|
$this->assertFalse( $user->exists(), 'WP_User->exists' );
|
||||||
|
$this->assertEquals( array(), get_blogs_of_user( $user_id ) );
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Test that usermeta cache is cleared after user deletion.
|
||||||
|
*
|
||||||
|
* @ticket 19500
|
||||||
|
*/
|
||||||
|
function test_is_user_member_of_blog() {
|
||||||
|
$old_current = get_current_user_id();
|
||||||
|
|
||||||
|
$user_id = $this->factory->user->create( array( 'role' => 'subscriber' ) );
|
||||||
|
wp_set_current_user( $user_id );
|
||||||
|
|
||||||
|
$this->assertTrue( is_user_member_of_blog() );
|
||||||
|
$this->assertTrue( is_user_member_of_blog( 0, 0 ) );
|
||||||
|
$this->assertTrue( is_user_member_of_blog( 0, get_current_blog_id() ) );
|
||||||
|
$this->assertTrue( is_user_member_of_blog( $user_id ) );
|
||||||
|
$this->assertTrue( is_user_member_of_blog( $user_id, get_current_blog_id() ) );
|
||||||
|
|
||||||
|
// Will only remove the user from the current site in multisite; this is desired
|
||||||
|
// and will achieve the desired effect with is_user_member_of_blog().
|
||||||
|
wp_delete_user( $user_id );
|
||||||
|
|
||||||
|
$this->assertFalse( is_user_member_of_blog( $user_id ) );
|
||||||
|
$this->assertFalse( is_user_member_of_blog( $user_id, get_current_blog_id() ) );
|
||||||
|
|
||||||
|
wp_set_current_user( $old_current );
|
||||||
|
}
|
||||||
|
|
||||||
|
function test_delete_user() {
|
||||||
|
$user_id = $this->factory->user->create( array( 'role' => 'author' ) );
|
||||||
|
$user = new WP_User( $user_id );
|
||||||
|
|
||||||
|
$post = array(
|
||||||
|
'post_author' => $user_id,
|
||||||
|
'post_status' => 'publish',
|
||||||
|
'post_content' => rand_str(),
|
||||||
|
'post_title' => rand_str(),
|
||||||
|
'post_type' => 'post',
|
||||||
|
);
|
||||||
|
|
||||||
|
// insert a post and make sure the ID is ok
|
||||||
|
$post_id = wp_insert_post($post);
|
||||||
|
$this->assertTrue(is_numeric($post_id));
|
||||||
|
$this->assertTrue($post_id > 0);
|
||||||
|
|
||||||
|
$post = get_post( $post_id );
|
||||||
|
$this->assertEquals( $post_id, $post->ID );
|
||||||
|
|
||||||
|
$post = array(
|
||||||
|
'post_author' => $user_id,
|
||||||
|
'post_status' => 'publish',
|
||||||
|
'post_content' => rand_str(),
|
||||||
|
'post_title' => rand_str(),
|
||||||
|
'post_type' => 'nav_menu_item',
|
||||||
|
);
|
||||||
|
|
||||||
|
// insert a post and make sure the ID is ok
|
||||||
|
$nav_id = wp_insert_post($post);
|
||||||
|
$this->assertTrue(is_numeric($nav_id));
|
||||||
|
$this->assertTrue($nav_id > 0);
|
||||||
|
|
||||||
|
$post = get_post( $nav_id );
|
||||||
|
$this->assertEquals( $nav_id, $post->ID );
|
||||||
|
|
||||||
|
wp_delete_user( $user_id );
|
||||||
|
$user = new WP_User( $user_id );
|
||||||
|
if ( is_multisite() )
|
||||||
|
$this->assertTrue( $user->exists() );
|
||||||
|
else
|
||||||
|
$this->assertFalse( $user->exists() );
|
||||||
|
|
||||||
|
$this->assertNotNull( get_post( $post_id ) );
|
||||||
|
$this->assertEquals( 'trash', get_post( $post_id )->post_status );
|
||||||
|
// nav_menu_item is delete_with_user = false so the nav post should remain published.
|
||||||
|
$this->assertNotNull( get_post( $nav_id ) );
|
||||||
|
$this->assertEquals( 'publish', get_post( $nav_id )->post_status );
|
||||||
|
wp_delete_post( $nav_id, true );
|
||||||
|
$this->assertNull( get_post( $nav_id ) );
|
||||||
|
wp_delete_post( $post_id, true );
|
||||||
|
$this->assertNull( get_post( $post_id ) );
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @ticket 20447
|
||||||
|
*/
|
||||||
|
function test_wp_delete_user_reassignment_clears_post_caches() {
|
||||||
|
$user_id = $this->factory->user->create();
|
||||||
|
$reassign = $this->factory->user->create();
|
||||||
|
$post_id = $this->factory->post->create( array( 'post_author' => $user_id ) );
|
||||||
|
|
||||||
|
get_post( $post_id ); // Ensure this post is in the cache.
|
||||||
|
|
||||||
|
wp_delete_user( $user_id, $reassign );
|
||||||
|
|
||||||
|
$post = get_post( $post_id );
|
||||||
|
$this->assertEquals( $reassign, $post->post_author );
|
||||||
|
}
|
||||||
|
}
|
Loading…
Reference in New Issue
Block a user