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

Timestamp:

12/15/2014 08:38:11 PM (12 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/groups/cache.php (modified) (1 diff)

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

@@ -201 +201 @@
                 $this->assertEquals( 2, count( groups_get_group_admins( $g ) ) );
         }
+
+        /**
+         * @group groups_get_total_group_count
+         * @group counts
+         */
+        public function test_groups_get_total_group_count_should_respect_cached_value_of_0() {
+                global $wpdb;
+
+                // prime cache
+                // no groups are created by default, so count is zero
+                groups_get_total_group_count();
+                $first_query_count = $wpdb->num_queries;
+
+                // run function again
+                groups_get_total_group_count();
+
+                // check if function references cache or hits the DB by comparing query count
+                $this->assertEquals( $first_query_count, $wpdb->num_queries );
+        }
 }