nextcloud / server

tests/Core/Service/LoginFlowV2ServiceUnitTest.php

Spacing +6 added lines, -6 removed lines

@@ -135,11 +135,11 @@ 
 		 * Cannot be mocked, because functions like getLoginName are magic functions.
 		 * To be able to set internal properties, we have to use the real class here.
 		 */
-		[$encrypted, $privateKey,] = $this->getOpenSSLEncryptedPublicAndPrivateKey('test');
+		[$encrypted, $privateKey, ] = $this->getOpenSSLEncryptedPublicAndPrivateKey('test');
 		$loginFlowV2 = new LoginFlowV2();
 		$loginFlowV2->setLoginName('test');
 		$loginFlowV2->setServer('test');
-		$loginFlowV2->setAppPassword('broken#' . $encrypted);
+		$loginFlowV2->setAppPassword('broken#'.$encrypted);
 		$loginFlowV2->setPrivateKey('encrypted(test)');
 
 		$this->crypto->expects($this->once())
@@ -252,7 +252,7 @@ 
 
 		$this->config->expects($this->exactly(1))
 			->method('getSystemValue')
-			->willReturnCallback(function ($key) use ($allowedClients) {
+			->willReturnCallback(function($key) use ($allowedClients) {
 				// Note: \OCP\IConfig::getSystemValue returns either an array or string.
 				return $key == 'core.login_flow_v2.allowed_user_agents' ? $allowedClients : '';
 			});
@@ -278,7 +278,7 @@ 
 
 		$this->config->expects($this->exactly(1))
 			->method('getSystemValue')
-			->willReturnCallback(function ($key) use ($allowedClients) {
+			->willReturnCallback(function($key) use ($allowedClients) {
 				// Note: \OCP\IConfig::getSystemValue returns either an array or string.
 				return $key == 'core.login_flow_v2.allowed_user_agents' ? $allowedClients : '';
 			});
@@ -352,7 +352,7 @@ 
 
 		$this->secureRandom->expects($this->once())
 			->method('generate')
-			->with(72, ISecureRandom::CHAR_UPPER . ISecureRandom::CHAR_LOWER . ISecureRandom::CHAR_DIGITS)
+			->with(72, ISecureRandom::CHAR_UPPER.ISecureRandom::CHAR_LOWER.ISecureRandom::CHAR_DIGITS)
 			->willReturn('test_pass');
 
 		// session token
@@ -431,7 +431,7 @@ 
 	public function testCreateTokens(): void {
 		$this->config->expects($this->exactly(2))
 			->method('getSystemValue')
-			->willReturnCallback(function ($key) {
+			->willReturnCallback(function($key) {
 				// Note: \OCP\IConfig::getSystemValue returns either an array or string.
 				return $key == 'openssl' ? [] : '';
 			});

tests/lib/Files/SimpleFS/InMemoryFileTest.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
 	#[\PHPUnit\Framework\Attributes\Before()]
 	public function setupTestPdf(): void {
 		$fileContents = file_get_contents(
-			__DIR__ . '/../../../data/test.pdf'
+			__DIR__.'/../../../data/test.pdf'
 		);
 		$this->testPdf = new InMemoryFile('test.pdf', $fileContents);
 	}
@@ -81,7 +81,7 @@ 
 	 */
 	public function testGetContent(): void {
 		self::assertEquals(
-			file_get_contents(__DIR__ . '/../../../data/test.pdf'),
+			file_get_contents(__DIR__.'/../../../data/test.pdf'),
 			$this->testPdf->getContent()
 		);
 	}

tests/lib/Repair/RepairCollationTest.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 		$this->logger = $this->createMock(LoggerInterface::class);
 		$this->config = Server::get(IConfig::class);
 		$dbPrefix = $this->config->getSystemValueString('dbtableprefix');
-		$this->tableName = $this->getUniqueID($dbPrefix . '_collation_test');
+		$this->tableName = $this->getUniqueID($dbPrefix.'_collation_test');
 		$this->connection->prepare("CREATE TABLE $this->tableName(text VARCHAR(16)) COLLATE utf8_unicode_ci")->execute();
 
 		$this->repair = new TestCollationRepair($this->config, $this->logger, $this->connection, false);

apps/settings/tests/SetupChecks/AppDirsWithDifferentOwnerTest.php

Spacing +7 added lines, -7 removed lines

@@ -31,7 +31,7 @@ 
 		$this->l10n = $this->createMock(IL10N::class);
 		$this->l10n->expects($this->any())
 			->method('t')
-			->willReturnCallback(function ($message, array $replace) {
+			->willReturnCallback(function($message, array $replace) {
 				return vsprintf($message, $replace);
 			});
 		$this->check = new AppDirsWithDifferentOwner(
@@ -46,12 +46,12 @@ 
 	 * there are no directories with different owners than the current user.
 	 */
 	public function testAppDirectoryOwnersOk(): void {
-		$tempDir = tempnam(sys_get_temp_dir(), 'apps') . 'dir';
+		$tempDir = tempnam(sys_get_temp_dir(), 'apps').'dir';
 		mkdir($tempDir);
-		mkdir($tempDir . DIRECTORY_SEPARATOR . 'app1');
-		mkdir($tempDir . DIRECTORY_SEPARATOR . 'app2');
-		$this->dirsToRemove[] = $tempDir . DIRECTORY_SEPARATOR . 'app1';
-		$this->dirsToRemove[] = $tempDir . DIRECTORY_SEPARATOR . 'app2';
+		mkdir($tempDir.DIRECTORY_SEPARATOR.'app1');
+		mkdir($tempDir.DIRECTORY_SEPARATOR.'app2');
+		$this->dirsToRemove[] = $tempDir.DIRECTORY_SEPARATOR.'app1';
+		$this->dirsToRemove[] = $tempDir.DIRECTORY_SEPARATOR.'app2';
 		$this->dirsToRemove[] = $tempDir;
 		\OC::$APPSROOTS = [
 			[
@@ -71,7 +71,7 @@ 
 	 * It's expected that no error happens since the check shouldn't apply.
 	 */
 	public function testAppDirectoryOwnersNotWritable(): void {
-		$tempDir = tempnam(sys_get_temp_dir(), 'apps') . 'dir';
+		$tempDir = tempnam(sys_get_temp_dir(), 'apps').'dir';
 		\OC::$APPSROOTS = [
 			[
 				'path' => $tempDir,

build/integration/features/bootstrap/Trashbin.php

Spacing +9 added lines, -9 removed lines

@@ -8,7 +8,7 @@ 
 
 use PHPUnit\Framework\Assert;
 
-require __DIR__ . '/../../vendor/autoload.php';
+require __DIR__.'/../../vendor/autoload.php';
 
 /**
  * Trashbin functions
@@ -67,12 +67,12 @@ 
 		$path = $this->getFullTrashPath($user, $path);
 		$client = $this->getSabreClient($user);
 
-		$results = $client->propfind($this->makeSabrePath($user, 'trash' . $path, 'trashbin'), [
+		$results = $client->propfind($this->makeSabrePath($user, 'trash'.$path, 'trashbin'), [
 			'{http://nextcloud.org/ns}trashbin-filename',
 			'{http://nextcloud.org/ns}trashbin-original-location',
 			'{http://nextcloud.org/ns}trashbin-deletion-time'
 		], 1);
-		$results = array_filter($results, function (array $item) {
+		$results = array_filter($results, function(array $item) {
 			return isset($item['{http://nextcloud.org/ns}trashbin-filename']);
 		});
 		if ($path !== '' && $path !== '/') {
@@ -89,7 +89,7 @@ 
 	 */
 	public function checkTrashContents($user, $folder, $expectedElements) {
 		$elementList = $this->listTrashbinFolder($user, $folder);
-		$trashContent = array_filter(array_map(function (array $item) {
+		$trashContent = array_filter(array_map(function(array $item) {
 			return $item['{http://nextcloud.org/ns}trashbin-filename'];
 		}, $elementList));
 		if ($expectedElements instanceof \Behat\Gherkin\Node\TableNode) {
@@ -98,7 +98,7 @@ 
 			foreach ($elementsSimplified as $expectedElement) {
 				$expectedElement = ltrim($expectedElement, '/');
 				if (array_search($expectedElement, $trashContent) === false) {
-					Assert::fail("$expectedElement" . ' is not in trash listing');
+					Assert::fail("$expectedElement".' is not in trash listing');
 				}
 			}
 		}
@@ -117,12 +117,12 @@ 
 		}
 		$name = basename($file);
 		$elementList = $this->listTrashbinFolder($user, $parent);
-		$trashContent = array_filter(array_map(function (array $item) {
+		$trashContent = array_filter(array_map(function(array $item) {
 			return $item['{http://nextcloud.org/ns}trashbin-filename'];
 		}, $elementList));
 
 		if (array_search($name, array_values($trashContent)) === false) {
-			Assert::fail("$name" . ' is not in trash listing');
+			Assert::fail("$name".' is not in trash listing');
 		}
 	}
 
@@ -144,10 +144,10 @@ 
 	 */
 	public function restoreFromTrash($user, $file) {
 		$file = $this->getFullTrashPath($user, $file);
-		$url = $this->makeSabrePath($user, 'trash' . $file, 'trashbin');
+		$url = $this->makeSabrePath($user, 'trash'.$file, 'trashbin');
 		$client = $this->getSabreClient($user);
 		$response = $client->request('MOVE', $url, null, [
-			'Destination' => $this->makeSabrePath($user, 'restore/' . basename($file), 'trashbin'),
+			'Destination' => $this->makeSabrePath($user, 'restore/'.basename($file), 'trashbin'),
 		]);
 		Assert::assertEquals(201, $response['statusCode']);
 		return;