1 files changed, 32 insertions, 31 deletions

diff --git a/apps/twofactor_backupcodes/tests/Unit/Provider/BackupCodesProviderTest.php b/apps/twofactor_backupcodes/tests/Unit/Provider/BackupCodesProviderTest.php
index f64f5735d77..512374fca8c 100644
--- a/apps/twofactor_backupcodes/tests/Unit/Provider/BackupCodesProviderTest.php
+++ b/apps/twofactor_backupcodes/tests/Unit/Provider/BackupCodesProviderTest.php
@@ -11,49 +11,50 @@ namespace OCA\TwoFactorBackupCodes\Tests\Unit\Provider;
 use OC\App\AppManager;
 use OCA\TwoFactorBackupCodes\Provider\BackupCodesProvider;
 use OCA\TwoFactorBackupCodes\Service\BackupCodeStorage;
-use OCP\IInitialStateService;
+use OCP\AppFramework\Services\IInitialState;
 use OCP\IL10N;
 use OCP\IUser;
-use OCP\Template;
+use OCP\Server;
+use OCP\Template\ITemplateManager;
+use PHPUnit\Framework\MockObject\MockObject;
 use Test\TestCase;
 
 class BackupCodesProviderTest extends TestCase {
+	private string $appName;
 
-	/** @var string */
-	private $appName;
+	private BackupCodeStorage&MockObject $storage;
+	private IL10N&MockObject $l10n;
+	private AppManager&MockObject $appManager;
+	private IInitialState&MockObject $initialState;
 
-	/** @var BackupCodeStorage|\PHPUnit\Framework\MockObject\MockObject */
-	private $storage;
-
-	/** @var IL10N|\PHPUnit\Framework\MockObject\MockObject */
-	private $l10n;
-
-	/** @var AppManager|\PHPUnit\Framework\MockObject\MockObject */
-	private $appManager;
-
-	/** @var IInitialStateService|\PHPUnit\Framework\MockObject\MockObject */
-	private $initialState;
-
-	/** @var BackupCodesProvider */
-	private $provider;
+	private ITemplateManager $templateManager;
+	private BackupCodesProvider $provider;
 
 	protected function setUp(): void {
 		parent::setUp();
 
-		$this->appName = "twofactor_backupcodes";
+		$this->appName = 'twofactor_backupcodes';
 		$this->storage = $this->createMock(BackupCodeStorage::class);
 		$this->l10n = $this->createMock(IL10N::class);
 		$this->appManager = $this->createMock(AppManager::class);
-		$this->initialState = $this->createMock(IInitialStateService::class);
-
-		$this->provider = new BackupCodesProvider($this->appName, $this->storage, $this->l10n, $this->appManager, $this->initialState);
+		$this->initialState = $this->createMock(IInitialState::class);
+		$this->templateManager = Server::get(ITemplateManager::class);
+
+		$this->provider = new BackupCodesProvider(
+			$this->appName,
+			$this->storage,
+			$this->l10n,
+			$this->appManager,
+			$this->initialState,
+			$this->templateManager,
+		);
 	}
 
-	public function testGetId() {
+	public function testGetId(): void {
 		$this->assertEquals('backup_codes', $this->provider->getId());
 	}
 
-	public function testGetDisplayName() {
+	public function testGetDisplayName(): void {
 		$this->l10n->expects($this->once())
 			->method('t')
 			->with('Backup code')
@@ -61,7 +62,7 @@ class BackupCodesProviderTest extends TestCase {
 		$this->assertSame('l10n backup code', $this->provider->getDisplayName());
 	}
 
-	public function testGetDescription() {
+	public function testGetDescription(): void {
 		$this->l10n->expects($this->once())
 			->method('t')
 			->with('Use backup code')
@@ -69,14 +70,14 @@ class BackupCodesProviderTest extends TestCase {
 		$this->assertSame('l10n use backup code', $this->provider->getDescription());
 	}
 
-	public function testGetTempalte() {
+	public function testGetTempalte(): void {
 		$user = $this->getMockBuilder(IUser::class)->getMock();
-		$expected = new Template('twofactor_backupcodes', 'challenge');
+		$expected = $this->templateManager->getTemplate('twofactor_backupcodes', 'challenge');
 
 		$this->assertEquals($expected, $this->provider->getTemplate($user));
 	}
 
-	public function testVerfiyChallenge() {
+	public function testVerfiyChallenge(): void {
 		$user = $this->getMockBuilder(IUser::class)->getMock();
 		$challenge = 'xyz';
 
@@ -88,7 +89,7 @@ class BackupCodesProviderTest extends TestCase {
 		$this->assertFalse($this->provider->verifyChallenge($user, $challenge));
 	}
 
-	public function testIsTwoFactorEnabledForUser() {
+	public function testIsTwoFactorEnabledForUser(): void {
 		$user = $this->getMockBuilder(IUser::class)->getMock();
 
 		$this->storage->expects($this->once())
@@ -99,7 +100,7 @@ class BackupCodesProviderTest extends TestCase {
 		$this->assertTrue($this->provider->isTwoFactorAuthEnabledForUser($user));
 	}
 
-	public function testIsActiveNoProviders() {
+	public function testIsActiveNoProviders(): void {
 		$user = $this->getMockBuilder(IUser::class)->getMock();
 
 		$this->appManager->expects($this->once())
@@ -119,7 +120,7 @@ class BackupCodesProviderTest extends TestCase {
 		$this->assertFalse($this->provider->isActive($user));
 	}
 
-	public function testIsActiveWithProviders() {
+	public function testIsActiveWithProviders(): void {
 		$user = $this->getMockBuilder(IUser::class)->getMock();
 
 		$this->appManager->expects($this->once())