Changeset 10815 for trunk/tests/phpunit/testcases/blogs/functions.php

Timestamp:

05/27/2016 05:21:20 AM (9 years ago)

Author:

r-a-y

Message:

Blogs: Improve how bp_blogs_record_existing_blogs() operates.

When the "Repopulate blogs records" is used from the "Tools > BuddyPress"
page, on large site installs, this option isn't very effective.

This commit improves bp_blogs_record_existing_blogs() by:

• Recording an offset so the blog record repopulation can be resumed on failure.
• Only recording the root blog for the current user if run from single site. (See #6940.)
• Fixing issues with blog count caches.
• Giving each blog a better last activity timestamp by using the last_updated column from the wp_blogs DB table.
• Supporting passing selected blog IDs in case a dev wants to re-record specific blogs.

Fixes #6370.

File:

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

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

@@ -985 +985 @@
     }
 
+    /**
+     * @group bp_blogs_record_existing_blogs
+     */
+    public function test_bp_blogs_record_existing_blogs_limit() {
+        if ( ! is_multisite() ) {
+            return;
+        }
+
+        $old_user = get_current_user_id();
+
+        $u = $this->factory->user->create();
+        $this->set_current_user( $u );
+
+        // Create three sites.
+        $this->factory->blog->create_many( 3, array(
+            'user_id' => $u
+        ) );
+
+        // Record each site one at a time
+        bp_blogs_record_existing_blogs( array(
+            'limit' => 1
+        ) );
+
+        // Assert!
+        $blogs = bp_blogs_get_blogs( array(
+            'user_id' => $u
+        ) );
+        $this->assertSame( 3, (int) $blogs['total'] );
+
+        $this->set_current_user( $old_user );
+    }
+
     protected function activity_exists_for_post( $post_id ) {
         $a = bp_activity_get( array(