diff options
author | Christopher Ng <chrng8@gmail.com> | 2022-01-05 07:08:36 +0300 |
---|---|---|
committer | backportbot[bot] <backportbot[bot]@users.noreply.github.com> | 2022-01-07 13:52:36 +0300 |
commit | c699f1bec2d1bee62fd141c91b6dab8ae0b7754c (patch) | |
tree | 069719ff59d2c4441f48b70842914b573e711c4d /tests/lib | |
parent | deeee361327e523a07da2298eb44c88051fa87c7 (diff) |
Fix email verification
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Diffstat (limited to 'tests/lib')
-rw-r--r-- | tests/lib/Accounts/AccountManagerTest.php | 39 |
1 files changed, 39 insertions, 0 deletions
diff --git a/tests/lib/Accounts/AccountManagerTest.php b/tests/lib/Accounts/AccountManagerTest.php index 8bb60a1a516..18419d73423 100644 --- a/tests/lib/Accounts/AccountManagerTest.php +++ b/tests/lib/Accounts/AccountManagerTest.php @@ -24,6 +24,7 @@ namespace Test\Accounts; use OC\Accounts\Account; use OC\Accounts\AccountManager; +use OCA\Settings\BackgroundJobs\VerifyUserData; use OCP\Accounts\IAccountManager; use OCP\BackgroundJob\IJobList; use OCP\Defaults; @@ -770,4 +771,42 @@ class AccountManagerTest extends TestCase { ], ]; } + + 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 { + $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]); + } } |