Changeset 12785 for branches/6.0/tests/phpunit/testcases/xprofile/class-bp-xprofile-profiledata.php

Timestamp:

11/11/2020 11:29:36 PM (6 years ago)

Author:

imath

Message:

Ensure cached data is cleared in BP_XProfile_ProfileData::delete_data_for_user().

Data was being deleted in bulk using a direct SQL query, bypassing the xprofile_data_before_delete and xprofile_data_after_delete actions, resulting in stale data persisting in the xprofile cache. This change uses xprofile_delete_field_data() to delete the user's data so that the action hooks are called.

Props dd32, r-a-y, dcavins.

See #8388 (branch 6.0).

File:

• branches/6.0/tests/phpunit/testcases/xprofile/class-bp-xprofile-profiledata.php (modified) (1 diff)

branches/6.0/tests/phpunit/testcases/xprofile/class-bp-xprofile-profiledata.php

@@ -420 +420 @@
         }
 
+        /**
+         * @ticket BP8388
+         * @group xprofile_remove_data
+         */
+        public function test_cache_invalidated_when_xprofile_remove_data_called() {
+                $u = self::factory()->user->create();
+                $g = self::factory()->xprofile_group->create();
+                $f = self::factory()->xprofile_field->create( array(
+                        'field_group_id' => $g,
+                ) );
+
+                xprofile_set_field_data( $f, $u, 'Foo' );
+                // Prime the cache.
+                xprofile_get_field_data( $f, $u );
+                xprofile_remove_data( $u );
+
+                $d = new BP_XProfile_ProfileData( $f, $u );
+                $this->assertFalse( $d->exists() );
+        }
+
         public function filter_time() {
                 return $this->last_updated;