Changeset 8177 for trunk/tests/testcases/activity/functions.php

Timestamp:

03/27/2014 07:29:08 PM (10 years ago)

Author:

boonebgorges

Message:

Bail from bp_activity_remove_user_favorite() if the user has not previously favorited the item in question

The failure to do this check could result in the incorrect decrementing of the
item's favorite_count.

Fixes #5478

File:

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

trunk/tests/testcases/activity/functions.php

@@ -711 +711 @@
     }
 
+    /**
+     * @group favorites
+     * @group bp_activity_remove_user_favorite
+     */
+    public function test_remove_user_favorite_bad_activity_id() {
+        $u1 = $this->create_user();
+        $u2 = $this->create_user();
+        $a = $this->factory->activity->create();
+
+        // Only favorite for user 1
+        bp_activity_add_user_favorite( $a, $u1 );
+
+        // Removing for user 2 should fail
+        $this->assertFalse( bp_activity_remove_user_favorite( $a, $u2 ) );
+        $this->assertEquals( 1, bp_activity_get_meta( $a, 'favorite_count' ) );
+    }
+
     public function check_activity_caches() {
         foreach ( $this->acaches as $k => $v ) {