Changeset 9231 for trunk/tests/phpunit/testcases/blogs/cache.php

Timestamp:

12/15/2014 08:38:11 PM (11 years ago)

Author:

r-a-y

Message:

Check cache for count functions that return zero.

Some of our count functions were previously setting counts of zero
correctly, but the functions themselves were not referencing the cache
properly when the count returned zero. This led to unnecessary database
queries and we hate extra queries!

This commit addresses the problem and adds unit tests.

Props r-a-y, boonebgorges.

Fixes #6012.

File:

• trunk/tests/phpunit/testcases/blogs/cache.php (modified) (1 diff)

trunk/tests/phpunit/testcases/blogs/cache.php

@@ -232 +232 @@
         $this->set_current_user( $old_user );
     }
+
+    /**
+     * @group bp_blogs_total_blogs
+     * @group counts
+     */
+    public function test_bp_blogs_total_count_should_respect_cached_value_of_0() {
+        if ( ! is_multisite() ) {
+            return;
+        }
+
+        global $wpdb;
+
+        // prime cache
+        // no blogs are created by default, so count is zero
+        bp_blogs_total_blogs();
+        $first_query_count = $wpdb->num_queries;
+
+        // run function again
+        bp_blogs_total_blogs();
+
+        // check if function references cache or hits the DB by comparing query count
+        $this->assertEquals( $first_query_count, $wpdb->num_queries );
+    }
 }