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

Timestamp:

02/25/2021 11:10:46 AM (5 years ago)

Author:

SergeyBiryukov

Message:

Tests: Reset current screen after setting it to dashboard in add_submenu_page() tests.

This avoids polluting other tests and allows the ms-excluded test group to successfully run on its own.

Props johnbillion, SergeyBiryukov.
See #52607.

File:

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

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

@@ -90 +90 @@
                 // Clean up the temporary user.
                 wp_delete_user( $admin_user );
+                // Reset current screen.
+                set_current_screen( 'front' );
 
                 // Verify the menu was inserted at the expected position.
@@ -198 +200 @@
                 // Clean up the temporary user.
                 wp_delete_user( $admin_user );
+                // Reset current screen.
+                set_current_screen( 'front' );
 
                 foreach ( $actual_positions as $test => $actual_position ) {
@@ -281 +285 @@
                 wp_set_current_user( $current_user );
                 wp_delete_user( $admin_user );
+                // Reset current screen.
+                set_current_screen( 'front' );
 
                 // Verify the menu was inserted at the expected position.
@@ -312 +318 @@
                 wp_set_current_user( $current_user );
                 wp_delete_user( $admin_user );
+                // Reset current screen.
+                set_current_screen( 'front' );
 
                 // Verify the menu was inserted at the expected position.