Changeset 13642 for trunk/tests/phpunit/testcases/blogs/activity.php

Timestamp:

11/10/2023 02:09:10 AM (10 months ago)

Author:

imath

Message:

Make Activity loop reliable when post & activity comments are synced

As the $action key of the bp_has_activities() function arguments array accepts a string, a boolean as well as an array, we need to make sure the bp_blogs_new_blog_comment_query_backpat() function doesn't trigger an error when an array of activity type is passed.

Props btwebdesign, magland, teeboy4real

Fixes #9010
Closes ​https://github.com/buddypress/buddypress/pull/190

File:

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

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

@@ -618 +618 @@
 
     /**
+     * @ticket BP9010
+     * @group post_type_comment_activities
+     */
+    public function test_bp_blogs_new_blog_comment_query_backpat() {
+        $old_user = get_current_user_id();
+        $u        = self::factory()->user->create();
+        $reset_at = isset( $GLOBALS['activities_template'] ) ? $GLOBALS['activities_template'] : null;
+
+        $this->set_current_user( $u );
+
+        // let's use activity comments instead of single "new_blog_comment" activity items
+        add_filter( 'bp_disable_blogforum_comments', '__return_false' );
+
+        // create the blog post
+        $post_id = self::factory()->post->create(
+            array(
+                'post_status' => 'publish',
+                'post_type'   => 'post',
+                'post_title'  => 'Test new_blog_comment_query_backpat',
+            )
+        );
+
+        // Grab the activity ID for the activity comment.
+        $a = bp_activity_get_activity_id(
+            array(
+                'type'      => 'new_blog_post',
+                'component' => buddypress()->blogs->id,
+                'filter'    => array(
+                    'item_id'           => get_current_blog_id(),
+                    'secondary_item_id' => $post_id
+                ),
+            )
+        );
+
+        $c = bp_activity_new_comment(
+            array(
+                'content'     => 'this activity should have the `new_blog_comment` type',
+                'user_id'     => $u,
+                'activity_id' => $a,
+            )
+        );
+
+        // Check activity is fetch.
+        bp_has_activities(
+            array(
+                'action' => 'new_blog_comment',
+            )
+        );
+
+        $ids = wp_list_pluck( $GLOBALS['activities_template']->activities, 'id' );
+        $this->assertEquals( array( $c ), $ids );
+
+        // Reset.
+        remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
+        $GLOBALS['activities_template'] = $reset_at;
+        $this->set_current_user( $old_user );
+    }
+
+    /**
+     * @ticket BP9010
+     * @group post_type_comment_activities
+     */
+    public function test_bp_blogs_new_blog_comment_query_backpat_with_array_of_actions() {
+        $old_user = get_current_user_id();
+        $u        = self::factory()->user->create();
+        $reset_at = isset( $GLOBALS['activities_template'] ) ? $GLOBALS['activities_template'] : null;
+
+        $this->set_current_user( $u );
+
+        // let's use activity comments instead of single "new_blog_comment" activity items
+        add_filter( 'bp_disable_blogforum_comments', '__return_false' );
+
+        // create the blog post
+        $post_id = self::factory()->post->create(
+            array(
+                'post_status' => 'publish',
+                'post_type'   => 'post',
+                'post_title'  => 'Test new_blog_comment_query_backpat with an array',
+            )
+        );
+
+        // Grab the activity ID for the activity comment.
+        $a = bp_activity_get_activity_id(
+            array(
+                'type'      => 'new_blog_post',
+                'component' => buddypress()->blogs->id,
+                'filter'    => array(
+                    'item_id'           => get_current_blog_id(),
+                    'secondary_item_id' => $post_id
+                ),
+            )
+        );
+
+        $c = bp_activity_new_comment(
+            array(
+                'content'     => 'Check no fatal error is triggered!',
+                'user_id'     => $u,
+                'activity_id' => $a,
+            )
+        );
+
+        // Check no fatal error is triggered.
+        bp_has_activities(
+            array(
+                'action' => array( 'new_blog_comment', 'new_blog_comment' ),
+            )
+        );
+
+        $this->assertEmpty( $GLOBALS['activities_template']->activities );
+
+        // Reset.
+        remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
+        $GLOBALS['activities_template'] = $reset_at;
+        $this->set_current_user( $old_user );
+    }
+
+    /**
      * Dopey passthrough method so we can check that the correct values
      * are being passed to the filter