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

Timestamp:

07/27/2024 08:16:17 PM (2 years 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' );