diff options
| author | Robin Appelman <robin@icewind.nl> | 2020-05-04 17:25:47 +0200 |
|---|---|---|
| committer | Robin Appelman <robin@icewind.nl> | 2020-05-07 17:16:43 +0200 |
| commit | aa83b0b22d26e2d16b7798d3df09f5caf19df1ae (patch) | |
| tree | 5d1346c8b61d72697f4ab63f246f829a399ac4cf /tests/lib/Group | |
| parent | 0643801d9beec3e0ba5827c5d526ef3dde223348 (diff) | |
| download | nextcloud-server-aa83b0b22d26e2d16b7798d3df09f5caf19df1ae.tar.gz nextcloud-server-aa83b0b22d26e2d16b7798d3df09f5caf19df1ae.zip | |
dont get the group details if we only ask for the id
currenty when getting the groups for a user, the full group object is always created (and cached)
even if only the groupid is required
Signed-off-by: Robin Appelman <robin@icewind.nl>
Diffstat (limited to 'tests/lib/Group')
| -rw-r--r-- | tests/lib/Group/ManagerTest.php | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/tests/lib/Group/ManagerTest.php b/tests/lib/Group/ManagerTest.php index 05ffa7973c0..ff1d6e641ea 100644 --- a/tests/lib/Group/ManagerTest.php +++ b/tests/lib/Group/ManagerTest.php @@ -390,17 +390,15 @@ class ManagerTest extends TestCase { } public function testGetUserGroupIds() { - /** @var \PHPUnit_Framework_MockObject_MockObject|\OC\Group\Manager $manager */ - $manager = $this->getMockBuilder(\OC\Group\Manager::class) - ->disableOriginalConstructor() - ->setMethods(['getUserGroups']) - ->getMock(); - $manager->expects($this->once()) - ->method('getUserGroups') - ->willReturn([ - '123' => '123', - 'abc' => 'abc', - ]); + /** + * @var \PHPUnit_Framework_MockObject_MockObject | \OC\Group\Backend $backend + */ + $backend = $this->getTestBackend(); + $backend->method('getUserGroups') + ->willReturn(['123', 'abc']); + + $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger); + $manager->addBackend($backend); /** @var \OC\User\User|\PHPUnit_Framework_MockObject_MockObject $user */ $user = $this->createMock(IUser::class); |