From 33b048dd1c144aabab3efccd3e1c167c2328d89d Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Tue, 28 Jan 2020 12:27:47 +0100 Subject: [PATCH 1/3] Use contacts name on federated activities Signed-off-by: Joas Schilling --- apps/files/lib/Activity/Provider.php | 46 ++++++++++++++----- .../lib/Activity/Providers/Base.php | 46 +++++++++++++++---- .../lib/Activity/Providers/Groups.php | 17 ++++--- .../lib/Activity/Providers/RemoteShares.php | 13 ++---- 4 files changed, 84 insertions(+), 38 deletions(-) diff --git a/apps/files/lib/Activity/Provider.php b/apps/files/lib/Activity/Provider.php index 86f5e71b5804b..5749473783e96 100644 --- a/apps/files/lib/Activity/Provider.php +++ b/apps/files/lib/Activity/Provider.php @@ -28,6 +28,7 @@ use OCP\Activity\IEventMerger; use OCP\Activity\IManager; use OCP\Activity\IProvider; +use OCP\Contacts\IManager as IContactsManager; use OCP\Files\Folder; use OCP\Files\InvalidPathException; use OCP\Files\IRootFolder; @@ -64,25 +65,27 @@ class Provider implements IProvider { /** @var IEventMerger */ protected $eventMerger; + /** @var IContactsManager */ + protected $contactsManager; + /** @var string[] cached displayNames - key is the UID and value the displayname */ protected $displayNames = []; protected $fileIsEncrypted = false; - /** - * @param IFactory $languageFactory - * @param IURLGenerator $url - * @param IManager $activityManager - * @param IUserManager $userManager - * @param IRootFolder $rootFolder - * @param IEventMerger $eventMerger - */ - public function __construct(IFactory $languageFactory, IURLGenerator $url, IManager $activityManager, IUserManager $userManager, IRootFolder $rootFolder, IEventMerger $eventMerger) { + public function __construct(IFactory $languageFactory, + IURLGenerator $url, + IManager $activityManager, + IUserManager $userManager, + IRootFolder $rootFolder, + IContactsManager $contactsManager, + IEventMerger $eventMerger) { $this->languageFactory = $languageFactory; $this->url = $url; $this->activityManager = $activityManager; $this->userManager = $userManager; $this->rootFolder = $rootFolder; + $this->contactsManager = $contactsManager; $this->eventMerger = $eventMerger; } @@ -495,8 +498,29 @@ protected function getDisplayName($uid) { $user = $this->userManager->get($uid); if ($user instanceof IUser) { return $user->getDisplayName(); - } else { - return $uid; } + + $addressBookContacts = $this->contactsManager->search($uid, ['CLOUD']); + foreach ($addressBookContacts as $contact) { + if (isset($contact['isLocalSystemBook'])) { + continue; + } + + if (isset($contact['CLOUD'])) { + $cloudIds = $contact['CLOUD']; + if (is_string($cloudIds)) { + $cloudIds = [$cloudIds]; + } + + $lowerSearch = strtolower($uid); + foreach ($cloudIds as $cloudId) { + if (strtolower($cloudId) === $lowerSearch) { + return $contact['FN'] . " ($cloudId)"; + } + } + } + } + + return $uid; } } diff --git a/apps/files_sharing/lib/Activity/Providers/Base.php b/apps/files_sharing/lib/Activity/Providers/Base.php index 237e4c5f377fd..77ebfc26447be 100644 --- a/apps/files_sharing/lib/Activity/Providers/Base.php +++ b/apps/files_sharing/lib/Activity/Providers/Base.php @@ -26,6 +26,7 @@ use OCP\Activity\IEvent; use OCP\Activity\IManager; use OCP\Activity\IProvider; +use OCP\Contacts\IManager as IContactsManager; use OCP\IL10N; use OCP\IURLGenerator; use OCP\IUser; @@ -49,20 +50,22 @@ abstract class Base implements IProvider { /** @var IUserManager */ protected $userManager; + /** @var IContactsManager */ + protected $contactsManager; + /** @var array */ protected $displayNames = []; - /** - * @param IFactory $languageFactory - * @param IURLGenerator $url - * @param IManager $activityManager - * @param IUserManager $userManager - */ - public function __construct(IFactory $languageFactory, IURLGenerator $url, IManager $activityManager, IUserManager $userManager) { + public function __construct(IFactory $languageFactory, + IURLGenerator $url, + IManager $activityManager, + IUserManager $userManager, + IContactsManager $contactsManager) { $this->languageFactory = $languageFactory; $this->url = $url; $this->activityManager = $activityManager; $this->userManager = $userManager; + $this->contactsManager = $contactsManager; } /** @@ -179,8 +182,33 @@ protected function getDisplayName($uid) { $user = $this->userManager->get($uid); if ($user instanceof IUser) { return $user->getDisplayName(); - } else { - return $uid; } + + return $this->getDisplayNameForContact($uid); + } + + protected function getDisplayNameForContact(string $search): string { + $addressBookContacts = $this->contactsManager->search($search, ['CLOUD']); + foreach ($addressBookContacts as $contact) { + if (isset($contact['isLocalSystemBook'])) { + continue; + } + + if (isset($contact['CLOUD'])) { + $cloudIds = $contact['CLOUD']; + if (is_string($cloudIds)) { + $cloudIds = [$cloudIds]; + } + + $lowerSearch = strtolower($search); + foreach ($cloudIds as $cloudId) { + if (strtolower($cloudId) === $lowerSearch) { + return $contact['FN'] . " ($cloudId)"; + } + } + } + } + + return $search; } } diff --git a/apps/files_sharing/lib/Activity/Providers/Groups.php b/apps/files_sharing/lib/Activity/Providers/Groups.php index 184e4a5b70d6f..35fd03e0caa7b 100644 --- a/apps/files_sharing/lib/Activity/Providers/Groups.php +++ b/apps/files_sharing/lib/Activity/Providers/Groups.php @@ -26,6 +26,7 @@ use OCP\Activity\IEvent; use OCP\Activity\IManager; +use OCP\Contacts\IManager as IContactsManager; use OCP\IGroup; use OCP\IGroupManager; use OCP\IURLGenerator; @@ -48,15 +49,13 @@ class Groups extends Base { /** @var string[] */ protected $groupDisplayNames = []; - /** - * @param IFactory $languageFactory - * @param IURLGenerator $url - * @param IManager $activityManager - * @param IUserManager $userManager - * @param IGroupManager $groupManager - */ - public function __construct(IFactory $languageFactory, IURLGenerator $url, IManager $activityManager, IUserManager $userManager, IGroupManager $groupManager) { - parent::__construct($languageFactory, $url, $activityManager, $userManager); + public function __construct(IFactory $languageFactory, + IURLGenerator $url, + IManager $activityManager, + IUserManager $userManager, + IContactsManager $contactsManager, + IGroupManager $groupManager) { + parent::__construct($languageFactory, $url, $activityManager, $userManager, $contactsManager); $this->groupManager = $groupManager; } diff --git a/apps/files_sharing/lib/Activity/Providers/RemoteShares.php b/apps/files_sharing/lib/Activity/Providers/RemoteShares.php index 2bdf36f7eead4..28c62845f8178 100644 --- a/apps/files_sharing/lib/Activity/Providers/RemoteShares.php +++ b/apps/files_sharing/lib/Activity/Providers/RemoteShares.php @@ -26,6 +26,7 @@ use OCP\Activity\IEvent; use OCP\Activity\IManager; +use OCP\Contacts\IManager as IContactsManager; use OCP\Federation\ICloudIdManager; use OCP\IURLGenerator; use OCP\IUserManager; @@ -40,20 +41,14 @@ class RemoteShares extends Base { const SUBJECT_REMOTE_SHARE_RECEIVED = 'remote_share_received'; const SUBJECT_REMOTE_SHARE_UNSHARED = 'remote_share_unshared'; - /** - * @param IFactory $languageFactory - * @param IURLGenerator $url - * @param IManager $activityManager - * @param IUserManager $userManager - * @param ICloudIdManager $cloudIdManager - */ public function __construct(IFactory $languageFactory, IURLGenerator $url, IManager $activityManager, IUserManager $userManager, + IContactsManager $contactsManager, ICloudIdManager $cloudIdManager ) { - parent::__construct($languageFactory, $url, $activityManager, $userManager); + parent::__construct($languageFactory, $url, $activityManager, $userManager, $contactsManager); $this->cloudIdManager = $cloudIdManager; } @@ -153,7 +148,7 @@ protected function getFederatedUser($cloudId) { return [ 'type' => 'user', 'id' => $remoteUser->getUser(), - 'name' => $cloudId,// Todo display name from contacts + 'name' => $this->getDisplayNameForContact($remoteUser->getDisplayId()), 'server' => $remoteUser->getRemote(), ]; } From 26568fa08b67b39d129c0e2f33c347cfba74e028 Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Tue, 28 Jan 2020 12:51:43 +0100 Subject: [PATCH 2/3] Unify the code a bit Signed-off-by: Joas Schilling --- apps/files/lib/Activity/Provider.php | 52 +++++++++++++------ .../lib/Activity/Providers/Base.php | 48 +++++++++++------ .../lib/Activity/Providers/Groups.php | 4 +- .../lib/Activity/Providers/RemoteShares.php | 23 ++------ 4 files changed, 74 insertions(+), 53 deletions(-) diff --git a/apps/files/lib/Activity/Provider.php b/apps/files/lib/Activity/Provider.php index 5749473783e96..07771b7f12e2d 100644 --- a/apps/files/lib/Activity/Provider.php +++ b/apps/files/lib/Activity/Provider.php @@ -29,6 +29,7 @@ use OCP\Activity\IManager; use OCP\Activity\IProvider; use OCP\Contacts\IManager as IContactsManager; +use OCP\Federation\ICloudIdManager; use OCP\Files\Folder; use OCP\Files\InvalidPathException; use OCP\Files\IRootFolder; @@ -65,10 +66,13 @@ class Provider implements IProvider { /** @var IEventMerger */ protected $eventMerger; + /** @var ICloudIdManager */ + protected $cloudIdManager; + /** @var IContactsManager */ protected $contactsManager; - /** @var string[] cached displayNames - key is the UID and value the displayname */ + /** @var string[] cached displayNames - key is the cloud id and value the displayname */ protected $displayNames = []; protected $fileIsEncrypted = false; @@ -78,6 +82,7 @@ public function __construct(IFactory $languageFactory, IManager $activityManager, IUserManager $userManager, IRootFolder $rootFolder, + ICloudIdManager $cloudIdManager, IContactsManager $contactsManager, IEventMerger $eventMerger) { $this->languageFactory = $languageFactory; @@ -85,6 +90,7 @@ public function __construct(IFactory $languageFactory, $this->activityManager = $activityManager; $this->userManager = $userManager; $this->rootFolder = $rootFolder; + $this->cloudIdManager = $cloudIdManager; $this->contactsManager = $contactsManager; $this->eventMerger = $eventMerger; } @@ -479,28 +485,41 @@ protected function getParentEndToEndEncryptionContainer(Folder $userFolder, Node * @return array */ protected function getUser($uid) { - if (!isset($this->displayNames[$uid])) { - $this->displayNames[$uid] = $this->getDisplayName($uid); + // First try local user + $user = $this->userManager->get($uid); + if ($user instanceof IUser) { + return [ + 'type' => 'user', + 'id' => $user->getUID(), + 'name' => $user->getDisplayName(), + ]; + } + + // Then a contact from the addressbook + if ($this->cloudIdManager->isValidCloudId($uid)) { + $cloudId = $this->cloudIdManager->resolveCloudId($uid); + return [ + 'type' => 'user', + 'id' => $cloudId->getUser(), + 'name' => $this->getDisplayNameFromAddressBook($cloudId->getDisplayId()), + 'server' => $cloudId->getRemote(), + ]; } + // Fallback to empty dummy data return [ 'type' => 'user', 'id' => $uid, - 'name' => $this->displayNames[$uid], + 'name' => $uid, ]; } - /** - * @param string $uid - * @return string - */ - protected function getDisplayName($uid) { - $user = $this->userManager->get($uid); - if ($user instanceof IUser) { - return $user->getDisplayName(); + protected function getDisplayNameFromAddressBook(string $search): string { + if (isset($this->displayNames[$search])) { + return $this->displayNames[$search]; } - $addressBookContacts = $this->contactsManager->search($uid, ['CLOUD']); + $addressBookContacts = $this->contactsManager->search($search, ['CLOUD']); foreach ($addressBookContacts as $contact) { if (isset($contact['isLocalSystemBook'])) { continue; @@ -512,15 +531,16 @@ protected function getDisplayName($uid) { $cloudIds = [$cloudIds]; } - $lowerSearch = strtolower($uid); + $lowerSearch = strtolower($search); foreach ($cloudIds as $cloudId) { if (strtolower($cloudId) === $lowerSearch) { - return $contact['FN'] . " ($cloudId)"; + $this->displayNames[$search] = $contact['FN'] . " ($cloudId)"; + return $this->displayNames[$search]; } } } } - return $uid; + return $search; } } diff --git a/apps/files_sharing/lib/Activity/Providers/Base.php b/apps/files_sharing/lib/Activity/Providers/Base.php index 77ebfc26447be..b63eb346f7cb0 100644 --- a/apps/files_sharing/lib/Activity/Providers/Base.php +++ b/apps/files_sharing/lib/Activity/Providers/Base.php @@ -27,6 +27,7 @@ use OCP\Activity\IManager; use OCP\Activity\IProvider; use OCP\Contacts\IManager as IContactsManager; +use OCP\Federation\ICloudIdManager; use OCP\IL10N; use OCP\IURLGenerator; use OCP\IUser; @@ -53,6 +54,9 @@ abstract class Base implements IProvider { /** @var IContactsManager */ protected $contactsManager; + /** @var ICloudIdManager */ + protected $cloudIdManager; + /** @var array */ protected $displayNames = []; @@ -60,11 +64,13 @@ public function __construct(IFactory $languageFactory, IURLGenerator $url, IManager $activityManager, IUserManager $userManager, + ICloudIdManager $cloudIdManager, IContactsManager $contactsManager) { $this->languageFactory = $languageFactory; $this->url = $url; $this->activityManager = $activityManager; $this->userManager = $userManager; + $this->cloudIdManager = $cloudIdManager; $this->contactsManager = $contactsManager; } @@ -163,31 +169,40 @@ protected function getFile($parameter, IEvent $event = null) { * @return array */ protected function getUser($uid) { - if (!isset($this->displayNames[$uid])) { - $this->displayNames[$uid] = $this->getDisplayName($uid); + // First try local user + $user = $this->userManager->get($uid); + if ($user instanceof IUser) { + return [ + 'type' => 'user', + 'id' => $user->getUID(), + 'name' => $user->getDisplayName(), + ]; } + // Then a contact from the addressbook + if ($this->cloudIdManager->isValidCloudId($uid)) { + $cloudId = $this->cloudIdManager->resolveCloudId($uid); + return [ + 'type' => 'user', + 'id' => $cloudId->getUser(), + 'name' => $this->getDisplayNameFromAddressBook($cloudId->getDisplayId()), + 'server' => $cloudId->getRemote(), + ]; + } + + // Fallback to empty dummy data return [ 'type' => 'user', 'id' => $uid, - 'name' => $this->displayNames[$uid], + 'name' => $uid, ]; } - /** - * @param string $uid - * @return string - */ - protected function getDisplayName($uid) { - $user = $this->userManager->get($uid); - if ($user instanceof IUser) { - return $user->getDisplayName(); + protected function getDisplayNameFromAddressBook(string $search): string { + if (isset($this->displayNames[$search])) { + return $this->displayNames[$search]; } - return $this->getDisplayNameForContact($uid); - } - - protected function getDisplayNameForContact(string $search): string { $addressBookContacts = $this->contactsManager->search($search, ['CLOUD']); foreach ($addressBookContacts as $contact) { if (isset($contact['isLocalSystemBook'])) { @@ -203,7 +218,8 @@ protected function getDisplayNameForContact(string $search): string { $lowerSearch = strtolower($search); foreach ($cloudIds as $cloudId) { if (strtolower($cloudId) === $lowerSearch) { - return $contact['FN'] . " ($cloudId)"; + $this->displayNames[$search] = $contact['FN'] . " ($cloudId)"; + return $this->displayNames[$search]; } } } diff --git a/apps/files_sharing/lib/Activity/Providers/Groups.php b/apps/files_sharing/lib/Activity/Providers/Groups.php index 35fd03e0caa7b..8a061f9373def 100644 --- a/apps/files_sharing/lib/Activity/Providers/Groups.php +++ b/apps/files_sharing/lib/Activity/Providers/Groups.php @@ -27,6 +27,7 @@ use OCP\Activity\IEvent; use OCP\Activity\IManager; use OCP\Contacts\IManager as IContactsManager; +use OCP\Federation\ICloudIdManager; use OCP\IGroup; use OCP\IGroupManager; use OCP\IURLGenerator; @@ -53,9 +54,10 @@ public function __construct(IFactory $languageFactory, IURLGenerator $url, IManager $activityManager, IUserManager $userManager, + ICloudIdManager $cloudIdManager, IContactsManager $contactsManager, IGroupManager $groupManager) { - parent::__construct($languageFactory, $url, $activityManager, $userManager, $contactsManager); + parent::__construct($languageFactory, $url, $activityManager, $userManager, $cloudIdManager, $contactsManager); $this->groupManager = $groupManager; } diff --git a/apps/files_sharing/lib/Activity/Providers/RemoteShares.php b/apps/files_sharing/lib/Activity/Providers/RemoteShares.php index 28c62845f8178..36d4dbbb8fdb1 100644 --- a/apps/files_sharing/lib/Activity/Providers/RemoteShares.php +++ b/apps/files_sharing/lib/Activity/Providers/RemoteShares.php @@ -34,8 +34,6 @@ class RemoteShares extends Base { - protected $cloudIdManager; - const SUBJECT_REMOTE_SHARE_ACCEPTED = 'remote_share_accepted'; const SUBJECT_REMOTE_SHARE_DECLINED = 'remote_share_declined'; const SUBJECT_REMOTE_SHARE_RECEIVED = 'remote_share_received'; @@ -48,8 +46,7 @@ public function __construct(IFactory $languageFactory, IContactsManager $contactsManager, ICloudIdManager $cloudIdManager ) { - parent::__construct($languageFactory, $url, $activityManager, $userManager, $contactsManager); - $this->cloudIdManager = $cloudIdManager; + parent::__construct($languageFactory, $url, $activityManager, $userManager, $cloudIdManager, $contactsManager); } /** @@ -123,7 +120,7 @@ protected function getParsedParameters(IEvent $event) { 'id' => $parameters[1], 'name' => $parameters[1], ], - 'user' => $this->getFederatedUser($parameters[0]), + 'user' => $this->getUser($parameters[0]), ]; case self::SUBJECT_REMOTE_SHARE_ACCEPTED: case self::SUBJECT_REMOTE_SHARE_DECLINED: @@ -133,23 +130,9 @@ protected function getParsedParameters(IEvent $event) { } return [ 'file' => $this->getFile($fileParameter), - 'user' => $this->getFederatedUser($parameters[0]), + 'user' => $this->getUser($parameters[0]), ]; } throw new \InvalidArgumentException(); } - - /** - * @param string $cloudId - * @return array - */ - protected function getFederatedUser($cloudId) { - $remoteUser = $this->cloudIdManager->resolveCloudId($cloudId); - return [ - 'type' => 'user', - 'id' => $remoteUser->getUser(), - 'name' => $this->getDisplayNameForContact($remoteUser->getDisplayId()), - 'server' => $remoteUser->getRemote(), - ]; - } } From d2687d69ab903d40ef43f0f4ba28dd0d91798f61 Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Fri, 7 Feb 2020 10:51:22 +0100 Subject: [PATCH 3/3] Fix unit tests Signed-off-by: Joas Schilling --- apps/files/tests/Activity/ProviderTest.php | 131 ++++++++++++--------- 1 file changed, 75 insertions(+), 56 deletions(-) diff --git a/apps/files/tests/Activity/ProviderTest.php b/apps/files/tests/Activity/ProviderTest.php index 61a7f105aa7b5..5e4b18c0bb166 100644 --- a/apps/files/tests/Activity/ProviderTest.php +++ b/apps/files/tests/Activity/ProviderTest.php @@ -29,11 +29,15 @@ use OCP\Activity\IEvent; use OCP\Activity\IEventMerger; use OCP\Activity\IManager; +use OCP\Contacts\IManager as IContactsManager; +use OCP\Federation\ICloudId; +use OCP\Federation\ICloudIdManager; use OCP\Files\IRootFolder; use OCP\IURLGenerator; use OCP\IUser; use OCP\IUserManager; use OCP\L10N\IFactory; +use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; /** @@ -43,17 +47,21 @@ */ class ProviderTest extends TestCase { - /** @var IFactory|\PHPUnit_Framework_MockObject_MockObject */ + /** @var IFactory|MockObject */ protected $l10nFactory; - /** @var IURLGenerator|\PHPUnit_Framework_MockObject_MockObject */ + /** @var IURLGenerator|MockObject */ protected $url; - /** @var IManager|\PHPUnit_Framework_MockObject_MockObject */ + /** @var IManager|MockObject */ protected $activityManager; - /** @var IUserManager|\PHPUnit_Framework_MockObject_MockObject */ + /** @var IUserManager|MockObject */ protected $userManager; - /** @var IRootFolder|\PHPUnit_Framework_MockObject_MockObject */ + /** @var IRootFolder|MockObject */ protected $rootFolder; - /** @var IEventMerger|\PHPUnit_Framework_MockObject_MockObject */ + /** @var ICloudIdManager|MockObject */ + protected $cloudIdManager; + /** @var IContactsManager|MockObject */ + protected $contactsManager; + /** @var IEventMerger|MockObject */ protected $eventMerger; protected function setUp(): void { @@ -64,12 +72,14 @@ protected function setUp(): void { $this->activityManager = $this->createMock(IManager::class); $this->userManager = $this->createMock(IUserManager::class); $this->rootFolder = $this->createMock(IRootFolder::class); + $this->cloudIdManager = $this->createMock(ICloudIdManager::class); + $this->contactsManager = $this->createMock(IContactsManager::class); $this->eventMerger = $this->createMock(IEventMerger::class); } /** * @param string[] $methods - * @return Provider|\PHPUnit_Framework_MockObject_MockObject + * @return Provider|MockObject */ protected function getProvider(array $methods = []) { if (!empty($methods)) { @@ -80,6 +90,8 @@ protected function getProvider(array $methods = []) { $this->activityManager, $this->userManager, $this->rootFolder, + $this->cloudIdManager, + $this->contactsManager, $this->eventMerger, ]) ->setMethods($methods) @@ -91,6 +103,8 @@ protected function getProvider(array $methods = []) { $this->activityManager, $this->userManager, $this->rootFolder, + $this->cloudIdManager, + $this->contactsManager, $this->eventMerger ); } @@ -99,7 +113,7 @@ public function dataGetFile() { return [ [[42 => '/FortyTwo.txt'], null, '42', 'FortyTwo.txt', 'FortyTwo.txt'], [['23' => '/Twenty/Three.txt'], null, '23', 'Three.txt', 'Twenty/Three.txt'], - ['/Foo/Bar.txt', '128', 128, 'Bar.txt', 'Foo/Bar.txt'], // Legacy from ownCloud 8.2 and before + ['/Foo/Bar.txt', 128, 128, 'Bar.txt', 'Foo/Bar.txt'], // Legacy from ownCloud 8.2 and before ]; } @@ -137,7 +151,7 @@ public function testGetFile($parameter, $eventId, $id, $name, $path) { $this->assertSame('link-' . $id, $result['link']); } - + public function testGetFileThrows() { $this->expectException(\InvalidArgumentException::class); @@ -147,71 +161,76 @@ public function testGetFileThrows() { public function dataGetUser() { return [ - ['test', [], false, 'Test'], - ['foo', ['admin' => 'Admin'], false, 'Bar'], - ['admin', ['admin' => 'Administrator'], true, 'Administrator'], + ['test', 'Test user', null, ['type' => 'user', 'id' => 'test', 'name' => 'Test user']], + ['test@http://localhost', null, ['user' => 'test', 'displayId' => 'test@localhost', 'remote' => 'localhost', 'name' => null], ['type' => 'user', 'id' => 'test', 'name' => 'test@localhost', 'server' => 'localhost']], + ['test@http://localhost', null, ['user' => 'test', 'displayId' => 'test@localhost', 'remote' => 'localhost', 'name' => 'Remote user'], ['type' => 'user', 'id' => 'test', 'name' => 'Remote user (test@localhost)', 'server' => 'localhost']], + ['test', null, null, ['type' => 'user', 'id' => 'test', 'name' => 'test']], ]; } /** * @dataProvider dataGetUser * @param string $uid - * @param array $cache - * @param bool $cacheHit - * @param string $name + * @param string|null $userDisplayName + * @param array|null $cloudIdData + * @param array $expected */ - public function testGetUser($uid, $cache, $cacheHit, $name) { - $provider = $this->getProvider(['getDisplayName']); - - self::invokePrivate($provider, 'displayNames', [$cache]); - - if (!$cacheHit) { - $provider->expects($this->once()) - ->method('getDisplayName') - ->with($uid) - ->willReturn($name); - } else { - $provider->expects($this->never()) - ->method('getDisplayName'); - } - - $result = self::invokePrivate($provider, 'getUser', [$uid]); - $this->assertSame('user', $result['type']); - $this->assertSame($uid, $result['id']); - $this->assertSame($name, $result['name']); - } - - public function dataGetDisplayName() { - return [ - ['test', true, 'Test'], - ['foo', false, 'foo'], - ]; - } - - /** - * @dataProvider dataGetDisplayName - * @param string $uid - * @param string $name - */ - public function testGetDisplayNamer($uid, $validUser, $name) { + public function testGetUser(string $uid, ?string $userDisplayName, ?array $cloudIdData, array $expected): void { $provider = $this->getProvider(); - if ($validUser) { + if ($userDisplayName !== null) { $user = $this->createMock(IUser::class); + $user->expects($this->once()) + ->method('getUID') + ->willReturn($uid); $user->expects($this->once()) ->method('getDisplayName') - ->willReturn($name); + ->willReturn($userDisplayName); $this->userManager->expects($this->once()) ->method('get') ->with($uid) ->willReturn($user); - } else { - $this->userManager->expects($this->once()) - ->method('get') + } + if ($cloudIdData !== null) { + $this->cloudIdManager->expects($this->once()) + ->method('isValidCloudId') + ->willReturn(true); + + $cloudId = $this->createMock(ICloudId::class); + $cloudId->expects($this->once()) + ->method('getUser') + ->willReturn($cloudIdData['user']); + $cloudId->expects($this->once()) + ->method('getDisplayId') + ->willReturn($cloudIdData['displayId']); + $cloudId->expects($this->once()) + ->method('getRemote') + ->willReturn($cloudIdData['remote']); + + $this->cloudIdManager->expects($this->once()) + ->method('resolveCloudId') ->with($uid) - ->willReturn(null); + ->willReturn($cloudId); + + if ($cloudIdData['name'] !== null) { + $this->contactsManager->expects($this->once()) + ->method('search') + ->with($cloudIdData['displayId'], ['CLOUD']) + ->willReturn([ + [ + 'CLOUD' => $cloudIdData['displayId'], + 'FN' => $cloudIdData['name'], + ] + ]); + } else { + $this->contactsManager->expects($this->once()) + ->method('search') + ->with($cloudIdData['displayId'], ['CLOUD']) + ->willReturn([]); + } } - $this->assertSame($name, self::invokePrivate($provider, 'getDisplayName', [$uid])); + $result = self::invokePrivate($provider, 'getUser', [$uid]); + $this->assertEquals($expected, $result); } }