Changeset 7883 for trunk/tests/testcases/xprofile/functions.php

Timestamp:

02/14/2014 07:02:16 PM (11 years ago)

Author:

boonebgorges

Message:

Add delete_all support for all _delete_meta() functions

The delete_all parameter allows you to delete all meta items matching the
specified meta_key, regardless of the associated object.

Because our meta delete functions also support the deletion of all metadata
associated with an object (by leaving out the meta_key param), a decision had
to be made regarding the behavior when _delete_meta() is called with delete_all

true and meta_key = false. The most logical (and least destructive) strategy

was deemed to be: force delete_all to false when meta_key is also false. This
ensures that you don't accidentally wipe out all metadata for a component.

See #5400

File:

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

trunk/tests/testcases/xprofile/functions.php

@@ -234 +234 @@
         $this->assertTrue( bp_xprofile_delete_meta( $g, 'group' ) );
 
-        // These will fail because of a caching bug
         $this->assertEquals( '', bp_xprofile_get_meta( $g, 'group', 'foo' ) );
         $this->assertEquals( '', bp_xprofile_get_meta( $g, 'group', 'foo2' ) );
+    }
+
+    /**
+     * @group xprofilemeta
+     * @group bp_xprofile_delete_meta
+     */
+    public function test_bp_xprofile_delete_meta_with_delete_all_but_no_meta_key() {
+        // With no meta key, don't delete for all items - just delete
+        // all for a single item
+        $g1 = $this->factory->xprofile_group->create();
+        $g2 = $this->factory->xprofile_group->create();
+        bp_xprofile_add_meta( $g1, 'group', 'foo', 'bar' );
+        bp_xprofile_add_meta( $g1, 'group', 'foo1', 'bar1' );
+        bp_xprofile_add_meta( $g2, 'group', 'foo', 'bar' );
+        bp_xprofile_add_meta( $g2, 'group', 'foo1', 'bar1' );
+
+        $this->assertTrue( bp_xprofile_delete_meta( $g1, 'group', '', '', true ) );
+        $this->assertEmpty( bp_xprofile_get_meta( $g1, 'group' ) );
+        $this->assertSame( 'bar', bp_xprofile_get_meta( $g2, 'group', 'foo' ) );
+        $this->assertSame( 'bar1', bp_xprofile_get_meta( $g2, 'group', 'foo1' ) );
+    }
+
+    /**
+     * @group xprofilemeta
+     * @group bp_xprofile_delete_meta
+     */
+    public function test_bp_xprofile_delete_meta_with_delete_all() {
+        // With no meta key, don't delete for all items - just delete
+        // all for a single item
+        $g1 = $this->factory->xprofile_group->create();
+        $g2 = $this->factory->xprofile_group->create();
+        bp_xprofile_add_meta( $g1, 'group', 'foo', 'bar' );
+        bp_xprofile_add_meta( $g1, 'group', 'foo1', 'bar1' );
+        bp_xprofile_add_meta( $g2, 'group', 'foo', 'bar' );
+        bp_xprofile_add_meta( $g2, 'group', 'foo1', 'bar1' );
+
+        $this->assertTrue( bp_xprofile_delete_meta( $g1, 'group', 'foo', '', true ) );
+        $this->assertEmpty( '', bp_xprofile_get_meta( $g1, 'group', 'foo' ) );
+        $this->assertEmpty( '', bp_xprofile_get_meta( $g2, 'group', 'foo' ) );
+        $this->assertSame( 'bar1', bp_xprofile_get_meta( $g1, 'group', 'foo1' ) );
+        $this->assertSame( 'bar1', bp_xprofile_get_meta( $g2, 'group', 'foo1' ) );
     }