Changeset 13990 for trunk/tests/phpunit/testcases/core/invitations.php

Timestamp:

07/27/2024 08:14:28 PM (9 months ago)

Author:

espellcaste

Message:

Core: Add cache_results flag to the BP_Invitation::get getter.

When performing a request with cache_results, it stops the invite information retrieved from being added to the cache.

See #8552
Closes ​https://github.com/buddypress/buddypress/pull/346/

File:

• trunk/tests/phpunit/testcases/core/invitations.php (modified) (1 diff)

trunk/tests/phpunit/testcases/core/invitations.php

@@ -8 +8 @@
  */
  class BP_Tests_Invitations extends BP_UnitTestCase {
+
+     /**
+      * @ticket BP8552
+      * @group cache
+      */
+     public function test_invitation_query_with_ids_cache_results() {
+         global $wpdb;
+
+         $u1 = self::factory()->user->create();
+         $u2 = self::factory()->user->create();
+         $u3 = self::factory()->user->create();
+
+         $invites_class = new BPTest_Invitation_Manager_Extension();
+
+         // Create a couple of invitations.
+         $invite_args = array(
+             'user_id'           => $u3,
+             'inviter_id'       => $u1,
+             'item_id'           => 1,
+             'send_invite'       => 'sent',
+         );
+
+         $invites_class->add_invitation( $invite_args );
+
+         $invite_args['inviter_id'] = $u2;
+
+         $invites_class->add_invitation( $invite_args );
+
+         $wpdb->num_queries = 0;
+
+         $first_query = BP_Invitation::get(
+             array(
+                 'cache_results' => true,
+                 'fields'        => 'ids',
+             )
+         );
+
+         $queries_before = get_num_queries();
+
+         $second_query = BP_Invitation::get(
+             array(
+                 'cache_results' => false,
+                 'fields'        => 'ids',
+             )
+         );
+
+         $queries_after = get_num_queries();
+
+         $this->assertNotSame( $queries_before, $queries_after, 'Assert that queries are run' );
+         $this->assertSame( 2, $queries_after, 'Assert that the uncached query was run' );
+         $this->assertSameSets( $first_query, $second_query, 'Results of the query are expected to match.' );
+     }
+
+     /**
+      * @ticket BP8552
+      * @group cache
+      */
+     public function test_invitation_query_with_all_cache_results() {
+         global $wpdb;
+
+         $u1 = self::factory()->user->create();
+         $u2 = self::factory()->user->create();
+         $u3 = self::factory()->user->create();
+
+         $invites_class = new BPTest_Invitation_Manager_Extension();
+
+         // Create a couple of invitations.
+         $invite_args = array(
+             'user_id'     => $u3,
+             'inviter_id'  => $u1,
+             'item_id'     => 1,
+             'send_invite' => 'sent',
+         );
+
+         $invites_class->add_invitation( $invite_args );
+
+         $invite_args['inviter_id'] = $u2;
+
+         $invites_class->add_invitation( $invite_args );
+
+         $wpdb->num_queries = 0;
+
+         $first_query = BP_Invitation::get(
+             array( 'cache_results' => true )
+         );
+
+         $queries_before = get_num_queries();
+
+         $second_query = BP_Invitation::get(
+             array( 'cache_results' => false )
+         );
+
+         $queries_after = get_num_queries();
+
+         $this->assertNotSame( $queries_before, $queries_after, 'Assert that queries are run' );
+         $this->assertSame( 3, $queries_after, 'Assert that the uncached query was run' );
+         $this->assertEquals( $first_query, $second_query, 'Results of the query are expected to match.' );
+     }
+
     public function test_bp_invitations_add_invitation_vanilla() {
         $old_current_user = get_current_user_id();