1 files changed, 14 insertions, 23 deletions

diff --git a/apps/twofactor_backupcodes/tests/Unit/Service/BackupCodeStorageTest.php b/apps/twofactor_backupcodes/tests/Unit/Service/BackupCodeStorageTest.php
index 7725102db88..069e50b71fd 100644
--- a/apps/twofactor_backupcodes/tests/Unit/Service/BackupCodeStorageTest.php
+++ b/apps/twofactor_backupcodes/tests/Unit/Service/BackupCodeStorageTest.php
@@ -16,24 +16,15 @@ use OCP\EventDispatcher\IEventDispatcher;
 use OCP\IUser;
 use OCP\Security\IHasher;
 use OCP\Security\ISecureRandom;
+use PHPUnit\Framework\MockObject\MockObject;
 use Test\TestCase;
 
 class BackupCodeStorageTest extends TestCase {
-
-	/** @var BackupCodeMapper|\PHPUnit\Framework\MockObject\MockObject */
-	private $mapper;
-
-	/** @var ISecureRandom|\PHPUnit\Framework\MockObject\MockObject */
-	private $random;
-
-	/** @var IHasher|\PHPUnit\Framework\MockObject\MockObject */
-	private $hasher;
-
-	/** @var IEventDispatcher|\PHPUnit\Framework\MockObject\MockObject */
-	private $eventDispatcher;
-
-	/** @var BackupCodeStorage */
-	private $storage;
+	private BackupCodeMapper&MockObject $mapper;
+	private ISecureRandom&MockObject $random;
+	private IHasher&MockObject $hasher;
+	private IEventDispatcher&MockObject $eventDispatcher;
+	private BackupCodeStorage $storage;
 
 	protected function setUp(): void {
 		parent::setUp();
@@ -46,7 +37,7 @@ class BackupCodeStorageTest extends TestCase {
 		$this->storage = new BackupCodeStorage($this->mapper, $this->random, $this->hasher, $this->eventDispatcher);
 	}
 
-	public function testCreateCodes() {
+	public function testCreateCodes(): void {
 		$user = $this->createMock(IUser::class);
 		$number = 5;
 		$user->method('getUID')->willReturn('fritz');
@@ -78,7 +69,7 @@ class BackupCodeStorageTest extends TestCase {
 		}
 	}
 
-	public function testHasBackupCodes() {
+	public function testHasBackupCodes(): void {
 		$user = $this->createMock(IUser::class);
 		$codes = [
 			new BackupCode(),
@@ -93,7 +84,7 @@ class BackupCodeStorageTest extends TestCase {
 		$this->assertTrue($this->storage->hasBackupCodes($user));
 	}
 
-	public function testHasBackupCodesNoCodes() {
+	public function testHasBackupCodesNoCodes(): void {
 		$user = $this->createMock(IUser::class);
 		$codes = [];
 
@@ -105,7 +96,7 @@ class BackupCodeStorageTest extends TestCase {
 		$this->assertFalse($this->storage->hasBackupCodes($user));
 	}
 
-	public function testGetBackupCodeState() {
+	public function testGetBackupCodeState(): void {
 		$user = $this->createMock(IUser::class);
 
 		$code1 = new BackupCode();
@@ -130,7 +121,7 @@ class BackupCodeStorageTest extends TestCase {
 		$this->assertEquals($expected, $this->storage->getBackupCodesState($user));
 	}
 
-	public function testGetBackupCodeDisabled() {
+	public function testGetBackupCodeDisabled(): void {
 		$user = $this->createMock(IUser::class);
 
 		$codes = [];
@@ -148,7 +139,7 @@ class BackupCodeStorageTest extends TestCase {
 		$this->assertEquals($expected, $this->storage->getBackupCodesState($user));
 	}
 
-	public function testValidateCode() {
+	public function testValidateCode(): void {
 		$user = $this->createMock(IUser::class);
 		$code = new BackupCode();
 		$code->setUsed(0);
@@ -174,7 +165,7 @@ class BackupCodeStorageTest extends TestCase {
 		$this->assertEquals(1, $code->getUsed());
 	}
 
-	public function testValidateUsedCode() {
+	public function testValidateUsedCode(): void {
 		$user = $this->createMock(IUser::class);
 		$code = new BackupCode();
 		$code->setUsed('1');
@@ -195,7 +186,7 @@ class BackupCodeStorageTest extends TestCase {
 		$this->assertFalse($this->storage->validateCode($user, 'CHALLENGE'));
 	}
 
-	public function testValidateCodeWithWrongHash() {
+	public function testValidateCodeWithWrongHash(): void {
 		$user = $this->createMock(IUser::class);
 		$code = new BackupCode();
 		$code->setUsed(0);