From 40a0d1f3cbb2bcd6d42d593994068c20f41d2072 Mon Sep 17 00:00:00 2001
From: Boone Gorges <boonebgorges@git.wordpress.org>
Date: Fri, 11 Sep 2015 01:50:40 +0000
Subject: [PATCH] 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
---
 tests/phpunit/tests/user.php              | 126 ---------------------
 tests/phpunit/tests/user/wpDeleteUser.php | 128 ++++++++++++++++++++++
 2 files changed, 128 insertions(+), 126 deletions(-)
 create mode 100644 tests/phpunit/tests/user/wpDeleteUser.php

diff --git a/tests/phpunit/tests/user.php b/tests/phpunit/tests/user.php
index 206fe2b497..7d548e89f2 100644
--- a/tests/phpunit/tests/user.php
+++ b/tests/phpunit/tests/user.php
@@ -334,62 +334,6 @@ class Tests_User extends WP_UnitTestCase {
 			$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
 	 */
@@ -461,60 +405,6 @@ class Tests_User extends WP_UnitTestCase {
 			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
 	 */
@@ -555,22 +445,6 @@ class Tests_User extends WP_UnitTestCase {
 		$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
 	 */
diff --git a/tests/phpunit/tests/user/wpDeleteUser.php b/tests/phpunit/tests/user/wpDeleteUser.php
new file mode 100644
index 0000000000..d47e98c9cf
--- /dev/null
+++ b/tests/phpunit/tests/user/wpDeleteUser.php
@@ -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 );
+	}
+}