1 files changed, 15 insertions, 21 deletions

diff --git a/tests/lib/Security/RateLimiting/Backend/MemoryCacheBackendTest.php b/tests/lib/Security/RateLimiting/Backend/MemoryCacheBackendTest.php
index e7cb4a93a29..24e3ab1a209 100644
--- a/tests/lib/Security/RateLimiting/Backend/MemoryCacheBackendTest.php
+++ b/tests/lib/Security/RateLimiting/Backend/MemoryCacheBackendTest.php
@@ -3,23 +3,8 @@
 declare(strict_types=1);
 
 /**
- * @copyright Copyright (c) 2017 Lukas Reschke <lukas@statuscode.ch>
- *
- * @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 <http://www.gnu.org/licenses/>.
- *
+ * SPDX-FileCopyrightText: 2017 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
 namespace Test\Security\RateLimiting\Backend;
@@ -28,9 +13,12 @@ use OC\Security\RateLimiting\Backend\MemoryCacheBackend;
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\ICache;
 use OCP\ICacheFactory;
+use OCP\IConfig;
 use Test\TestCase;
 
 class MemoryCacheBackendTest extends TestCase {
+	/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject */
+	private $config;
 	/** @var ICacheFactory|\PHPUnit\Framework\MockObject\MockObject */
 	private $cacheFactory;
 	/** @var ITimeFactory|\PHPUnit\Framework\MockObject\MockObject */
@@ -43,6 +31,7 @@ class MemoryCacheBackendTest extends TestCase {
 	protected function setUp(): void {
 		parent::setUp();
 
+		$this->config = $this->createMock(IConfig::class);
 		$this->cacheFactory = $this->createMock(ICacheFactory::class);
 		$this->timeFactory = $this->createMock(ITimeFactory::class);
 		$this->cache = $this->createMock(ICache::class);
@@ -53,13 +42,18 @@ class MemoryCacheBackendTest extends TestCase {
 			->with('OC\Security\RateLimiting\Backend\MemoryCacheBackend')
 			->willReturn($this->cache);
 
+		$this->config->method('getSystemValueBool')
+			->with('ratelimit.protection.enabled')
+			->willReturn(true);
+
 		$this->memoryCache = new MemoryCacheBackend(
+			$this->config,
 			$this->cacheFactory,
 			$this->timeFactory
 		);
 	}
 
-	public function testGetAttemptsWithNoAttemptsBefore() {
+	public function testGetAttemptsWithNoAttemptsBefore(): void {
 		$this->cache
 			->expects($this->once())
 			->method('get')
@@ -69,7 +63,7 @@ class MemoryCacheBackendTest extends TestCase {
 		$this->assertSame(0, $this->memoryCache->getAttempts('Method', 'User'));
 	}
 
-	public function testGetAttempts() {
+	public function testGetAttempts(): void {
 		$this->timeFactory
 			->expects($this->once())
 			->method('getTime')
@@ -90,7 +84,7 @@ class MemoryCacheBackendTest extends TestCase {
 		$this->assertSame(3, $this->memoryCache->getAttempts('Method', 'User'));
 	}
 
-	public function testRegisterAttemptWithNoAttemptsBefore() {
+	public function testRegisterAttemptWithNoAttemptsBefore(): void {
 		$this->timeFactory
 			->expects($this->once())
 			->method('getTime')
@@ -112,7 +106,7 @@ class MemoryCacheBackendTest extends TestCase {
 		$this->memoryCache->registerAttempt('Method', 'User', 100);
 	}
 
-	public function testRegisterAttempt() {
+	public function testRegisterAttempt(): void {
 		$this->timeFactory
 			->expects($this->once())
 			->method('getTime')