Changeset 11378 for trunk/tests/phpunit/testcases/groups/class-bp-groups-member.php

Timestamp:

01/06/2017 03:56:09 AM (9 years ago)

Author:

boonebgorges

Message:

Lazy-load user property in BP_Groups_Member.

The user property is a BP_Core_User object, but is rarely
used by BP core. Only loading it when needed can save database
queries in certain cases, such as when a user is being promoted
or demoted within a group.

Props r-a-y.
Fixes #7382.

File:

• trunk/tests/phpunit/testcases/groups/class-bp-groups-member.php (modified) (1 diff)

trunk/tests/phpunit/testcases/groups/class-bp-groups-member.php

@@ -1301 +1301 @@
         $this->assertEqualSets( array( $m0, $m1 ), wp_list_pluck( $found, 'id' ) );
     }
+
+    /**
+     * @ticket BP7382
+     */
+    public function test_user_property_should_be_accessible() {
+        $user = $this->factory->user->create();
+        $group = $this->factory->group->create();
+
+        $this->add_user_to_group( $user, $group );
+
+        $membership = new BP_Groups_Member( $user, $group );
+
+        $user_obj = $membership->user;
+
+        $this->assertInstanceOf( 'BP_Core_User', $user_obj );
+        $this->assertEquals( $user, $user_obj->id );
+    }
 }