nextcloud / server

apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionUploadTest.php

Spacing +2 added lines, -2 removed lines

@@ -27,11 +27,11 @@
 	protected function setupUser($name, $password): View {
 		$this->createUser($name, $password);
 		$tmpFolder = Server::get(ITempManager::class)->getTemporaryFolder();
-		$this->registerMount($name, '\OC\Files\Storage\Local', '/' . $name, ['datadir' => $tmpFolder]);
+		$this->registerMount($name, '\OC\Files\Storage\Local', '/'.$name, ['datadir' => $tmpFolder]);
 		// we use per-user keys
 		Server::get(IConfig::class)->setAppValue('encryption', 'useMasterKey', '0');
 		$this->setupForUser($name, $password);
 		$this->loginWithEncryption($name);
-		return new View('/' . $name . '/files');
+		return new View('/'.$name.'/files');
 	}
 }

apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionMasterKeyUploadTest.php

Spacing +2 added lines, -2 removed lines

@@ -27,11 +27,11 @@
 	protected function setupUser($name, $password): View {
 		$this->createUser($name, $password);
 		$tmpFolder = Server::get(ITempManager::class)->getTemporaryFolder();
-		$this->registerMount($name, '\OC\Files\Storage\Local', '/' . $name, ['datadir' => $tmpFolder]);
+		$this->registerMount($name, '\OC\Files\Storage\Local', '/'.$name, ['datadir' => $tmpFolder]);
 		// we use the master key
 		Server::get(IConfig::class)->setAppValue('encryption', 'useMasterKey', '1');
 		$this->setupForUser($name, $password);
 		$this->loginWithEncryption($name);
-		return new View('/' . $name . '/files');
+		return new View('/'.$name.'/files');
 	}
 }

apps/dav/tests/unit/Connector/Sabre/RequestTest/RequestTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -60,9 +60,9 @@ 
 	protected function setupUser($name, $password): View {
 		$this->createUser($name, $password);
 		$tmpFolder = \OCP\Server::get(ITempManager::class)->getTemporaryFolder();
-		$this->registerMount($name, '\OC\Files\Storage\Local', '/' . $name, ['datadir' => $tmpFolder]);
+		$this->registerMount($name, '\OC\Files\Storage\Local', '/'.$name, ['datadir' => $tmpFolder]);
 		self::loginAsUser($name);
-		return new View('/' . $name . '/files');
+		return new View('/'.$name.'/files');
 	}
 
 	/**
@@ -90,7 +90,7 @@ 
 		$serverParams = [];
 		if (is_array($headers)) {
 			foreach ($headers as $header => $value) {
-				$serverParams['HTTP_' . strtoupper(str_replace('-', '_', $header))] = $value;
+				$serverParams['HTTP_'.strtoupper(str_replace('-', '_', $header))] = $value;
 			}
 		}
 		$ncRequest = new \OC\AppFramework\Http\Request([
@@ -133,7 +133,7 @@ 
 		$authBackend = new Auth($user, $password);
 		$authPlugin = new \Sabre\DAV\Auth\Plugin($authBackend);
 
-		$server = $this->serverFactory->createServer(false, '/', 'dummy', $authPlugin, function () use ($view) {
+		$server = $this->serverFactory->createServer(false, '/', 'dummy', $authPlugin, function() use ($view) {
 			return $view;
 		});
 		$server->addPlugin($exceptionPlugin);

apps/dav/tests/unit/Connector/Sabre/RequestTest/PartFileInRootUploadTest.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 		$mockConfig = $this->createMock(IConfig::class);
 		$mockConfig->expects($this->any())
 			->method('getSystemValue')
-			->willReturnCallback(function ($key, $default) use ($config) {
+			->willReturnCallback(function($key, $default) use ($config) {
 				if ($key === 'part_file_in_storage') {
 					return false;
 				} else {

apps/dav/tests/unit/Connector/Sabre/QuotaPluginTest.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 	private QuotaPlugin $plugin;
 
 	private function init(int $quota, string $checkedPath = ''): void {
-		$view = $this->buildFileViewMock((string)$quota, $checkedPath);
+		$view = $this->buildFileViewMock((string) $quota, $checkedPath);
 		$this->server = new \Sabre\DAV\Server();
 		$this->plugin = new QuotaPlugin($view);
 		$this->plugin->initialize($this->server);

apps/dav/tests/unit/Connector/Sabre/FileTest.php

Spacing +16 added lines, -16 removed lines

@@ -159,7 +159,7 @@ 
 			->onlyMethods(['writeStream'])
 			->setConstructorArgs([['datadir' => Server::get(ITempManager::class)->getTemporaryFolder()]])
 			->getMock();
-		Filesystem::mount($storage, [], $this->user . '/');
+		Filesystem::mount($storage, [], $this->user.'/');
 		/** @var View&MockObject $view */
 		$view = $this->getMockBuilder(View::class)
 			->onlyMethods(['getRelativePath', 'resolvePath'])
@@ -167,7 +167,7 @@ 
 		$view->expects($this->atLeastOnce())
 			->method('resolvePath')
 			->willReturnCallback(
-				function ($path) use ($storage) {
+				function($path) use ($storage) {
 					return [$storage, $path];
 				}
 			);
@@ -223,11 +223,11 @@ 
 		if (!is_null($viewRoot)) {
 			$view = new View($viewRoot);
 		} else {
-			$viewRoot = '/' . $this->user . '/files';
+			$viewRoot = '/'.$this->user.'/files';
 		}
 
 		$info = new \OC\Files\FileInfo(
-			$viewRoot . '/' . ltrim($path, '/'),
+			$viewRoot.'/'.ltrim($path, '/'),
 			$this->getMockStorage(),
 			null,
 			[
@@ -321,7 +321,7 @@ 
 	public function testPutSingleFileLegalMtime(mixed $requestMtime, ?int $resultMtime): void {
 		$request = new Request([
 			'server' => [
-				'HTTP_X_OC_MTIME' => (string)$requestMtime,
+				'HTTP_X_OC_MTIME' => (string) $requestMtime,
 			]
 		], $this->requestId, $this->config, null);
 		$file = 'foo.txt';
@@ -414,7 +414,7 @@ 
 		HookHelper::setUpHooks();
 
 		// happens with public webdav where the view root is the share root
-		$this->assertNotEmpty($this->doPut('/foo.txt', '/' . $this->user . '/files/noderoot'));
+		$this->assertNotEmpty($this->doPut('/foo.txt', '/'.$this->user.'/files/noderoot'));
 
 		$this->assertCount(4, HookHelper::$hookCalls);
 		$this->assertHookCall(
@@ -526,12 +526,12 @@ 
 	 * Test exception during final rename in simple upload mode
 	 */
 	public function testSimplePutFailsMoveFromStorage(): void {
-		$view = new View('/' . $this->user . '/files');
+		$view = new View('/'.$this->user.'/files');
 
 		// simulate situation where the target file is locked
 		$view->lockFile('/test.txt', ILockingProvider::LOCK_EXCLUSIVE);
 
-		$info = new \OC\Files\FileInfo('/' . $this->user . '/files/test.txt', $this->getMockStorage(), null, [
+		$info = new \OC\Files\FileInfo('/'.$this->user.'/files/test.txt', $this->getMockStorage(), null, [
 			'permissions' => Constants::PERMISSION_ALL,
 			'type' => FileInfo::TYPE_FOLDER,
 		], null);
@@ -781,11 +781,11 @@ 
 	 * Test whether locks are set before and after the operation
 	 */
 	public function testPutLocking(): void {
-		$view = new View('/' . $this->user . '/files/');
+		$view = new View('/'.$this->user.'/files/');
 
 		$path = 'test-locking.txt';
 		$info = new \OC\Files\FileInfo(
-			'/' . $this->user . '/files/' . $path,
+			'/'.$this->user.'/files/'.$path,
 			$this->getMockStorage(),
 			null,
 			[
@@ -817,7 +817,7 @@ 
 		$eventHandler->expects($this->once())
 			->method('writeCallback')
 			->willReturnCallback(
-				function () use ($view, $path, &$wasLockedPre): void {
+				function() use ($view, $path, &$wasLockedPre): void {
 					$wasLockedPre = $this->isFileLocked($view, $path, ILockingProvider::LOCK_SHARED);
 					$wasLockedPre = $wasLockedPre && !$this->isFileLocked($view, $path, ILockingProvider::LOCK_EXCLUSIVE);
 				}
@@ -825,7 +825,7 @@ 
 		$eventHandler->expects($this->once())
 			->method('postWriteCallback')
 			->willReturnCallback(
-				function () use ($view, $path, &$wasLockedPost): void {
+				function() use ($view, $path, &$wasLockedPost): void {
 					$wasLockedPost = $this->isFileLocked($view, $path, ILockingProvider::LOCK_SHARED);
 					$wasLockedPost = $wasLockedPost && !$this->isFileLocked($view, $path, ILockingProvider::LOCK_EXCLUSIVE);
 				}
@@ -986,11 +986,11 @@ 
 			'mask' => Constants::PERMISSION_ALL - Constants::PERMISSION_CREATE
 		]);
 
-		$this->registerMount($this->user, $noCreateStorage, '/' . $this->user . '/files/root');
+		$this->registerMount($this->user, $noCreateStorage, '/'.$this->user.'/files/root');
 
 		$this->loginAsUser($this->user);
 
-		$view = new View('/' . $this->user . '/files');
+		$view = new View('/'.$this->user.'/files');
 
 		$info = $view->getFileInfo('root/file.txt');
 
@@ -1008,11 +1008,11 @@ 
 	}
 
 	public function testPutLockExpired(): void {
-		$view = new View('/' . $this->user . '/files/');
+		$view = new View('/'.$this->user.'/files/');
 
 		$path = 'test-locking.txt';
 		$info = new \OC\Files\FileInfo(
-			'/' . $this->user . '/files/' . $path,
+			'/'.$this->user.'/files/'.$path,
 			$this->getMockStorage(),
 			null,
 			[

apps/dav/tests/unit/Connector/Sabre/FilesReportPluginTest.php

Spacing +6 added lines, -6 removed lines

@@ -107,7 +107,7 @@ 
 
 		$this->tree->expects($this->any())
 			->method('getNodeForPath')
-			->with('/' . $path)
+			->with('/'.$path)
 			->willReturn($this->createMock(INode::class));
 
 		$this->server->expects($this->any())
@@ -115,7 +115,7 @@ 
 			->willReturn($path);
 		$this->plugin->initialize($this->server);
 
-		$this->assertNull($this->plugin->onReport(FilesReportPluginImplementation::REPORT_NAME, [], '/' . $path));
+		$this->assertNull($this->plugin->onReport(FilesReportPluginImplementation::REPORT_NAME, [], '/'.$path));
 	}
 
 	public function testOnReportInvalidReportName(): void {
@@ -123,7 +123,7 @@ 
 
 		$this->tree->expects($this->any())
 			->method('getNodeForPath')
-			->with('/' . $path)
+			->with('/'.$path)
 			->willReturn(
 				$this->getMockBuilder(INode::class)
 					->disableOriginalConstructor()
@@ -135,7 +135,7 @@ 
 			->willReturn($path);
 		$this->plugin->initialize($this->server);
 
-		$this->assertNull($this->plugin->onReport('{whoever}whatever', [], '/' . $path));
+		$this->assertNull($this->plugin->onReport('{whoever}whatever', [], '/'.$path));
 	}
 
 	public function testOnReport(): void {
@@ -182,7 +182,7 @@ 
 
 		$this->tree->expects($this->any())
 			->method('getNodeForPath')
-			->with('/' . $path)
+			->with('/'.$path)
 			->willReturn($reportTargetNode);
 
 		$filesNode1 = $this->createMock(File::class);
@@ -227,7 +227,7 @@ 
 		$this->server->httpResponse = $response;
 		$this->plugin->initialize($this->server);
 
-		$this->assertFalse($this->plugin->onReport(FilesReportPluginImplementation::REPORT_NAME, $parameters, '/' . $path));
+		$this->assertFalse($this->plugin->onReport(FilesReportPluginImplementation::REPORT_NAME, $parameters, '/'.$path));
 	}
 
 	public function testFindNodesByFileIdsRoot(): void {

apps/dav/tests/unit/Connector/Sabre/PublicAuthTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 	private IURLGenerator&MockObject $urlGenerator;
 	private PublicAuth $auth;
 
-	private bool|string $oldUser;
+	private bool | string $oldUser;
 
 	protected function setUp(): void {
 		parent::setUp();

apps/dav/tests/unit/Connector/Sabre/NodeTest.php

Spacing +5 added lines, -5 removed lines

@@ -39,8 +39,8 @@ 
 			[Constants::PERMISSION_ALL, 'dir', false, Constants::PERMISSION_ALL, false, 'test', 'RGDNVCK'],
 			[Constants::PERMISSION_ALL, 'file', true, Constants::PERMISSION_ALL, false, 'test', 'SRGDNVW'],
 			[Constants::PERMISSION_ALL, 'file', true, Constants::PERMISSION_ALL, true, 'test', 'SRMGDNVW'],
-			[Constants::PERMISSION_ALL, 'file', true, Constants::PERMISSION_ALL, true, '' , 'SRMGDNVW'],
-			[Constants::PERMISSION_ALL, 'file', true, Constants::PERMISSION_ALL - Constants::PERMISSION_UPDATE, true, '' , 'SRMGDNV'],
+			[Constants::PERMISSION_ALL, 'file', true, Constants::PERMISSION_ALL, true, '', 'SRMGDNVW'],
+			[Constants::PERMISSION_ALL, 'file', true, Constants::PERMISSION_ALL - Constants::PERMISSION_UPDATE, true, '', 'SRMGDNV'],
 			[Constants::PERMISSION_ALL - Constants::PERMISSION_SHARE, 'file', true, Constants::PERMISSION_ALL, false, 'test', 'SGDNVW'],
 			[Constants::PERMISSION_ALL - Constants::PERMISSION_UPDATE, 'file', false, Constants::PERMISSION_ALL, false, 'test', 'RGD'],
 			[Constants::PERMISSION_ALL - Constants::PERMISSION_DELETE, 'file', false, Constants::PERMISSION_ALL, false, 'test', 'RGNVW'],
@@ -70,7 +70,7 @@ 
 		$info->method('getInternalPath')
 			->willReturn($internalPath);
 		$info->method('getMountPoint')
-			->willReturnCallback(function () use ($shared) {
+			->willReturnCallback(function() use ($shared) {
 				if ($shared) {
 					return $this->createMock(SharedMount::class);
 				} else {
@@ -245,7 +245,7 @@ 
 	/**
 	 * @dataProvider sanitizeMtimeProvider
 	 */
-	public function testSanitizeMtime(string|int $mtime, int $expected): void {
+	public function testSanitizeMtime(string | int $mtime, int $expected): void {
 		$view = $this->getMockBuilder(View::class)
 			->disableOriginalConstructor()
 			->getMock();
@@ -267,7 +267,7 @@ 
 	/**
 	 * @dataProvider invalidSanitizeMtimeProvider
 	 */
-	public function testInvalidSanitizeMtime(int|string $mtime): void {
+	public function testInvalidSanitizeMtime(int | string $mtime): void {
 		$this->expectException(\InvalidArgumentException::class);
 
 		$view = $this->createMock(View::class);