Changeset 60253 for trunk/tests/phpunit/tests/post/getPostsByAuthorSql.php

Timestamp:

05/26/2025 02:34:12 PM (9 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/post/getPostsByAuthorSql.php (modified) (6 diffs)

trunk/tests/phpunit/tests/post/getPostsByAuthorSql.php

@@ -5 +5 @@
  */
 class Tests_Post_GetPostsByAuthorSql extends WP_UnitTestCase {
+
+    /**
+     * ID of the first user.
+     *
+     * @var int
+     */
+    public static $user_id_1;
+
+    /**
+     * ID of the second user.
+     *
+     * @var int
+     */
+    public static $user_id_2;
+
+    /**
+     * Set up the shared fixture.
+     *
+     * @param WP_UnitTest_Factory $factory Factory instance.
+     */
+    public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
+        self::$user_id_1 = $factory->user->create();
+        self::$user_id_2 = $factory->user->create();
+    }
 
     public function test_post_type_post() {
@@ -60 +84 @@
     public function test_public_only_true_should_not_allow_any_private_posts_for_loggedin_user() {
         $current_user = get_current_user_id();
-        $u            = self::factory()->user->create();
+        $u            = self::$user_id_1;
         wp_set_current_user( $u );
 
@@ -71 +95 @@
     public function test_public_only_should_default_to_false() {
         $current_user = get_current_user_id();
-        $u            = self::factory()->user->create();
+        $u            = self::$user_id_1;
         wp_set_current_user( $u );
 
@@ -81 +105 @@
     public function test_public_only_false_should_allow_current_user_access_to_own_private_posts_when_current_user_matches_post_author() {
         $current_user = get_current_user_id();
-        $u            = self::factory()->user->create();
+        $u            = self::$user_id_1;
         wp_set_current_user( $u );
 
@@ -92 +116 @@
     public function test_public_only_false_should_not_allow_access_to_private_posts_if_current_user_is_not_post_author() {
         $current_user = get_current_user_id();
-        $u1           = self::factory()->user->create();
-        $u2           = self::factory()->user->create();
+        $u1           = self::$user_id_1;
+        $u2           = self::$user_id_2;
         wp_set_current_user( $u1 );
 
@@ -104 +128 @@
     public function test_public_only_false_should_allow_current_user_access_to_own_private_posts_when_post_author_is_not_provided() {
         $current_user = get_current_user_id();
-        $u            = self::factory()->user->create();
+        $u            = self::$user_id_1;
         wp_set_current_user( $u );