Changeset 12605 for trunk/tests/phpunit/testcases/notifications/functions.php

Timestamp:

03/29/2020 05:32:05 PM (6 years ago)

Author:

boonebgorges

Message:

Don't delete user data on delete_user hook on Multisite.

The new function bp_remove_user_data_on_delete_user_hook(), which defaults
to false on Multisite and true otherwise, helps BuddyPress to differentiate
between the use of wp_delete_user() to delete a user account from an
installation (typical on non-Multisite) and its use to remove a user from
a site in a Multisite instance.

Props imath.

Fixes #8175.

File:

• trunk/tests/phpunit/testcases/notifications/functions.php (modified) (2 diffs)

trunk/tests/phpunit/testcases/notifications/functions.php

@@ -347 +347 @@
         $this->assertEqualSets( array( $n1, $n2, $n3, $n4, $n5 ), $found1 );
 
-        wp_delete_user( $u );
+        $this->delete_user( $u );
 
         // Check if notifications are deleted.
@@ -464 +464 @@
         $this->assertSame( 2, count( $actual['data'] ) );
     }
+
+    /**
+     * @ticket BP8175
+     */
+    public function test_notifications_data_should_be_deleted_on_user_delete_non_multisite() {
+        if ( is_multisite() ) {
+            $this->markTestSkipped( __METHOD__ . ' requires non-multisite.' );
+        }
+
+        $u = self::factory()->user->create();
+
+        $n1 = self::factory()->notification->create( array(
+            'component_name'    => 'messages',
+            'component_action'  => 'new_message',
+            'item_id'           => 99,
+            'user_id'           => $u,
+        ) );
+
+        $found = bp_notifications_get_notifications_for_user( $u, 'object' );
+        $this->assertEqualSets( [ $n1 ], wp_list_pluck( $found, 'id' ) );
+
+        wp_delete_user( $u );
+
+        $found = bp_notifications_get_notifications_for_user( $u, 'object' );
+        $this->assertEmpty( '', $found );
+    }
+
+    /**
+     * @ticket BP8175
+     */
+    public function test_notifications_data_should_be_deleted_on_user_delete_multisite() {
+        if ( ! is_multisite() ) {
+            $this->markTestSkipped( __METHOD__ . ' requires multisite.' );
+        }
+
+        $u = self::factory()->user->create();
+
+        $n1 = self::factory()->notification->create( array(
+            'component_name'    => 'messages',
+            'component_action'  => 'new_message',
+            'item_id'           => 99,
+            'user_id'           => $u,
+        ) );
+
+        $found = bp_notifications_get_notifications_for_user( $u, 'object' );
+        $this->assertEqualSets( [ $n1 ], wp_list_pluck( $found, 'id' ) );
+
+        wpmu_delete_user( $u );
+
+        $found = bp_notifications_get_notifications_for_user( $u, 'object' );
+        $this->assertEmpty( '', $found );
+    }
+
+    /**
+     * @ticket BP8175
+     */
+    public function test_notifications_data_should_not_be_deleted_on_wp_delete_user_multisite() {
+        if ( ! is_multisite() ) {
+            $this->markTestSkipped( __METHOD__ . ' requires multisite.' );
+        }
+
+        $u = self::factory()->user->create();
+
+        $n1 = self::factory()->notification->create( array(
+            'component_name'    => 'messages',
+            'component_action'  => 'new_message',
+            'item_id'           => 99,
+            'user_id'           => $u,
+        ) );
+
+        $found = bp_notifications_get_notifications_for_user( $u, 'object' );
+        $this->assertEqualSets( [ $n1 ], wp_list_pluck( $found, 'id' ) );
+
+        wp_delete_user( $u );
+
+        $found = bp_notifications_get_notifications_for_user( $u, 'object' );
+        $this->assertEqualSets( [ $n1 ], wp_list_pluck( $found, 'id' ) );
+    }
 }