nextcloud / server

tests/lib/Security/RateLimiting/Backend/MemoryCacheBackendTest.php

Indentation +123 added lines, -123 removed lines

@@ -17,127 +17,127 @@
 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 */
-	private $timeFactory;
-	/** @var ICache|\PHPUnit\Framework\MockObject\MockObject */
-	private $cache;
-	/** @var MemoryCacheBackend */
-	private $memoryCache;
-
-	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);
-
-		$this->cacheFactory
-			->expects($this->once())
-			->method('createDistributed')
-			->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(): void {
-		$this->cache
-			->expects($this->once())
-			->method('get')
-			->with('eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b')
-			->willReturn(null);
-
-		$this->assertSame(0, $this->memoryCache->getAttempts('Method', 'User'));
-	}
-
-	public function testGetAttempts(): void {
-		$this->timeFactory
-			->expects($this->once())
-			->method('getTime')
-			->willReturn(210);
-		$this->cache
-			->expects($this->once())
-			->method('get')
-			->with('eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b')
-			->willReturn(json_encode([
-				'1',
-				'2',
-				'87',
-				'223',
-				'223',
-				'224',
-			]));
-
-		$this->assertSame(3, $this->memoryCache->getAttempts('Method', 'User'));
-	}
-
-	public function testRegisterAttemptWithNoAttemptsBefore(): void {
-		$this->timeFactory
-			->expects($this->once())
-			->method('getTime')
-			->willReturn(123);
-
-		$this->cache
-			->expects($this->once())
-			->method('get')
-			->with('eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b')
-			->willReturn(null);
-		$this->cache
-			->expects($this->once())
-			->method('set')
-			->with(
-				'eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b',
-				json_encode(['223'])
-			);
-
-		$this->memoryCache->registerAttempt('Method', 'User', 100);
-	}
-
-	public function testRegisterAttempt(): void {
-		$this->timeFactory
-			->expects($this->once())
-			->method('getTime')
-			->willReturn(86);
-
-		$this->cache
-			->expects($this->once())
-			->method('get')
-			->with('eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b')
-			->willReturn(json_encode([
-				'1',
-				'2',
-				'87',
-				'123',
-				'123',
-				'124',
-			]));
-		$this->cache
-			->expects($this->once())
-			->method('set')
-			->with(
-				'eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b',
-				json_encode([
-					'87',
-					'123',
-					'123',
-					'124',
-					'186',
-				])
-			);
-
-		$this->memoryCache->registerAttempt('Method', 'User', 100);
-	}
+    /** @var IConfig|\PHPUnit\Framework\MockObject\MockObject */
+    private $config;
+    /** @var ICacheFactory|\PHPUnit\Framework\MockObject\MockObject */
+    private $cacheFactory;
+    /** @var ITimeFactory|\PHPUnit\Framework\MockObject\MockObject */
+    private $timeFactory;
+    /** @var ICache|\PHPUnit\Framework\MockObject\MockObject */
+    private $cache;
+    /** @var MemoryCacheBackend */
+    private $memoryCache;
+
+    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);
+
+        $this->cacheFactory
+            ->expects($this->once())
+            ->method('createDistributed')
+            ->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(): void {
+        $this->cache
+            ->expects($this->once())
+            ->method('get')
+            ->with('eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b')
+            ->willReturn(null);
+
+        $this->assertSame(0, $this->memoryCache->getAttempts('Method', 'User'));
+    }
+
+    public function testGetAttempts(): void {
+        $this->timeFactory
+            ->expects($this->once())
+            ->method('getTime')
+            ->willReturn(210);
+        $this->cache
+            ->expects($this->once())
+            ->method('get')
+            ->with('eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b')
+            ->willReturn(json_encode([
+                '1',
+                '2',
+                '87',
+                '223',
+                '223',
+                '224',
+            ]));
+
+        $this->assertSame(3, $this->memoryCache->getAttempts('Method', 'User'));
+    }
+
+    public function testRegisterAttemptWithNoAttemptsBefore(): void {
+        $this->timeFactory
+            ->expects($this->once())
+            ->method('getTime')
+            ->willReturn(123);
+
+        $this->cache
+            ->expects($this->once())
+            ->method('get')
+            ->with('eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b')
+            ->willReturn(null);
+        $this->cache
+            ->expects($this->once())
+            ->method('set')
+            ->with(
+                'eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b',
+                json_encode(['223'])
+            );
+
+        $this->memoryCache->registerAttempt('Method', 'User', 100);
+    }
+
+    public function testRegisterAttempt(): void {
+        $this->timeFactory
+            ->expects($this->once())
+            ->method('getTime')
+            ->willReturn(86);
+
+        $this->cache
+            ->expects($this->once())
+            ->method('get')
+            ->with('eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b')
+            ->willReturn(json_encode([
+                '1',
+                '2',
+                '87',
+                '123',
+                '123',
+                '124',
+            ]));
+        $this->cache
+            ->expects($this->once())
+            ->method('set')
+            ->with(
+                'eea460b8d756885099c7f0a4c083bf6a745069ee4a301984e726df58fd4510bffa2dac4b7fd5d835726a6753ffa8343ba31c7e902bbef78fc68c2e743667cb4b',
+                json_encode([
+                    '87',
+                    '123',
+                    '123',
+                    '124',
+                    '186',
+                ])
+            );
+
+        $this->memoryCache->registerAttempt('Method', 'User', 100);
+    }
 }

tests/lib/EventSourceFactoryTest.php

Indentation +6 added lines, -6 removed lines

@@ -11,11 +11,11 @@
 use OCP\IRequest;
 
 class EventSourceFactoryTest extends TestCase {
-	public function testCreate(): void {
-		$request = $this->createMock(IRequest::class);
-		$factory = new EventSourceFactory($request);
+    public function testCreate(): void {
+        $request = $this->createMock(IRequest::class);
+        $factory = new EventSourceFactory($request);
 
-		$instance = $factory->create();
-		$this->assertInstanceOf(IEventSource::class, $instance);
-	}
+        $instance = $factory->create();
+        $this->assertInstanceOf(IEventSource::class, $instance);
+    }
 }

tests/lib/Template/JSResourceLocatorTest.php

Indentation +162 added lines, -162 removed lines

@@ -17,166 +17,166 @@
 use Psr\Log\LoggerInterface;
 
 class JSResourceLocatorTest extends \Test\TestCase {
-	/** @var IAppData|\PHPUnit\Framework\MockObject\MockObject */
-	protected $appData;
-	/** @var IURLGenerator|\PHPUnit\Framework\MockObject\MockObject */
-	protected $urlGenerator;
-	/** @var SystemConfig|\PHPUnit\Framework\MockObject\MockObject */
-	protected $config;
-	/** @var ICacheFactory|\PHPUnit\Framework\MockObject\MockObject */
-	protected $cacheFactory;
-	/** @var LoggerInterface|\PHPUnit\Framework\MockObject\MockObject */
-	protected $logger;
-	/** @var IAppManager|\PHPUnit\Framework\MockObject\MockObject */
-	protected $appManager;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->appData = $this->createMock(IAppData::class);
-		$this->urlGenerator = $this->createMock(IURLGenerator::class);
-		$this->config = $this->createMock(SystemConfig::class);
-		$this->cacheFactory = $this->createMock(ICacheFactory::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->appManager = $this->createMock(IAppManager::class);
-	}
-
-	private function jsResourceLocator() {
-		$jsCombiner = new JSCombiner(
-			$this->appData,
-			$this->urlGenerator,
-			$this->cacheFactory,
-			$this->config,
-			$this->logger
-		);
-		return new JSResourceLocator(
-			$this->logger,
-			$jsCombiner,
-			$this->appManager,
-		);
-	}
-
-	private function rrmdir($directory) {
-		$files = array_diff(scandir($directory), ['.','..']);
-		foreach ($files as $file) {
-			if (is_dir($directory . '/' . $file)) {
-				$this->rrmdir($directory . '/' . $file);
-			} else {
-				unlink($directory . '/' . $file);
-			}
-		}
-		return rmdir($directory);
-	}
-
-	private function randomString() {
-		return sha1(uniqid(mt_rand(), true));
-	}
-
-	public function testFindWithAppPathSymlink(): void {
-		$appName = 'test-js-app';
-
-		// First create new apps path, and a symlink to it
-		$apps_dirname = $this->randomString();
-		$new_apps_path = sys_get_temp_dir() . '/' . $apps_dirname;
-		$new_apps_path_symlink = $new_apps_path . '_link';
-		$this->assertTrue((
-			mkdir($new_apps_path) && symlink($apps_dirname, $new_apps_path_symlink)
-		), 'Setup of apps path failed');
-
-		// Create an app within that path
-		$this->assertTrue((
-			mkdir($new_apps_path . '/' . $appName) && touch($new_apps_path . '/' . $appName . '/' . 'test-file.js')
-		), 'Setup of app within the new apps path failed');
-
-		// Use the symlink as the app path
-		$this->appManager->expects($this->once())
-			->method('getAppPath')
-			->with($appName)
-			->willReturn("$new_apps_path_symlink/$appName");
-		$this->appManager->expects($this->once())
-			->method('getAppWebPath')
-			->with($appName)
-			->willReturn("/js-apps-test/$appName");
-
-		// Run the tests
-		$locator = $this->jsResourceLocator();
-		$locator->find(["$appName/test-file"]);
-
-		$resources = $locator->getResources();
-		$this->assertCount(1, $resources);
-		$resource = $resources[0];
-		$this->assertCount(3, $resource);
-		$root = $resource[0];
-		$webRoot = $resource[1];
-		$file = $resource[2];
-
-		$expectedRoot = $new_apps_path;
-		$expectedWebRoot = \OC::$WEBROOT . '/js-apps-test';
-		$expectedFile = $appName . '/test-file.js';
-
-		$this->assertEquals($expectedRoot, $root,
-			'Ensure the app path symlink is resolved into the real path');
-		$this->assertEquals($expectedWebRoot, $webRoot);
-		$this->assertEquals($expectedFile, $file);
-
-		unlink($new_apps_path_symlink);
-		$this->rrmdir($new_apps_path);
-	}
-
-	public function testNotExistingTranslationHandledSilent(): void {
-		$this->appManager->expects($this->once())
-			->method('getAppPath')
-			->with('core')
-			->willThrowException(new AppPathNotFoundException());
-		$this->appManager->expects($this->atMost(1))
-			->method('getAppWebPath')
-			->with('core')
-			->willThrowException(new AppPathNotFoundException());
-		// Assert logger is not called
-		$this->logger->expects($this->never())
-			->method('error');
-
-		// Run the tests
-		$locator = $this->jsResourceLocator();
-		$locator->find(['core/l10n/en.js']);
-
-		$resources = $locator->getResources();
-		$this->assertCount(0, $resources);
-	}
-
-	public function testFindModuleJSWithFallback(): void {
-		// First create new apps path, and a symlink to it
-		$apps_dirname = $this->randomString();
-		$new_apps_path = sys_get_temp_dir() . '/' . $apps_dirname;
-		mkdir($new_apps_path);
-
-		// Create an app within that path
-		mkdir("$new_apps_path/test-js-app");
-		touch("$new_apps_path/test-js-app/module.mjs");
-		touch("$new_apps_path/test-js-app/both.mjs");
-		touch("$new_apps_path/test-js-app/both.js");
-		touch("$new_apps_path/test-js-app/plain.js");
-
-		// Use the app path
-		$this->appManager->expects($this->any())
-			->method('getAppPath')
-			->with('test-js-app')
-			->willReturn("$new_apps_path/test-js-app");
-
-		$locator = $this->jsResourceLocator();
-		$locator->find(['test-js-app/module', 'test-js-app/both', 'test-js-app/plain']);
-
-		$resources = $locator->getResources();
-		$this->assertCount(3, $resources);
-
-		$expectedWebRoot = \OC::$WEBROOT . '/js-apps-test/test-js-app';
-		$expectedFiles = ['module.mjs', 'both.mjs', 'plain.js'];
-
-		for ($idx = 0; $idx++; $idx < 3) {
-			$this->assertEquals($expectedWebRoot, $resources[$idx][1]);
-			$this->assertEquals($expectedFiles[$idx], $resources[$idx][2]);
-		}
-
-		$this->rrmdir($new_apps_path);
-	}
+    /** @var IAppData|\PHPUnit\Framework\MockObject\MockObject */
+    protected $appData;
+    /** @var IURLGenerator|\PHPUnit\Framework\MockObject\MockObject */
+    protected $urlGenerator;
+    /** @var SystemConfig|\PHPUnit\Framework\MockObject\MockObject */
+    protected $config;
+    /** @var ICacheFactory|\PHPUnit\Framework\MockObject\MockObject */
+    protected $cacheFactory;
+    /** @var LoggerInterface|\PHPUnit\Framework\MockObject\MockObject */
+    protected $logger;
+    /** @var IAppManager|\PHPUnit\Framework\MockObject\MockObject */
+    protected $appManager;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->appData = $this->createMock(IAppData::class);
+        $this->urlGenerator = $this->createMock(IURLGenerator::class);
+        $this->config = $this->createMock(SystemConfig::class);
+        $this->cacheFactory = $this->createMock(ICacheFactory::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->appManager = $this->createMock(IAppManager::class);
+    }
+
+    private function jsResourceLocator() {
+        $jsCombiner = new JSCombiner(
+            $this->appData,
+            $this->urlGenerator,
+            $this->cacheFactory,
+            $this->config,
+            $this->logger
+        );
+        return new JSResourceLocator(
+            $this->logger,
+            $jsCombiner,
+            $this->appManager,
+        );
+    }
+
+    private function rrmdir($directory) {
+        $files = array_diff(scandir($directory), ['.','..']);
+        foreach ($files as $file) {
+            if (is_dir($directory . '/' . $file)) {
+                $this->rrmdir($directory . '/' . $file);
+            } else {
+                unlink($directory . '/' . $file);
+            }
+        }
+        return rmdir($directory);
+    }
+
+    private function randomString() {
+        return sha1(uniqid(mt_rand(), true));
+    }
+
+    public function testFindWithAppPathSymlink(): void {
+        $appName = 'test-js-app';
+
+        // First create new apps path, and a symlink to it
+        $apps_dirname = $this->randomString();
+        $new_apps_path = sys_get_temp_dir() . '/' . $apps_dirname;
+        $new_apps_path_symlink = $new_apps_path . '_link';
+        $this->assertTrue((
+            mkdir($new_apps_path) && symlink($apps_dirname, $new_apps_path_symlink)
+        ), 'Setup of apps path failed');
+
+        // Create an app within that path
+        $this->assertTrue((
+            mkdir($new_apps_path . '/' . $appName) && touch($new_apps_path . '/' . $appName . '/' . 'test-file.js')
+        ), 'Setup of app within the new apps path failed');
+
+        // Use the symlink as the app path
+        $this->appManager->expects($this->once())
+            ->method('getAppPath')
+            ->with($appName)
+            ->willReturn("$new_apps_path_symlink/$appName");
+        $this->appManager->expects($this->once())
+            ->method('getAppWebPath')
+            ->with($appName)
+            ->willReturn("/js-apps-test/$appName");
+
+        // Run the tests
+        $locator = $this->jsResourceLocator();
+        $locator->find(["$appName/test-file"]);
+
+        $resources = $locator->getResources();
+        $this->assertCount(1, $resources);
+        $resource = $resources[0];
+        $this->assertCount(3, $resource);
+        $root = $resource[0];
+        $webRoot = $resource[1];
+        $file = $resource[2];
+
+        $expectedRoot = $new_apps_path;
+        $expectedWebRoot = \OC::$WEBROOT . '/js-apps-test';
+        $expectedFile = $appName . '/test-file.js';
+
+        $this->assertEquals($expectedRoot, $root,
+            'Ensure the app path symlink is resolved into the real path');
+        $this->assertEquals($expectedWebRoot, $webRoot);
+        $this->assertEquals($expectedFile, $file);
+
+        unlink($new_apps_path_symlink);
+        $this->rrmdir($new_apps_path);
+    }
+
+    public function testNotExistingTranslationHandledSilent(): void {
+        $this->appManager->expects($this->once())
+            ->method('getAppPath')
+            ->with('core')
+            ->willThrowException(new AppPathNotFoundException());
+        $this->appManager->expects($this->atMost(1))
+            ->method('getAppWebPath')
+            ->with('core')
+            ->willThrowException(new AppPathNotFoundException());
+        // Assert logger is not called
+        $this->logger->expects($this->never())
+            ->method('error');
+
+        // Run the tests
+        $locator = $this->jsResourceLocator();
+        $locator->find(['core/l10n/en.js']);
+
+        $resources = $locator->getResources();
+        $this->assertCount(0, $resources);
+    }
+
+    public function testFindModuleJSWithFallback(): void {
+        // First create new apps path, and a symlink to it
+        $apps_dirname = $this->randomString();
+        $new_apps_path = sys_get_temp_dir() . '/' . $apps_dirname;
+        mkdir($new_apps_path);
+
+        // Create an app within that path
+        mkdir("$new_apps_path/test-js-app");
+        touch("$new_apps_path/test-js-app/module.mjs");
+        touch("$new_apps_path/test-js-app/both.mjs");
+        touch("$new_apps_path/test-js-app/both.js");
+        touch("$new_apps_path/test-js-app/plain.js");
+
+        // Use the app path
+        $this->appManager->expects($this->any())
+            ->method('getAppPath')
+            ->with('test-js-app')
+            ->willReturn("$new_apps_path/test-js-app");
+
+        $locator = $this->jsResourceLocator();
+        $locator->find(['test-js-app/module', 'test-js-app/both', 'test-js-app/plain']);
+
+        $resources = $locator->getResources();
+        $this->assertCount(3, $resources);
+
+        $expectedWebRoot = \OC::$WEBROOT . '/js-apps-test/test-js-app';
+        $expectedFiles = ['module.mjs', 'both.mjs', 'plain.js'];
+
+        for ($idx = 0; $idx++; $idx < 3) {
+            $this->assertEquals($expectedWebRoot, $resources[$idx][1]);
+            $this->assertEquals($expectedFiles[$idx], $resources[$idx][2]);
+        }
+
+        $this->rrmdir($new_apps_path);
+    }
 }

tests/lib/Talk/ConversationOptionsTest.php

Indentation +4 added lines, -4 removed lines

@@ -13,9 +13,9 @@
 use Test\TestCase;
 
 class ConversationOptionsTest extends TestCase {
-	public function testDefaults(): void {
-		ConversationOptions::default();
+    public function testDefaults(): void {
+        ConversationOptions::default();
 
-		$this->addToAssertionCount(1);
-	}
+        $this->addToAssertionCount(1);
+    }
 }

tests/lib/Talk/BrokerTest.php

Indentation +110 added lines, -110 removed lines

@@ -22,114 +22,114 @@
 use Test\TestCase;
 
 class BrokerTest extends TestCase {
-	private Coordinator $coordinator;
-
-	private IServerContainer $container;
-
-	private LoggerInterface $logger;
-
-	private Broker $broker;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->coordinator = $this->createMock(Coordinator::class);
-		$this->container = $this->createMock(IServerContainer::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-
-		$this->broker = new Broker(
-			$this->coordinator,
-			$this->container,
-			$this->logger,
-		);
-	}
-
-	public function testHasNoBackendCalledTooEarly(): void {
-		$this->expectException(RuntimeException::class);
-
-		$this->broker->hasBackend();
-	}
-
-	public function testHasNoBackend(): void {
-		$this->coordinator->expects(self::once())
-			->method('getRegistrationContext')
-			->willReturn($this->createMock(RegistrationContext::class));
-
-		self::assertFalse(
-			$this->broker->hasBackend()
-		);
-	}
-
-	public function testHasFaultyBackend(): void {
-		$fakeTalkServiceClass = '\\OCA\\Spreed\\TalkBackend';
-		$registrationContext = $this->createMock(RegistrationContext::class);
-		$this->coordinator->expects(self::once())
-			->method('getRegistrationContext')
-			->willReturn($registrationContext);
-		$registrationContext->expects(self::once())
-			->method('getTalkBackendRegistration')
-			->willReturn(new ServiceRegistration('spreed', $fakeTalkServiceClass));
-		$this->container->expects(self::once())
-			->method('get')
-			->willThrowException(new QueryException());
-		$this->logger->expects(self::once())
-			->method('error');
-
-		self::assertFalse(
-			$this->broker->hasBackend()
-		);
-	}
-
-	public function testHasBackend(): void {
-		$fakeTalkServiceClass = '\\OCA\\Spreed\\TalkBackend';
-		$registrationContext = $this->createMock(RegistrationContext::class);
-		$this->coordinator->expects(self::once())
-			->method('getRegistrationContext')
-			->willReturn($registrationContext);
-		$registrationContext->expects(self::once())
-			->method('getTalkBackendRegistration')
-			->willReturn(new ServiceRegistration('spreed', $fakeTalkServiceClass));
-		$talkService = $this->createMock(ITalkBackend::class);
-		$this->container->expects(self::once())
-			->method('get')
-			->with($fakeTalkServiceClass)
-			->willReturn($talkService);
-
-		self::assertTrue(
-			$this->broker->hasBackend()
-		);
-	}
-
-	public function testNewConversationOptions(): void {
-		$this->broker->newConversationOptions();
-
-		// Nothing to assert
-		$this->addToAssertionCount(1);
-	}
-
-	public function testCreateConversation(): void {
-		$fakeTalkServiceClass = '\\OCA\\Spreed\\TalkBackend';
-		$registrationContext = $this->createMock(RegistrationContext::class);
-		$this->coordinator->expects(self::once())
-			->method('getRegistrationContext')
-			->willReturn($registrationContext);
-		$registrationContext->expects(self::once())
-			->method('getTalkBackendRegistration')
-			->willReturn(new ServiceRegistration('spreed', $fakeTalkServiceClass));
-		$talkService = $this->createMock(ITalkBackend::class);
-		$this->container->expects(self::once())
-			->method('get')
-			->with($fakeTalkServiceClass)
-			->willReturn($talkService);
-		$options = $this->createMock(IConversationOptions::class);
-		$talkService->expects(self::once())
-			->method('createConversation')
-			->with('Watercooler', [], $options);
-
-		$this->broker->createConversation(
-			'Watercooler',
-			[],
-			$options
-		);
-	}
+    private Coordinator $coordinator;
+
+    private IServerContainer $container;
+
+    private LoggerInterface $logger;
+
+    private Broker $broker;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->coordinator = $this->createMock(Coordinator::class);
+        $this->container = $this->createMock(IServerContainer::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+
+        $this->broker = new Broker(
+            $this->coordinator,
+            $this->container,
+            $this->logger,
+        );
+    }
+
+    public function testHasNoBackendCalledTooEarly(): void {
+        $this->expectException(RuntimeException::class);
+
+        $this->broker->hasBackend();
+    }
+
+    public function testHasNoBackend(): void {
+        $this->coordinator->expects(self::once())
+            ->method('getRegistrationContext')
+            ->willReturn($this->createMock(RegistrationContext::class));
+
+        self::assertFalse(
+            $this->broker->hasBackend()
+        );
+    }
+
+    public function testHasFaultyBackend(): void {
+        $fakeTalkServiceClass = '\\OCA\\Spreed\\TalkBackend';
+        $registrationContext = $this->createMock(RegistrationContext::class);
+        $this->coordinator->expects(self::once())
+            ->method('getRegistrationContext')
+            ->willReturn($registrationContext);
+        $registrationContext->expects(self::once())
+            ->method('getTalkBackendRegistration')
+            ->willReturn(new ServiceRegistration('spreed', $fakeTalkServiceClass));
+        $this->container->expects(self::once())
+            ->method('get')
+            ->willThrowException(new QueryException());
+        $this->logger->expects(self::once())
+            ->method('error');
+
+        self::assertFalse(
+            $this->broker->hasBackend()
+        );
+    }
+
+    public function testHasBackend(): void {
+        $fakeTalkServiceClass = '\\OCA\\Spreed\\TalkBackend';
+        $registrationContext = $this->createMock(RegistrationContext::class);
+        $this->coordinator->expects(self::once())
+            ->method('getRegistrationContext')
+            ->willReturn($registrationContext);
+        $registrationContext->expects(self::once())
+            ->method('getTalkBackendRegistration')
+            ->willReturn(new ServiceRegistration('spreed', $fakeTalkServiceClass));
+        $talkService = $this->createMock(ITalkBackend::class);
+        $this->container->expects(self::once())
+            ->method('get')
+            ->with($fakeTalkServiceClass)
+            ->willReturn($talkService);
+
+        self::assertTrue(
+            $this->broker->hasBackend()
+        );
+    }
+
+    public function testNewConversationOptions(): void {
+        $this->broker->newConversationOptions();
+
+        // Nothing to assert
+        $this->addToAssertionCount(1);
+    }
+
+    public function testCreateConversation(): void {
+        $fakeTalkServiceClass = '\\OCA\\Spreed\\TalkBackend';
+        $registrationContext = $this->createMock(RegistrationContext::class);
+        $this->coordinator->expects(self::once())
+            ->method('getRegistrationContext')
+            ->willReturn($registrationContext);
+        $registrationContext->expects(self::once())
+            ->method('getTalkBackendRegistration')
+            ->willReturn(new ServiceRegistration('spreed', $fakeTalkServiceClass));
+        $talkService = $this->createMock(ITalkBackend::class);
+        $this->container->expects(self::once())
+            ->method('get')
+            ->with($fakeTalkServiceClass)
+            ->willReturn($talkService);
+        $options = $this->createMock(IConversationOptions::class);
+        $talkService->expects(self::once())
+            ->method('createConversation')
+            ->with('Watercooler', [], $options);
+
+        $this->broker->createConversation(
+            'Watercooler',
+            [],
+            $options
+        );
+    }
 }

tests/lib/Mail/Provider/AttachmentTest.php

Indentation +39 added lines, -39 removed lines

@@ -13,59 +13,59 @@
 
 class AttachmentTest extends TestCase {
 
-	/** @var Attachment&MockObject */
-	private Attachment $attachment;
+    /** @var Attachment&MockObject */
+    private Attachment $attachment;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->attachment = new Attachment(
-			'This is the contents of a file',
-			'example1.txt',
-			'text/plain',
-			false
-		);
+        $this->attachment = new Attachment(
+            'This is the contents of a file',
+            'example1.txt',
+            'text/plain',
+            false
+        );
 
-	}
+    }
 
-	public function testName(): void {
+    public function testName(): void {
 		
-		// test set by constructor
-		$this->assertEquals('example1.txt', $this->attachment->getName());
-		// test set by setter
-		$this->attachment->setName('example2.txt');
-		$this->assertEquals('example2.txt', $this->attachment->getName());
+        // test set by constructor
+        $this->assertEquals('example1.txt', $this->attachment->getName());
+        // test set by setter
+        $this->attachment->setName('example2.txt');
+        $this->assertEquals('example2.txt', $this->attachment->getName());
 
-	}
+    }
 
-	public function testType(): void {
+    public function testType(): void {
 		
-		// test set by constructor
-		$this->assertEquals('text/plain', $this->attachment->getType());
-		// test set by setter
-		$this->attachment->setType('text/html');
-		$this->assertEquals('text/html', $this->attachment->getType());
+        // test set by constructor
+        $this->assertEquals('text/plain', $this->attachment->getType());
+        // test set by setter
+        $this->attachment->setType('text/html');
+        $this->assertEquals('text/html', $this->attachment->getType());
 
-	}
+    }
 
-	public function testContents(): void {
+    public function testContents(): void {
 		
-		// test set by constructor
-		$this->assertEquals('This is the contents of a file', $this->attachment->getContents());
-		// test set by setter
-		$this->attachment->setContents('This is the modified contents of a file');
-		$this->assertEquals('This is the modified contents of a file', $this->attachment->getContents());
+        // test set by constructor
+        $this->assertEquals('This is the contents of a file', $this->attachment->getContents());
+        // test set by setter
+        $this->attachment->setContents('This is the modified contents of a file');
+        $this->assertEquals('This is the modified contents of a file', $this->attachment->getContents());
 
-	}
+    }
 
-	public function testEmbedded(): void {
+    public function testEmbedded(): void {
 		
-		// test set by constructor
-		$this->assertEquals(false, $this->attachment->getEmbedded());
-		// test set by setter
-		$this->attachment->setEmbedded(true);
-		$this->assertEquals(true, $this->attachment->getEmbedded());
+        // test set by constructor
+        $this->assertEquals(false, $this->attachment->getEmbedded());
+        // test set by setter
+        $this->attachment->setEmbedded(true);
+        $this->assertEquals(true, $this->attachment->getEmbedded());
 
-	}
+    }
 
 }

tests/lib/Mail/Provider/ManagerTest.php

Indentation +163 added lines, -163 removed lines

@@ -21,169 +21,169 @@
 
 class ManagerTest extends TestCase {
 
-	/** @var Coordinator&MockObject */
-	private Coordinator $coordinator;
-	/** @var ContainerInterface&MockObject */
-	private ContainerInterface $container;
-	/** @var LoggerInterface&MockObject */
-	private LoggerInterface $logger;
-	/** @var IProvider&MockObject */
-	private IProvider $provider;
-	/** @var IService&MockObject */
-	private IService $service;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->logger = $this->createMock(LoggerInterface::class);
-
-		// construct service registration
-		$registration = $this->createMock(ServiceRegistration::class);
-		$registration
-			->method('getService')
-			->willReturn('Mock\Provider\MailProvider');
-		// construct registration context
-		$context = $this->createMock(RegistrationContext::class);
-		$context
-			->method('getMailProviders')
-			->willReturn([$registration]);
-		// construct coordinator
-		$this->coordinator = $this->createMock(Coordinator::class);
-		$this->coordinator
-			->method('getRegistrationContext')
-			->willReturn($context);
-
-		// construct mail service
-		$this->service = $this->createMock(IService::class);
-		$this->service
-			->method('id')
-			->willReturn('100');
-		$this->service
-			->method('getLabel')
-			->willReturn('Mock Mail Service');
-		$this->service
-			->method('getPrimaryAddress')
-			->willReturn((new Address('user1@testing.com', 'User One')));
-		// construct mail provider
-		$this->provider = $this->createMock(IProvider::class);
-		$this->provider
-			->method('id')
-			->willReturn('mock-provider');
-		$this->provider
-			->method('label')
-			->willReturn('Mock Provider');
-		$this->provider
-			->method('listServices')
-			->willReturnMap([
-				['user0', []],
-				['user1', [$this->service->id() => $this->service]]
-			]);
-		$this->provider
-			->method('findServiceById')
-			->willReturnMap([
-				['user0', '100', null],
-				['user1', '100', $this->service]
-			]);
-		$this->provider
-			->method('findServiceByAddress')
-			->willReturnMap([
-				['user0', 'user0@testing.com', null],
-				['user1', 'user1@testing.com', $this->service]
-			]);
-		// construct container interface
-		$this->container = $this->createMock(ContainerInterface::class);
-		$this->container
-			->method('get')
-			->willReturnMap([
-				['Mock\Provider\MailProvider', $this->provider]
-			]);
-
-	}
-
-	public function testHas(): void {
-
-		// construct mail manager
-		$manager = new Manager($this->coordinator, $this->container, $this->logger);
-		// test result with providers found
-		$this->assertTrue($manager->has());
-
-	}
-
-	public function testCount(): void {
-
-		// construct mail manager
-		$manager = new Manager($this->coordinator, $this->container, $this->logger);
-		// test result with providers found
-		$this->assertGreaterThan(0, $manager->count());
-
-	}
-
-	public function testTypes(): void {
-
-		// construct mail manager
-		$manager = new Manager($this->coordinator, $this->container, $this->logger);
-		// test result with providers found
-		$this->assertEquals(['mock-provider' => 'Mock Provider'], $manager->types());
-
-	}
-
-	public function testProviders(): void {
-
-		// construct mail manager
-		$manager = new Manager($this->coordinator, $this->container, $this->logger);
-		// test result with providers found
-		$this->assertEquals([$this->provider->id() => $this->provider], $manager->providers());
-
-	}
-
-	public function testFindProviderById(): void {
-
-		// construct mail manager
-		$manager = new Manager($this->coordinator, $this->container, $this->logger);
-		// test result with providers found
-		$this->assertEquals($this->provider, $manager->findProviderById($this->provider->id()));
-
-	}
-
-	public function testServices(): void {
-
-		// construct mail manager
-		$manager = new Manager($this->coordinator, $this->container, $this->logger);
-		// test result with no services found
-		$this->assertEquals([], $manager->services('user0'));
-		// test result with services found
-		$this->assertEquals([$this->provider->id() => [$this->service->id() => $this->service]], $manager->services('user1'));
-
-	}
-
-	public function testFindServiceById(): void {
-
-		// construct mail manager
-		$manager = new Manager($this->coordinator, $this->container, $this->logger);
-		// test result with no services found and not provider specified
-		$this->assertEquals(null, $manager->findServiceById('user0', '100'));
-		// test result with no services found and provider specified
-		$this->assertEquals(null, $manager->findServiceById('user0', '100', $this->provider->id()));
-		// test result with services found and not provider specified
-		$this->assertEquals($this->service, $manager->findServiceById('user1', '100'));
-		// test result with services found and provider specified
-		$this->assertEquals($this->service, $manager->findServiceById('user1', '100', $this->provider->id()));
-
-	}
-
-	public function testFindServiceByAddress(): void {
-
-		// construct mail manager
-		$manager = new Manager($this->coordinator, $this->container, $this->logger);
-		// test result with no services found and not provider specified
-		$this->assertEquals(null, $manager->findServiceByAddress('user0', 'user0@testing.com'));
-		// test result with no services found and provider specified
-		$this->assertEquals(null, $manager->findServiceByAddress('user0', 'user0@testing.com', $this->provider->id()));
-		// test result with services found and not provider specified
-		$this->assertEquals($this->service, $manager->findServiceByAddress('user1', 'user1@testing.com'));
-		// test result with services found and provider specified
-		$this->assertEquals($this->service, $manager->findServiceByAddress('user1', 'user1@testing.com', $this->provider->id()));
+    /** @var Coordinator&MockObject */
+    private Coordinator $coordinator;
+    /** @var ContainerInterface&MockObject */
+    private ContainerInterface $container;
+    /** @var LoggerInterface&MockObject */
+    private LoggerInterface $logger;
+    /** @var IProvider&MockObject */
+    private IProvider $provider;
+    /** @var IService&MockObject */
+    private IService $service;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->logger = $this->createMock(LoggerInterface::class);
+
+        // construct service registration
+        $registration = $this->createMock(ServiceRegistration::class);
+        $registration
+            ->method('getService')
+            ->willReturn('Mock\Provider\MailProvider');
+        // construct registration context
+        $context = $this->createMock(RegistrationContext::class);
+        $context
+            ->method('getMailProviders')
+            ->willReturn([$registration]);
+        // construct coordinator
+        $this->coordinator = $this->createMock(Coordinator::class);
+        $this->coordinator
+            ->method('getRegistrationContext')
+            ->willReturn($context);
+
+        // construct mail service
+        $this->service = $this->createMock(IService::class);
+        $this->service
+            ->method('id')
+            ->willReturn('100');
+        $this->service
+            ->method('getLabel')
+            ->willReturn('Mock Mail Service');
+        $this->service
+            ->method('getPrimaryAddress')
+            ->willReturn((new Address('user1@testing.com', 'User One')));
+        // construct mail provider
+        $this->provider = $this->createMock(IProvider::class);
+        $this->provider
+            ->method('id')
+            ->willReturn('mock-provider');
+        $this->provider
+            ->method('label')
+            ->willReturn('Mock Provider');
+        $this->provider
+            ->method('listServices')
+            ->willReturnMap([
+                ['user0', []],
+                ['user1', [$this->service->id() => $this->service]]
+            ]);
+        $this->provider
+            ->method('findServiceById')
+            ->willReturnMap([
+                ['user0', '100', null],
+                ['user1', '100', $this->service]
+            ]);
+        $this->provider
+            ->method('findServiceByAddress')
+            ->willReturnMap([
+                ['user0', 'user0@testing.com', null],
+                ['user1', 'user1@testing.com', $this->service]
+            ]);
+        // construct container interface
+        $this->container = $this->createMock(ContainerInterface::class);
+        $this->container
+            ->method('get')
+            ->willReturnMap([
+                ['Mock\Provider\MailProvider', $this->provider]
+            ]);
+
+    }
+
+    public function testHas(): void {
+
+        // construct mail manager
+        $manager = new Manager($this->coordinator, $this->container, $this->logger);
+        // test result with providers found
+        $this->assertTrue($manager->has());
+
+    }
+
+    public function testCount(): void {
+
+        // construct mail manager
+        $manager = new Manager($this->coordinator, $this->container, $this->logger);
+        // test result with providers found
+        $this->assertGreaterThan(0, $manager->count());
+
+    }
+
+    public function testTypes(): void {
+
+        // construct mail manager
+        $manager = new Manager($this->coordinator, $this->container, $this->logger);
+        // test result with providers found
+        $this->assertEquals(['mock-provider' => 'Mock Provider'], $manager->types());
+
+    }
+
+    public function testProviders(): void {
+
+        // construct mail manager
+        $manager = new Manager($this->coordinator, $this->container, $this->logger);
+        // test result with providers found
+        $this->assertEquals([$this->provider->id() => $this->provider], $manager->providers());
+
+    }
+
+    public function testFindProviderById(): void {
+
+        // construct mail manager
+        $manager = new Manager($this->coordinator, $this->container, $this->logger);
+        // test result with providers found
+        $this->assertEquals($this->provider, $manager->findProviderById($this->provider->id()));
+
+    }
+
+    public function testServices(): void {
+
+        // construct mail manager
+        $manager = new Manager($this->coordinator, $this->container, $this->logger);
+        // test result with no services found
+        $this->assertEquals([], $manager->services('user0'));
+        // test result with services found
+        $this->assertEquals([$this->provider->id() => [$this->service->id() => $this->service]], $manager->services('user1'));
+
+    }
+
+    public function testFindServiceById(): void {
+
+        // construct mail manager
+        $manager = new Manager($this->coordinator, $this->container, $this->logger);
+        // test result with no services found and not provider specified
+        $this->assertEquals(null, $manager->findServiceById('user0', '100'));
+        // test result with no services found and provider specified
+        $this->assertEquals(null, $manager->findServiceById('user0', '100', $this->provider->id()));
+        // test result with services found and not provider specified
+        $this->assertEquals($this->service, $manager->findServiceById('user1', '100'));
+        // test result with services found and provider specified
+        $this->assertEquals($this->service, $manager->findServiceById('user1', '100', $this->provider->id()));
+
+    }
+
+    public function testFindServiceByAddress(): void {
+
+        // construct mail manager
+        $manager = new Manager($this->coordinator, $this->container, $this->logger);
+        // test result with no services found and not provider specified
+        $this->assertEquals(null, $manager->findServiceByAddress('user0', 'user0@testing.com'));
+        // test result with no services found and provider specified
+        $this->assertEquals(null, $manager->findServiceByAddress('user0', 'user0@testing.com', $this->provider->id()));
+        // test result with services found and not provider specified
+        $this->assertEquals($this->service, $manager->findServiceByAddress('user1', 'user1@testing.com'));
+        // test result with services found and provider specified
+        $this->assertEquals($this->service, $manager->findServiceByAddress('user1', 'user1@testing.com', $this->provider->id()));
 
-	}
+    }
 
 }

tests/lib/Mail/Provider/AddressTest.php

Indentation +20 added lines, -20 removed lines

@@ -13,34 +13,34 @@
 
 class AddressTest extends TestCase {
 
-	/** @var Address&MockObject */
-	private Address $address;
+    /** @var Address&MockObject */
+    private Address $address;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->address = new Address('user1@testing.com', 'User One');
+        $this->address = new Address('user1@testing.com', 'User One');
 
-	}
+    }
 
-	public function testAddress(): void {
+    public function testAddress(): void {
 		
-		// test set by constructor
-		$this->assertEquals('user1@testing.com', $this->address->getAddress());
-		// test set by setter
-		$this->address->setAddress('user2@testing.com');
-		$this->assertEquals('user2@testing.com', $this->address->getAddress());
+        // test set by constructor
+        $this->assertEquals('user1@testing.com', $this->address->getAddress());
+        // test set by setter
+        $this->address->setAddress('user2@testing.com');
+        $this->assertEquals('user2@testing.com', $this->address->getAddress());
 
-	}
+    }
 
-	public function testLabel(): void {
+    public function testLabel(): void {
 		
-		// test set by constructor
-		$this->assertEquals('User One', $this->address->getLabel());
-		// test set by setter
-		$this->address->setLabel('User Two');
-		$this->assertEquals('User Two', $this->address->getLabel());
+        // test set by constructor
+        $this->assertEquals('User One', $this->address->getLabel());
+        // test set by setter
+        $this->address->setLabel('User Two');
+        $this->assertEquals('User Two', $this->address->getLabel());
 
-	}
+    }
 
 }

tests/lib/Remote/InstanceTest.php

Indentation +44 added lines, -44 removed lines

@@ -13,63 +13,63 @@
 use Test\Traits\ClientServiceTrait;
 
 class InstanceTest extends TestCase {
-	use ClientServiceTrait;
+    use ClientServiceTrait;
 
-	/** @var ICache */
-	private $cache;
+    /** @var ICache */
+    private $cache;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->cache = new ArrayCache();
-	}
+        $this->cache = new ArrayCache();
+    }
 
-	public function testBasicStatus(): void {
-		$instance = new Instance('example.com', $this->cache, $this->getClientService());
-		$this->expectGetRequest('https://example.com/status.php', '{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
+    public function testBasicStatus(): void {
+        $instance = new Instance('example.com', $this->cache, $this->getClientService());
+        $this->expectGetRequest('https://example.com/status.php', '{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
 
-		$this->assertEquals(true, $instance->isActive());
-		$this->assertEquals('13.0.0.5', $instance->getVersion());
-		$this->assertEquals('https', $instance->getProtocol());
-		$this->assertEquals('https://example.com', $instance->getFullUrl());
-	}
+        $this->assertEquals(true, $instance->isActive());
+        $this->assertEquals('13.0.0.5', $instance->getVersion());
+        $this->assertEquals('https', $instance->getProtocol());
+        $this->assertEquals('https://example.com', $instance->getFullUrl());
+    }
 
-	public function testHttpFallback(): void {
-		$instance = new Instance('example.com', $this->cache, $this->getClientService());
-		$this->expectGetRequest('https://example.com/status.php', new \Exception());
-		$this->expectGetRequest('http://example.com/status.php', '{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
+    public function testHttpFallback(): void {
+        $instance = new Instance('example.com', $this->cache, $this->getClientService());
+        $this->expectGetRequest('https://example.com/status.php', new \Exception());
+        $this->expectGetRequest('http://example.com/status.php', '{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
 
-		$this->assertEquals('http', $instance->getProtocol());
-		$this->assertEquals('http://example.com', $instance->getFullUrl());
-	}
+        $this->assertEquals('http', $instance->getProtocol());
+        $this->assertEquals('http://example.com', $instance->getFullUrl());
+    }
 
-	public function testRerequestHttps(): void {
-		$instance = new Instance('example.com', $this->cache, $this->getClientService());
-		$this->expectGetRequest('https://example.com/status.php', '{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
+    public function testRerequestHttps(): void {
+        $instance = new Instance('example.com', $this->cache, $this->getClientService());
+        $this->expectGetRequest('https://example.com/status.php', '{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
 
-		$this->assertEquals('https', $instance->getProtocol());
-		$this->assertEquals(true, $instance->isActive());
+        $this->assertEquals('https', $instance->getProtocol());
+        $this->assertEquals(true, $instance->isActive());
 
-		$this->cache->remove('remote/example.com/status');
-		$this->expectGetRequest('https://example.com/status.php', '{"installed":true,"maintenance":true,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
-		$instance2 = new Instance('example.com', $this->cache, $this->getClientService());
-		$this->assertEquals('https', $instance2->getProtocol());
-		$this->assertEquals(false, $instance2->isActive());
-	}
+        $this->cache->remove('remote/example.com/status');
+        $this->expectGetRequest('https://example.com/status.php', '{"installed":true,"maintenance":true,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
+        $instance2 = new Instance('example.com', $this->cache, $this->getClientService());
+        $this->assertEquals('https', $instance2->getProtocol());
+        $this->assertEquals(false, $instance2->isActive());
+    }
 
 	
-	public function testPreventDowngradeAttach(): void {
-		$this->expectException(\Exception::class);
-		$this->expectExceptionMessage('refusing to connect to remote instance(example.com) over http that was previously accessible over https');
+    public function testPreventDowngradeAttach(): void {
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage('refusing to connect to remote instance(example.com) over http that was previously accessible over https');
 
-		$instance = new Instance('example.com', $this->cache, $this->getClientService());
-		$this->expectGetRequest('https://example.com/status.php', '{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
+        $instance = new Instance('example.com', $this->cache, $this->getClientService());
+        $this->expectGetRequest('https://example.com/status.php', '{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"13.0.0.5","versionstring":"13.0.0 alpha","edition":"","productname":"Nextcloud"}');
 
-		$this->assertEquals('https', $instance->getProtocol());
+        $this->assertEquals('https', $instance->getProtocol());
 
-		$this->expectGetRequest('https://example.com/status.php', new \Exception());
-		$this->cache->remove('remote/example.com/status');
-		$instance2 = new Instance('example.com', $this->cache, $this->getClientService());
-		$instance2->getProtocol();
-	}
+        $this->expectGetRequest('https://example.com/status.php', new \Exception());
+        $this->cache->remove('remote/example.com/status');
+        $instance2 = new Instance('example.com', $this->cache, $this->getClientService());
+        $instance2->getProtocol();
+    }
 }