diff options
| -rw-r--r-- | lib/private/group/manager.php | 4 | ||||
| -rw-r--r-- | tests/lib/group/manager.php | 26 |
2 files changed, 29 insertions, 1 deletions
diff --git a/lib/private/group/manager.php b/lib/private/group/manager.php index 0f6ba7f24e0..73ff0e537c6 100644 --- a/lib/private/group/manager.php +++ b/lib/private/group/manager.php @@ -263,7 +263,9 @@ class Manager extends PublicEmitter implements IGroupManager { * @return array with group ids */ public function getUserGroupIds($user) { - return array_keys($this->getUserGroups($user)); + return array_map(function($value) { + return (string) $value; + }, array_keys($this->getUserGroups($user))); } /** diff --git a/tests/lib/group/manager.php b/tests/lib/group/manager.php index e3e2a96e46d..6cf473b9156 100644 --- a/tests/lib/group/manager.php +++ b/tests/lib/group/manager.php @@ -304,6 +304,32 @@ class Manager extends \Test\TestCase { $this->assertEquals('group1', $group1->getGID()); } + public function testGetUserGroupIds() { + /** @var \PHPUnit_Framework_MockObject_MockObject|\OC\Group\Manager $manager */ + $manager = $this->getMockBuilder('OC\Group\Manager') + ->disableOriginalConstructor() + ->setMethods(['getUserGroups']) + ->getMock(); + $manager->expects($this->once()) + ->method('getUserGroups') + ->willReturn([ + '123' => '123', + 'abc' => 'abc', + ]); + + /** @var \OC\User\User $user */ + $user = $this->getMockBuilder('OC\User\User') + ->disableOriginalConstructor() + ->getMock(); + + $groups = $manager->getUserGroupIds($user); + $this->assertEquals(2, count($groups)); + + foreach ($groups as $group) { + $this->assertInternalType('string', $group); + } + } + public function testInGroup() { /** * @var \PHPUnit_Framework_MockObject_MockObject | \OC_Group_Backend $backend |