From 36bf522c8b06517fbc425305cd7f422fe1c18a5c Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Tue, 2 Oct 2018 18:42:06 +0200 Subject: Add job to check users for backup code reminders Now that we can enforce 2FA we also should notify users that have enabled 2FA but do not yet have backup codes generated. This adds a repair step that inserts a background job to do this. Signed-off-by: Roeland Jago Douma --- .../Unit/BackgroundJob/CheckBackupCodeTest.php | 133 +++++++++++++++++++++ .../tests/Unit/Migration/CheckBackupCodeTest.php | 65 ++++++++++ 2 files changed, 198 insertions(+) create mode 100644 apps/twofactor_backupcodes/tests/Unit/BackgroundJob/CheckBackupCodeTest.php create mode 100644 apps/twofactor_backupcodes/tests/Unit/Migration/CheckBackupCodeTest.php (limited to 'apps/twofactor_backupcodes/tests') diff --git a/apps/twofactor_backupcodes/tests/Unit/BackgroundJob/CheckBackupCodeTest.php b/apps/twofactor_backupcodes/tests/Unit/BackgroundJob/CheckBackupCodeTest.php new file mode 100644 index 00000000000..ad1ce71545d --- /dev/null +++ b/apps/twofactor_backupcodes/tests/Unit/BackgroundJob/CheckBackupCodeTest.php @@ -0,0 +1,133 @@ + + * + * @author Roeland Jago Douma + * + * @license GNU AGPL version 3 or any later version + * + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with this program. If not, see . + * + */ + +namespace OCA\TwoFactorBackupCodes\Tests\Unit\BackgroundJob; + +use OC\Authentication\TwoFactorAuth\Manager; +use OCA\TwoFactorBackupCodes\BackgroundJob\CheckBackupCodes; +use OCA\TwoFactorBackupCodes\BackgroundJob\RememberBackupCodesJob; +use OCP\Authentication\TwoFactorAuth\IRegistry; +use OCP\BackgroundJob\IJobList; +use OCP\IUser; +use OCP\IUserManager; +use PHPUnit\Framework\MockObject\MockObject; +use Test\TestCase; + +class CheckBackupCodeTest extends TestCase { + + /** @var IUserManager|MockObject */ + private $userManager; + + /** @var IJobList|MockObject */ + private $jobList; + + /** @var IRegistry|MockObject */ + private $registry; + + /** @var Manager|MockObject */ + private $manager; + + /** @var IUser|MockObject */ + private $user; + + /** @var CheckBackupCodes */ + private $checkBackupCodes; + + public function setUp() { + parent::setUp(); + + $this->userManager = $this->createMock(IUserManager::class); + $this->jobList = $this->createMock(IJobList::class); + $this->registry = $this->createMock(IRegistry::class); + $this->manager = $this->createMock(Manager::class); + + $this->user = $this->createMock(IUser::class); + + $this->userManager->method('callForSeenUsers') + ->will($this->returnCallback(function(\Closure $e) { + $e($this->user); + })); + + $this->checkBackupCodes = new CheckBackupCodes( + $this->userManager, + $this->jobList, + $this->manager, + $this->registry + ); + } + + public function testRunAlreadyGenerated() { + $this->registry->method('getProviderStates') + ->with($this->user) + ->willReturn(['backup_codes' => true]); + $this->manager->method('isTwoFactorAuthenticated') + ->with($this->user) + ->willReturn(true); + $this->jobList->expects($this->never()) + ->method($this->anything()); + + $this->invokePrivate($this->checkBackupCodes, 'run', [[]]); + } + + public function testRun() { + $this->user->method('getUID') + ->willReturn('myUID'); + + $this->registry->expects($this->once()) + ->method('getProviderStates') + ->with($this->user) + ->willReturn([ + 'backup_codes' => false, + ]); + $this->jobList->expects($this->once()) + ->method('add') + ->with( + $this->equalTo(RememberBackupCodesJob::class), + ['uid' => 'myUID'] + ); + $this->manager->method('isTwoFactorAuthenticated') + ->with($this->user) + ->willReturn(true); + + $this->invokePrivate($this->checkBackupCodes, 'run', [[]]); + } + + public function testRunNoProviders() { + $this->registry->expects($this->once()) + ->method('getProviderStates') + ->with($this->user) + ->willReturn([ + 'backup_codes' => false, + ]); + $this->jobList->expects($this->never()) + ->method($this->anything()); + $this->manager->method('isTwoFactorAuthenticated') + ->with($this->user) + ->willReturn(false); + + $this->invokePrivate($this->checkBackupCodes, 'run', [[]]); + } + + +} diff --git a/apps/twofactor_backupcodes/tests/Unit/Migration/CheckBackupCodeTest.php b/apps/twofactor_backupcodes/tests/Unit/Migration/CheckBackupCodeTest.php new file mode 100644 index 00000000000..6644fce7293 --- /dev/null +++ b/apps/twofactor_backupcodes/tests/Unit/Migration/CheckBackupCodeTest.php @@ -0,0 +1,65 @@ + + * + * @license GNU AGPL version 3 or any later version + * + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with this program. If not, see . + * + */ + +namespace OCA\TwoFactorBackupCodes\Tests\Unit\Migration; + +use OCA\TwoFactorBackupCodes\Event\CodesGenerated; +use OCA\TwoFactorBackupCodes\Listener\RegistryUpdater; +use OCA\TwoFactorBackupCodes\Migration\CheckBackupCodes; +use OCA\TwoFactorBackupCodes\Provider\BackupCodesProvider; +use OCP\Authentication\TwoFactorAuth\IRegistry; +use OCP\BackgroundJob\IJobList; +use OCP\IUser; +use OCP\Migration\IOutput; +use Symfony\Component\EventDispatcher\Event; +use Test\TestCase; + +class CheckBackupCodeTest extends TestCase { + + /** @var IJobList|\PHPunit\Framework\MockObject\MockObject */ + private $jobList; + + /** @var CheckBackupCodes */ + private $checkBackupsCodes; + + protected function setUp() { + parent::setUp(); + + $this->jobList = $this->createMock(IJobList::class); + $this->checkBackupsCodes = new CheckBackupCodes($this->jobList); + } + + public function testGetName() { + $this->assertSame('Add background job to check for backup codes', $this->checkBackupsCodes->getName()); + } + + public function testRun() { + $this->jobList->expects($this->once()) + ->method('add') + ->with( + $this->equalTo(\OCA\TwoFactorBackupCodes\BackgroundJob\CheckBackupCodes::class) + ); + + $this->checkBackupsCodes->run($this->createMock(IOutput::class)); + } +} -- cgit v1.2.3