Changeset 58876 for trunk/tests/phpunit/tests/admin/includesPlugin.php

Timestamp:

08/09/2024 06:29:47 PM (19 months ago)

Author:

flixos90

Message:

Build/Test Tools: Avoid using wp_delete_user() in PHPUnit tests unless explicitly acknowledging or ignoring Multisite.

wp_delete_user() does not actually delete the entire user when using WordPress Multisite. Therefore tests should typically use the test helper method to fully delete the user, unless explicitly ignoring Multisite or testing the wp_delete_user() function while taking Multisite behavior into account.

Fixes #61851.

File:

• trunk/tests/phpunit/tests/admin/includesPlugin.php (modified) (5 diffs)

trunk/tests/phpunit/tests/admin/includesPlugin.php

@@ -97 +97 @@
 
         // Clean up the temporary user.
-        wp_delete_user( $admin_user );
+        self::delete_user( $admin_user );
 
         // Verify the menu was inserted at the expected position.
@@ -205 +205 @@
 
         // Clean up the temporary user.
-        wp_delete_user( $admin_user );
+        self::delete_user( $admin_user );
 
         foreach ( $actual_positions as $test => $actual_position ) {
@@ -296 +296 @@
         // Clean up the temporary user.
         wp_set_current_user( $current_user );
-        wp_delete_user( $admin_user );
+        self::delete_user( $admin_user );
 
         // Verify the menu was inserted at the expected position.
@@ -327 +327 @@
         // Clean up the temporary user.
         wp_set_current_user( $current_user );
-        wp_delete_user( $admin_user );
+        self::delete_user( $admin_user );
 
         // Verify the menu was inserted at the expected position.
@@ -356 +356 @@
         // Clean up the temporary user.
         wp_set_current_user( $current_user );
-        wp_delete_user( $admin_user );
+        self::delete_user( $admin_user );
 
         // Verify the menus were inserted.