Changeset 60253 for trunk/tests/phpunit/tests/l10n/getUserLocale.php

Timestamp:

05/26/2025 02:34:12 PM (8 months ago)

Author:

johnbillion

Message:

Build/Test Tools: Increase shared user fixtures following switch to bcrypt hashing.

User creation is somewhat more expensive since switching the default password hashing algorithm to bcrypt in 6.8. This speeds up the tests by making more use of shared user fixtures, thus reducing the number of users that are created during tests.

Props peterwilsoncc.

See #63026

File:

• trunk/tests/phpunit/tests/l10n/getUserLocale.php (modified) (6 diffs)

trunk/tests/phpunit/tests/l10n/getUserLocale.php

@@ -10 +10 @@
     protected $user_id;
 
-    public function set_up() {
-        parent::set_up();
+    /**
+     * ID of the administrator user with de_DE local.
+     *
+     * @var int
+     */
+    public static $administrator_de_de;
 
-        $this->user_id = self::factory()->user->create(
+    /**
+     * ID of the user with es_ES local.
+     *
+     * @var int
+     */
+    public static $user_es_es;
+
+    /**
+     * Set up the shared fixtures.
+     *
+     * @param WP_UnitTest_Factory $factory Factory instance.
+     */
+    public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
+        self::$administrator_de_de = $factory->user->create(
             array(
                 'role'   => 'administrator',
@@ -20 +37 @@
         );
 
-        wp_set_current_user( $this->user_id );
+        self::$user_es_es = self::factory()->user->create(
+            array(
+                'locale' => 'es_ES',
+            )
+        );
+    }
+
+    public function set_up() {
+        parent::set_up();
+
+        wp_set_current_user( self::$administrator_de_de );
     }
 
@@ -26 +53 @@
         set_current_screen( 'dashboard' );
         $this->assertSame( 'de_DE', get_user_locale() );
-        $this->assertSame( get_user_by( 'id', $this->user_id )->locale, get_user_locale() );
+        $this->assertSame( get_user_by( 'id', self::$administrator_de_de )->locale, get_user_locale() );
     }
 
     public function test_update_user_locale() {
         set_current_screen( 'dashboard' );
-        update_user_meta( $this->user_id, 'locale', 'fr_FR' );
+        update_user_meta( self::$administrator_de_de, 'locale', 'fr_FR' );
         $this->assertSame( 'fr_FR', get_user_locale() );
     }
@@ -37 +64 @@
     public function test_returns_site_locale_if_empty() {
         set_current_screen( 'dashboard' );
-        update_user_meta( $this->user_id, 'locale', '' );
+        update_user_meta( self::$administrator_de_de, 'locale', '' );
         $this->assertSame( get_locale(), get_user_locale() );
     }
@@ -79 +106 @@
 
     public function test_user_id_argument_with_id() {
-        $user_id = self::factory()->user->create(
-            array(
-                'locale' => 'es_ES',
-            )
-        );
+        $user_id = self::$user_es_es;
 
         $user_locale1 = get_user_locale( $user_id );
@@ -96 +119 @@
 
     public function test_user_id_argument_with_wp_user_object() {
-        $user_id = self::factory()->user->create(
-            array(
-                'locale' => 'es_ES',
-            )
-        );
+        $user_id = self::$user_es_es;
 
         $user = get_user_by( 'id', $user_id );