Changeset 13154 for trunk/tests/phpunit/testcases/members/activity.php

Timestamp:

11/20/2021 02:21:24 PM (5 years ago)

Author:

imath

Message:

Avoid generating many new_avatar activities in a short time frame

For an hour time frame, if a user publishes many new_avatar activities, only the latest one will be kept.

Props oztaser

See #8581

File:

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

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

@@ -94 +94 @@
                 $this->assertSame( $expected, $found );
         }
+
+        /**
+         * @group bp_members_new_avatar_activity
+         */
+        public function test_bp_members_new_avatar_activity_throttled() {
+                $u = self::factory()->user->create();
+                $a = self::factory()->activity->create( array(
+                        'component' => 'members',
+                        'type'      => 'new_avatar',
+                        'user_id'   => $u,
+                ) );
+
+                bp_members_new_avatar_activity( $u );
+
+                $new_avatar_activities = bp_activity_get( array(
+                        'user_id'     => $u,
+                        'component'   => buddypress()->members->id,
+                        'type'        => 'new_avatar',
+                        'count_total' => 'count_query',
+                ) );
+
+                $this->assertEquals( 1, $new_avatar_activities['total'] );
+                $this->assertNotSame( $a, $new_avatar_activities['activities'][0]->id );
+        }
+
+        /**
+         * @group bp_members_new_avatar_activity
+         */
+        public function test_bp_members_new_avatar_activity_outside_of_throttle_time() {
+                $u = self::factory()->user->create();
+
+                $time      = strtotime( bp_core_current_time() );
+                $prev_time = date( 'Y-m-d H:i:s', $time - ( 121 * HOUR_IN_SECONDS ) );
+
+                $a = self::factory()->activity->create( array(
+                        'component'     => 'members',
+                        'type'          => 'new_avatar',
+                        'user_id'       => $u,
+                        'recorded_time' => $prev_time,
+                ) );
+
+                bp_members_new_avatar_activity( $u );
+
+                $new_avatar_activities = bp_activity_get( array(
+                        'user_id'     => $u,
+                        'component'   => buddypress()->members->id,
+                        'type'        => 'new_avatar',
+                        'count_total' => 'count_query',
+                ) );
+
+                $this->assertEquals( 2, $new_avatar_activities['total'] );
+        }
 }