Changeset 13991 for trunk/tests/phpunit/testcases/core/optouts.php

Timestamp:

07/27/2024 08:16:17 PM (9 months ago)

Author:

espellcaste

Message:

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

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

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

File:

• trunk/tests/phpunit/testcases/core/optouts.php (modified) (9 diffs)

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

@@ -5 +5 @@
  */
  class BP_Tests_Optouts extends BP_UnitTestCase {
+
+     /**
+      * @ticket BP8552
+      */
+     public function test_bp_optouts_query_cache_results() {
+         global $wpdb;
+
+         self::factory()->optout->create_many( 2 );
+
+         // Reset.
+         $wpdb->num_queries = 0;
+
+         $first_query = BP_Optout::get(
+             array( 'cache_results' => true )
+         );
+
+         $queries_before = get_num_queries();
+
+         $second_query = BP_Optout::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_optouts_add_optout_vanilla() {
         $old_current_user = get_current_user_id();
@@ -17 +46 @@
             'email_type'        => 'annoyance'
         );
-        $i1 = bp_add_optout( $args );
+        $i1 = self::factory()->optout->create( $args );
         $args['email_address'] = 'two@wp.org';
-        $i2 = bp_add_optout( $args );
+        $i2 = self::factory()->optout->create( $args );
 
         $get_args = array(
@@ -62 +91 @@
             'email_type'        => 'annoyance'
         );
-        $i1 = bp_add_optout( $args );
+        $i1 = self::factory()->optout->create( $args );
         bp_delete_optout_by_id( $i1 );
 
@@ -70 +99 @@
         );
         $optouts = bp_get_optouts( $get_args );
-        $this->assertTrue( empty( $optouts ) );
+        $this->assertEmpty( $optouts );
 
         self::set_current_user( $old_current_user );
@@ -87 +116 @@
             'email_type'        => 'annoyance'
         );
-        $i1 = bp_add_optout( $args );
+        $i1 = self::factory()->optout->create( $args );
         $args['email_address'] = 'two@wp.org';
-        $i2 = bp_add_optout( $args );
+        self::factory()->optout->create( $args );
 
         $get_args = array(
@@ -113 +142 @@
             'email_type'        => 'annoyance'
         );
-        $i1 = bp_add_optout( $args );
+        $i1 = self::factory()->optout->create( $args );
         $args['email_address'] = 'two@WP.org';
-        $i2 = bp_add_optout( $args );
+        self::factory()->optout->create( $args );
 
         $get_args = array(
@@ -139 +168 @@
             'email_type'        => 'annoyance'
         );
-        $i1 = bp_add_optout( $args );
+        $i1 = self::factory()->optout->create( $args );
         $args['email_address'] = 'two@WP.org';
-        $i2 = bp_add_optout( $args );
+        self::factory()->optout->create( $args );
 
         $get_args = array(
@@ -166 +195 @@
             'email_type'        => 'annoyance'
         );
-        $i1 = bp_add_optout( $args );
+        $i1 = self::factory()->optout->create( $args );
         // Update it.
         $oo_class                = new BP_Optout( $i1 );
@@ -193 +222 @@
             'email_type'        => 'annoyance'
         );
-        $i1 = bp_add_optout( $args );
+        self::factory()->optout->create( $args );
         $email = new BP_Email( 'activity-at-message' );
         $email->set_from( 'test1@example.com' )->set_to( 'test2@example.com' )->set_subject( 'testing' );