123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888 |
- <?php
- /**
- * SPDX-FileCopyrightText: 2016-2024 Nextcloud GmbH and Nextcloud contributors
- * SPDX-FileCopyrightText: 2016 ownCloud, Inc.
- * SPDX-License-Identifier: AGPL-3.0-only
- */
- namespace Test\Accounts;
- use OC\Accounts\Account;
- use OC\Accounts\AccountManager;
- use OC\PhoneNumberUtil;
- use OCA\Settings\BackgroundJobs\VerifyUserData;
- use OCP\Accounts\IAccountManager;
- use OCP\Accounts\UserUpdatedEvent;
- use OCP\BackgroundJob\IJobList;
- use OCP\Defaults;
- use OCP\EventDispatcher\IEventDispatcher;
- use OCP\IConfig;
- use OCP\IDBConnection;
- use OCP\IPhoneNumberUtil;
- use OCP\IURLGenerator;
- use OCP\IUser;
- use OCP\L10N\IFactory;
- use OCP\Mail\IMailer;
- use OCP\Security\ICrypto;
- use OCP\Security\VerificationToken\IVerificationToken;
- use PHPUnit\Framework\MockObject\MockObject;
- use Psr\Log\LoggerInterface;
- use Test\TestCase;
- /**
- * Class AccountManagerTest
- *
- * @group DB
- * @package Test\Accounts
- */
- class AccountManagerTest extends TestCase {
- /** @var IVerificationToken|MockObject */
- protected $verificationToken;
- /** @var IMailer|MockObject */
- protected $mailer;
- /** @var ICrypto|MockObject */
- protected $crypto;
- /** @var IURLGenerator|MockObject */
- protected $urlGenerator;
- /** @var Defaults|MockObject */
- protected $defaults;
- /** @var IFactory|MockObject */
- protected $l10nFactory;
- /** @var IDBConnection */
- private $connection;
- /** @var IConfig|MockObject */
- private $config;
- /** @var IEventDispatcher|MockObject */
- private $eventDispatcher;
- /** @var IJobList|MockObject */
- private $jobList;
- /** @var IPhoneNumberUtil */
- private $phoneNumberUtil;
- /** accounts table name */
- private string $table = 'accounts';
- /** @var LoggerInterface|MockObject */
- private $logger;
- private AccountManager $accountManager;
- protected function setUp(): void {
- parent::setUp();
- $this->eventDispatcher = $this->createMock(IEventDispatcher::class);
- $this->connection = \OC::$server->get(IDBConnection::class);
- $this->config = $this->createMock(IConfig::class);
- $this->jobList = $this->createMock(IJobList::class);
- $this->logger = $this->createMock(LoggerInterface::class);
- $this->verificationToken = $this->createMock(IVerificationToken::class);
- $this->mailer = $this->createMock(IMailer::class);
- $this->defaults = $this->createMock(Defaults::class);
- $this->l10nFactory = $this->createMock(IFactory::class);
- $this->urlGenerator = $this->createMock(IURLGenerator::class);
- $this->crypto = $this->createMock(ICrypto::class);
- $this->phoneNumberUtil = new PhoneNumberUtil();
- $this->accountManager = new AccountManager(
- $this->connection,
- $this->config,
- $this->eventDispatcher,
- $this->jobList,
- $this->logger,
- $this->verificationToken,
- $this->mailer,
- $this->defaults,
- $this->l10nFactory,
- $this->urlGenerator,
- $this->crypto,
- $this->phoneNumberUtil,
- );
- }
- protected function tearDown(): void {
- parent::tearDown();
- $query = $this->connection->getQueryBuilder();
- $query->delete($this->table)->executeStatement();
- }
- protected function makeUser(string $uid, string $name, ?string $email = null): IUser {
- $user = $this->createMock(IUser::class);
- $user->expects($this->any())
- ->method('getUid')
- ->willReturn($uid);
- $user->expects($this->any())
- ->method('getDisplayName')
- ->willReturn($name);
- if ($email !== null) {
- $user->expects($this->any())
- ->method('getEMailAddress')
- ->willReturn($email);
- }
- return $user;
- }
- protected function populateOrUpdate(): void {
- $users = [
- [
- 'user' => $this->makeUser('j.doe', 'Jane Doe', 'jane.doe@acme.com'),
- 'data' => [
- [
- 'name' => IAccountManager::PROPERTY_DISPLAYNAME,
- 'value' => 'Jane Doe',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_EMAIL,
- 'value' => 'jane.doe@acme.com',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_TWITTER,
- 'value' => '@sometwitter',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_FEDIVERSE,
- 'value' => '@someMastodon@mastodon.social',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_PHONE,
- 'value' => '+491601231212',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ADDRESS,
- 'value' => 'some street',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_WEBSITE,
- 'value' => 'https://acme.com',
- 'scope' => IAccountManager::SCOPE_PRIVATE
- ],
- [
- 'name' => IAccountManager::PROPERTY_ORGANISATION,
- 'value' => 'Some organisation',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_ROLE,
- 'value' => 'Human',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_HEADLINE,
- 'value' => 'Hi',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_BIOGRAPHY,
- 'value' => 'Biography',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- ],
- ],
- [
- 'user' => $this->makeUser('a.allison', 'Alice Allison', 'a.allison@example.org'),
- 'data' => [
- [
- 'name' => IAccountManager::PROPERTY_DISPLAYNAME,
- 'value' => 'Alice Allison',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_EMAIL,
- 'value' => 'a.allison@example.org',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_TWITTER,
- 'value' => '@a_alice',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_FEDIVERSE,
- 'value' => '@a_alice@cool.social',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_PHONE,
- 'value' => '+491602312121',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_ADDRESS,
- 'value' => 'Dundee Road 45',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_WEBSITE,
- 'value' => 'https://example.org',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_ORGANISATION,
- 'value' => 'Another organisation',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ROLE,
- 'value' => 'Alien',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_HEADLINE,
- 'value' => 'Hello',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_BIOGRAPHY,
- 'value' => 'Different biography',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- ],
- ],
- [
- 'user' => $this->makeUser('b32c5a5b-1084-4380-8856-e5223b16de9f', 'Armel Oliseh', 'oliseh@example.com'),
- 'data' => [
- [
- 'name' => IAccountManager::PROPERTY_DISPLAYNAME,
- 'value' => 'Armel Oliseh',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_EMAIL,
- 'value' => 'oliseh@example.com',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_TWITTER,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_FEDIVERSE,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_PHONE,
- 'value' => '+491603121212',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ADDRESS,
- 'value' => 'Sunflower Blvd. 77',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_WEBSITE,
- 'value' => 'https://example.com',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ORGANISATION,
- 'value' => 'Yet another organisation',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ROLE,
- 'value' => 'Being',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_HEADLINE,
- 'value' => 'This is a headline',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_BIOGRAPHY,
- 'value' => 'Some long biography',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- ],
- ],
- [
- 'user' => $this->makeUser('31b5316a-9b57-4b17-970a-315a4cbe73eb', 'K. Cheng', 'cheng@emca.com'),
- 'data' => [
- [
- 'name' => IAccountManager::PROPERTY_DISPLAYNAME,
- 'value' => 'K. Cheng',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_EMAIL,
- 'value' => 'cheng@emca.com',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_TWITTER,
- 'value' => '', '
- scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_FEDIVERSE,
- 'value' => '', '
- scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_PHONE,
- 'value' => '+71601212123',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_ADDRESS,
- 'value' => 'Pinapple Street 22',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_WEBSITE,
- 'value' => 'https://emca.com',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ORGANISATION,
- 'value' => 'Organisation A',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_ROLE,
- 'value' => 'Animal',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_HEADLINE,
- 'value' => 'My headline',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_BIOGRAPHY,
- 'value' => 'Short biography',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::COLLECTION_EMAIL,
- 'value' => 'k.cheng@emca.com',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::COLLECTION_EMAIL,
- 'value' => 'kai.cheng@emca.com',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- ],
- ],
- [
- 'user' => $this->makeUser('goodpal@elpmaxe.org', 'Goodpal, Kim', 'goodpal@elpmaxe.org'),
- 'data' => [
- [
- 'name' => IAccountManager::PROPERTY_DISPLAYNAME,
- 'value' => 'Goodpal, Kim',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_EMAIL,
- 'value' => 'goodpal@elpmaxe.org',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_TWITTER,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_FEDIVERSE,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL
- ],
- [
- 'name' => IAccountManager::PROPERTY_PHONE,
- 'value' => '+71602121231',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ADDRESS,
- 'value' => 'Octopus Ave 17',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_WEBSITE,
- 'value' => 'https://elpmaxe.org',
- 'scope' => IAccountManager::SCOPE_PUBLISHED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ORGANISATION,
- 'value' => 'Organisation B',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_ROLE,
- 'value' => 'Organism',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_HEADLINE,
- 'value' => 'Best headline',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_BIOGRAPHY,
- 'value' => 'Autobiography',
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- ],
- ],
- ];
- $this->config->expects($this->exactly(count($users)))->method('getSystemValue')->with('account_manager.default_property_scope', [])->willReturn([]);
- foreach ($users as $userInfo) {
- $this->invokePrivate($this->accountManager, 'updateUser', [$userInfo['user'], $userInfo['data'], null, false]);
- }
- }
- /**
- * get a instance of the accountManager
- *
- * @return MockObject | AccountManager
- */
- public function getInstance(?array $mockedMethods = null) {
- return $this->getMockBuilder(AccountManager::class)
- ->setConstructorArgs([
- $this->connection,
- $this->config,
- $this->eventDispatcher,
- $this->jobList,
- $this->logger,
- $this->verificationToken,
- $this->mailer,
- $this->defaults,
- $this->l10nFactory,
- $this->urlGenerator,
- $this->crypto,
- $this->phoneNumberUtil,
- ])
- ->onlyMethods($mockedMethods)
- ->getMock();
- }
- /**
- * @dataProvider dataTrueFalse
- *
- */
- public function testUpdateUser(array $newData, array $oldData, bool $insertNew, bool $updateExisting): void {
- $accountManager = $this->getInstance(['getUser', 'insertNewUser', 'updateExistingUser']);
- /** @var IUser $user */
- $user = $this->createMock(IUser::class);
- if ($updateExisting) {
- $accountManager->expects($this->once())->method('updateExistingUser')
- ->with($user, $newData);
- $accountManager->expects($this->never())->method('insertNewUser');
- }
- if ($insertNew) {
- $accountManager->expects($this->once())->method('insertNewUser')
- ->with($user, $newData);
- $accountManager->expects($this->never())->method('updateExistingUser');
- }
- if (!$insertNew && !$updateExisting) {
- $accountManager->expects($this->never())->method('updateExistingUser');
- $accountManager->expects($this->never())->method('insertNewUser');
- $this->eventDispatcher->expects($this->never())->method('dispatchTyped');
- } else {
- $this->eventDispatcher->expects($this->once())->method('dispatchTyped')
- ->willReturnCallback(
- function ($event) use ($user, $newData) {
- $this->assertInstanceOf(UserUpdatedEvent::class, $event);
- $this->assertSame($user, $event->getUser());
- $this->assertSame($newData, $event->getData());
- }
- );
- }
- $this->invokePrivate($accountManager, 'updateUser', [$user, $newData, $oldData]);
- }
- public function dataTrueFalse(): array {
- return [
- #$newData | $oldData | $insertNew | $updateExisting
- [['myProperty' => ['value' => 'newData']], ['myProperty' => ['value' => 'oldData']], false, true],
- [['myProperty' => ['value' => 'oldData']], ['myProperty' => ['value' => 'oldData']], false, false]
- ];
- }
- public function testAddMissingDefaults(): void {
- $user = $this->createMock(IUser::class);
- $this->config
- ->expects($this->once())
- ->method('getAppValue')
- ->with('settings', 'profile_enabled_by_default', '1')
- ->willReturn('1');
- $input = [
- [
- 'name' => IAccountManager::PROPERTY_DISPLAYNAME,
- 'value' => 'bob',
- 'verified' => IAccountManager::NOT_VERIFIED,
- ],
- [
- 'name' => IAccountManager::PROPERTY_EMAIL,
- 'value' => 'bob@bob.bob',
- ],
- ];
- $expected = [
- [
- 'name' => IAccountManager::PROPERTY_DISPLAYNAME,
- 'value' => 'bob',
- 'scope' => IAccountManager::SCOPE_FEDERATED,
- 'verified' => IAccountManager::NOT_VERIFIED,
- ],
- [
- 'name' => IAccountManager::PROPERTY_EMAIL,
- 'value' => 'bob@bob.bob',
- 'scope' => IAccountManager::SCOPE_FEDERATED,
- 'verified' => IAccountManager::NOT_VERIFIED,
- ],
- [
- 'name' => IAccountManager::PROPERTY_ADDRESS,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- 'verified' => IAccountManager::NOT_VERIFIED,
- ],
- [
- 'name' => IAccountManager::PROPERTY_WEBSITE,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- 'verified' => IAccountManager::NOT_VERIFIED,
- ],
- [
- 'name' => IAccountManager::PROPERTY_AVATAR,
- 'scope' => IAccountManager::SCOPE_FEDERATED
- ],
- [
- 'name' => IAccountManager::PROPERTY_PHONE,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- 'verified' => IAccountManager::NOT_VERIFIED,
- ],
- [
- 'name' => IAccountManager::PROPERTY_TWITTER,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- 'verified' => IAccountManager::NOT_VERIFIED,
- ],
- [
- 'name' => IAccountManager::PROPERTY_FEDIVERSE,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- 'verified' => IAccountManager::NOT_VERIFIED,
- ],
- [
- 'name' => IAccountManager::PROPERTY_ORGANISATION,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- ],
- [
- 'name' => IAccountManager::PROPERTY_ROLE,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- ],
- [
- 'name' => IAccountManager::PROPERTY_HEADLINE,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- ],
- [
- 'name' => IAccountManager::PROPERTY_BIOGRAPHY,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- ],
- [
- 'name' => IAccountManager::PROPERTY_BIRTHDATE,
- 'value' => '',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- ],
- [
- 'name' => IAccountManager::PROPERTY_PROFILE_ENABLED,
- 'value' => '1',
- ],
- ];
- $this->config->expects($this->once())->method('getSystemValue')->with('account_manager.default_property_scope', [])->willReturn([]);
- $defaultUserRecord = $this->invokePrivate($this->accountManager, 'buildDefaultUserRecord', [$user]);
- $result = $this->invokePrivate($this->accountManager, 'addMissingDefaultValues', [$input, $defaultUserRecord]);
- $this->assertSame($expected, $result);
- }
- public function testGetAccount(): void {
- $accountManager = $this->getInstance(['getUser']);
- /** @var IUser $user */
- $user = $this->createMock(IUser::class);
- $data = [
- [
- 'value' => '@twitterhandle',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- 'verified' => IAccountManager::NOT_VERIFIED,
- 'name' => IAccountManager::PROPERTY_TWITTER,
- ],
- [
- 'value' => '@mastohandle@mastodon.social',
- 'scope' => IAccountManager::SCOPE_LOCAL,
- 'verified' => IAccountManager::NOT_VERIFIED,
- 'name' => IAccountManager::PROPERTY_FEDIVERSE,
- ],
- [
- 'value' => 'test@example.com',
- 'scope' => IAccountManager::SCOPE_PUBLISHED,
- 'verified' => IAccountManager::VERIFICATION_IN_PROGRESS,
- 'name' => IAccountManager::PROPERTY_EMAIL,
- ],
- [
- 'value' => 'https://example.com',
- 'scope' => IAccountManager::SCOPE_FEDERATED,
- 'verified' => IAccountManager::VERIFIED,
- 'name' => IAccountManager::PROPERTY_WEBSITE,
- ],
- ];
- $expected = new Account($user);
- $expected->setProperty(IAccountManager::PROPERTY_TWITTER, '@twitterhandle', IAccountManager::SCOPE_LOCAL, IAccountManager::NOT_VERIFIED);
- $expected->setProperty(IAccountManager::PROPERTY_FEDIVERSE, '@mastohandle@mastodon.social', IAccountManager::SCOPE_LOCAL, IAccountManager::NOT_VERIFIED);
- $expected->setProperty(IAccountManager::PROPERTY_EMAIL, 'test@example.com', IAccountManager::SCOPE_PUBLISHED, IAccountManager::VERIFICATION_IN_PROGRESS);
- $expected->setProperty(IAccountManager::PROPERTY_WEBSITE, 'https://example.com', IAccountManager::SCOPE_FEDERATED, IAccountManager::VERIFIED);
- $accountManager->expects($this->once())
- ->method('getUser')
- ->willReturn($data);
- $this->assertEquals($expected, $accountManager->getAccount($user));
- }
- public function dataParsePhoneNumber(): array {
- return [
- ['0711 / 25 24 28-90', 'DE', '+4971125242890'],
- ['0711 / 25 24 28-90', '', null],
- ['+49 711 / 25 24 28-90', '', '+4971125242890'],
- ];
- }
- /**
- * @dataProvider dataParsePhoneNumber
- */
- public function testParsePhoneNumber(string $phoneInput, string $defaultRegion, ?string $phoneNumber): void {
- $this->config->method('getSystemValueString')
- ->willReturn($defaultRegion);
- if ($phoneNumber === null) {
- $this->expectException(\InvalidArgumentException::class);
- self::invokePrivate($this->accountManager, 'parsePhoneNumber', [$phoneInput]);
- } else {
- self::assertEquals($phoneNumber, self::invokePrivate($this->accountManager, 'parsePhoneNumber', [$phoneInput]));
- }
- }
- public function dataParseWebsite(): array {
- return [
- ['https://nextcloud.com', 'https://nextcloud.com'],
- ['http://nextcloud.com', 'http://nextcloud.com'],
- ['ftp://nextcloud.com', null],
- ['//nextcloud.com/', null],
- ['https:///?query', null],
- ];
- }
- /**
- * @dataProvider dataParseWebsite
- * @param string $websiteInput
- * @param string|null $websiteOutput
- */
- public function testParseWebsite(string $websiteInput, ?string $websiteOutput): void {
- if ($websiteOutput === null) {
- $this->expectException(\InvalidArgumentException::class);
- self::invokePrivate($this->accountManager, 'parseWebsite', [$websiteInput]);
- } else {
- self::assertEquals($websiteOutput, self::invokePrivate($this->accountManager, 'parseWebsite', [$websiteInput]));
- }
- }
- /**
- * @dataProvider searchDataProvider
- */
- public function testSearchUsers(string $property, array $values, array $expected): void {
- $this->populateOrUpdate();
- $matchedUsers = $this->accountManager->searchUsers($property, $values);
- foreach ($expected as $expectedEntry) {
- $this->assertContains($expectedEntry, $matchedUsers);
- }
- if (empty($expected)) {
- $this->assertEmpty($matchedUsers);
- }
- }
- public function searchDataProvider(): array {
- return [
- [ #0 Search for an existing name
- IAccountManager::PROPERTY_DISPLAYNAME,
- ['Jane Doe'],
- ['Jane Doe' => 'j.doe']
- ],
- [ #1 Search for part of a name (no result)
- IAccountManager::PROPERTY_DISPLAYNAME,
- ['Jane'],
- []
- ],
- [ #2 Search for part of a name (no result, test wildcard)
- IAccountManager::PROPERTY_DISPLAYNAME,
- ['Jane%'],
- []
- ],
- [ #3 Search for phone
- IAccountManager::PROPERTY_PHONE,
- ['+491603121212'],
- ['+491603121212' => 'b32c5a5b-1084-4380-8856-e5223b16de9f'],
- ],
- [ #4 Search for twitter handles
- IAccountManager::PROPERTY_TWITTER,
- ['@sometwitter', '@a_alice', '@unseen'],
- ['@sometwitter' => 'j.doe', '@a_alice' => 'a.allison'],
- ],
- [ #5 Search for email
- IAccountManager::PROPERTY_EMAIL,
- ['cheng@emca.com'],
- ['cheng@emca.com' => '31b5316a-9b57-4b17-970a-315a4cbe73eb'],
- ],
- [ #6 Search for email by additional email
- IAccountManager::PROPERTY_EMAIL,
- ['kai.cheng@emca.com'],
- ['kai.cheng@emca.com' => '31b5316a-9b57-4b17-970a-315a4cbe73eb'],
- ],
- [ #7 Search for additional email
- IAccountManager::COLLECTION_EMAIL,
- ['kai.cheng@emca.com', 'cheng@emca.com'],
- ['kai.cheng@emca.com' => '31b5316a-9b57-4b17-970a-315a4cbe73eb'],
- ],
- [ #8 Search for email by additional email (two valid search values, but the same user)
- IAccountManager::PROPERTY_EMAIL,
- ['kai.cheng@emca.com', 'cheng@emca.com'],
- [
- 'kai.cheng@emca.com' => '31b5316a-9b57-4b17-970a-315a4cbe73eb',
- ],
- ],
- ];
- }
- public function dataCheckEmailVerification(): array {
- return [
- [$this->makeUser('steve', 'Steve Smith', 'steve@steve.steve'), null],
- [$this->makeUser('emma', 'Emma Morales', 'emma@emma.com'), 'emma@morales.com'],
- [$this->makeUser('sarah@web.org', 'Sarah Foster', 'sarah@web.org'), null],
- [$this->makeUser('cole@web.org', 'Cole Harrison', 'cole@web.org'), 'cole@example.com'],
- [$this->makeUser('8d29e358-cf69-4849-bbf9-28076c0b908b', 'Alice McPherson', 'alice@example.com'), 'alice@mcpherson.com'],
- [$this->makeUser('11da2744-3f4d-4c17-8c13-4c057a379237', 'James Loranger', 'james@example.com'), ''],
- ];
- }
- /**
- * @dataProvider dataCheckEmailVerification
- */
- public function testCheckEmailVerification(IUser $user, ?string $newEmail): void {
- // Once because of getAccount, once because of getUser
- $this->config->expects($this->exactly(2))->method('getSystemValue')->with('account_manager.default_property_scope', [])->willReturn([]);
- $account = $this->accountManager->getAccount($user);
- $emailUpdated = false;
- if (!empty($newEmail)) {
- $account->getProperty(IAccountManager::PROPERTY_EMAIL)->setValue($newEmail);
- $emailUpdated = true;
- }
- if ($emailUpdated) {
- $this->jobList->expects($this->once())
- ->method('add')
- ->with(VerifyUserData::class);
- } else {
- $this->jobList->expects($this->never())
- ->method('add')
- ->with(VerifyUserData::class);
- }
- /** @var array $oldData */
- $oldData = $this->invokePrivate($this->accountManager, 'getUser', [$user, false]);
- $this->invokePrivate($this->accountManager, 'checkEmailVerification', [$account, $oldData]);
- }
- public function dataSetDefaultPropertyScopes(): array {
- return [
- [
- [],
- [
- IAccountManager::PROPERTY_DISPLAYNAME => IAccountManager::SCOPE_FEDERATED,
- IAccountManager::PROPERTY_ADDRESS => IAccountManager::SCOPE_LOCAL,
- IAccountManager::PROPERTY_EMAIL => IAccountManager::SCOPE_FEDERATED,
- IAccountManager::PROPERTY_ROLE => IAccountManager::SCOPE_LOCAL,
- ]
- ],
- [
- [
- IAccountManager::PROPERTY_DISPLAYNAME => IAccountManager::SCOPE_FEDERATED,
- IAccountManager::PROPERTY_EMAIL => IAccountManager::SCOPE_LOCAL,
- IAccountManager::PROPERTY_ROLE => IAccountManager::SCOPE_PRIVATE,
- ], [
- IAccountManager::PROPERTY_DISPLAYNAME => IAccountManager::SCOPE_FEDERATED,
- IAccountManager::PROPERTY_EMAIL => IAccountManager::SCOPE_LOCAL,
- IAccountManager::PROPERTY_ROLE => IAccountManager::SCOPE_PRIVATE,
- ]
- ],
- [
- [
- IAccountManager::PROPERTY_ADDRESS => 'invalid scope',
- 'invalid property' => IAccountManager::SCOPE_LOCAL,
- IAccountManager::PROPERTY_ROLE => IAccountManager::SCOPE_PRIVATE,
- ],
- [
- IAccountManager::PROPERTY_ADDRESS => IAccountManager::SCOPE_LOCAL,
- IAccountManager::PROPERTY_EMAIL => IAccountManager::SCOPE_FEDERATED,
- IAccountManager::PROPERTY_ROLE => IAccountManager::SCOPE_PRIVATE,
- ]
- ],
- ];
- }
- /**
- * @dataProvider dataSetDefaultPropertyScopes
- */
- public function testSetDefaultPropertyScopes(array $propertyScopes, array $expectedResultScopes): void {
- $user = $this->makeUser('steve', 'Steve Smith', 'steve@steve.steve');
- $this->config->expects($this->once())->method('getSystemValue')->with('account_manager.default_property_scope', [])->willReturn($propertyScopes);
- $result = $this->invokePrivate($this->accountManager, 'buildDefaultUserRecord', [$user]);
- $resultProperties = array_column($result, 'name');
- $this->assertEmpty(array_diff($resultProperties, IAccountManager::ALLOWED_PROPERTIES), 'Building default user record returned non-allowed properties');
- foreach ($expectedResultScopes as $expectedResultScopeKey => $expectedResultScopeValue) {
- $resultScope = $result[array_search($expectedResultScopeKey, $resultProperties)]['scope'];
- $this->assertEquals($expectedResultScopeValue, $resultScope, "The result scope doesn't follow the value set into the config or defaults correctly.");
- }
- }
- }
|