nextcloud / server

tests/lib/Contacts/ContactsMenu/ManagerTest.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 			$entry = $this->createMock(Entry::class);
 			$entry->expects($this->any())
 				->method('getFullName')
-				->willReturn('Contact ' . $char);
+				->willReturn('Contact '.$char);
 			$entries[] = $entry;
 		}
 		return $entries;

tests/lib/DB/QueryBuilder/QuoteHelperTest.php

Spacing +2 added lines, -2 removed lines

@@ -123,9 +123,9 @@
 
 		if (substr_count($string, '.')) {
 			[$alias, $columnName] = explode('.', $string);
-			return '`' . $alias . '`.`' . $columnName . '`';
+			return '`'.$alias.'`.`'.$columnName.'`';
 		}
 
-		return '`' . $string . '`';
+		return '`'.$string.'`';
 	}
 }

tests/lib/DB/MigratorTest.php

Spacing +9 added lines, -9 removed lines

@@ -64,18 +64,18 @@ 
 	}
 
 	private function getUniqueTableName() {
-		return strtolower($this->getUniqueID($this->config->getSystemValueString('dbtableprefix', 'oc_') . 'test_'));
+		return strtolower($this->getUniqueID($this->config->getSystemValueString('dbtableprefix', 'oc_').'test_'));
 	}
 
 	protected function tearDown(): void {
 		// Try to delete if exists (IF EXISTS NOT SUPPORTED IN ORACLE)
 		try {
-			$this->connection->exec('DROP TABLE ' . $this->connection->quoteIdentifier($this->tableNameTmp));
+			$this->connection->exec('DROP TABLE '.$this->connection->quoteIdentifier($this->tableNameTmp));
 		} catch (Exception $e) {
 		}
 
 		try {
-			$this->connection->exec('DROP TABLE ' . $this->connection->quoteIdentifier($this->tableName));
+			$this->connection->exec('DROP TABLE '.$this->connection->quoteIdentifier($this->tableName));
 		} catch (Exception $e) {
 		}
 		parent::tearDown();
@@ -89,13 +89,13 @@ 
 		$table = $startSchema->createTable($this->tableName);
 		$table->addColumn('id', 'integer');
 		$table->addColumn('name', 'string');
-		$table->addIndex(['id'], $this->tableName . '_id');
+		$table->addIndex(['id'], $this->tableName.'_id');
 
 		$endSchema = new Schema([], [], $this->getSchemaConfig());
 		$table = $endSchema->createTable($this->tableName);
 		$table->addColumn('id', 'integer');
 		$table->addColumn('name', 'string');
-		$table->addUniqueIndex(['id'], $this->tableName . '_id');
+		$table->addUniqueIndex(['id'], $this->tableName.'_id');
 
 		return [$startSchema, $endSchema];
 	}
@@ -108,13 +108,13 @@ 
 		$table = $startSchema->createTable($this->tableName);
 		$table->addColumn('id', $from);
 		$table->addColumn('name', 'string');
-		$table->addIndex(['id'], $this->tableName . '_id');
+		$table->addIndex(['id'], $this->tableName.'_id');
 
 		$endSchema = new Schema([], [], $this->getSchemaConfig());
 		$table = $endSchema->createTable($this->tableName);
 		$table->addColumn('id', $to);
 		$table->addColumn('name', 'string');
-		$table->addIndex(['id'], $this->tableName . '_id');
+		$table->addIndex(['id'], $this->tableName.'_id');
 
 		return [$startSchema, $endSchema];
 	}
@@ -143,7 +143,7 @@ 
 		$oldTablePrefix = $this->config->getSystemValueString('dbtableprefix', 'oc_');
 
 		$this->config->setSystemValue('dbtableprefix', 'ownc_');
-		$this->tableName = strtolower($this->getUniqueID($this->config->getSystemValueString('dbtableprefix') . 'test_'));
+		$this->tableName = strtolower($this->getUniqueID($this->config->getSystemValueString('dbtableprefix').'test_'));
 
 		[$startSchema, $endSchema] = $this->getDuplicateKeySchemas();
 		$migrator = $this->getMigrator();
@@ -291,7 +291,7 @@ 
 		$table->addColumn('will_it_blend', $columnType, [
 			'notnull' => true,
 		]);
-		$table->addIndex(['id'], $this->tableName . '_id');
+		$table->addIndex(['id'], $this->tableName.'_id');
 
 		$migrator = $this->getMigrator();
 		$migrator->migrate($startSchema);

tests/lib/RichObjectStrings/ValidatorTest.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
 		}
 
 		$v = new Validator(new Definitions());
-		$v->validate('{' . $key . '}', [
+		$v->validate('{'.$key.'}', [
 			$key => [
 				'type' => 'highlight',
 				'id' => 'identifier',

tests/lib/RichObjectStrings/DefinitionsTest.php

Spacing +6 added lines, -6 removed lines

@@ -58,18 +58,18 @@
 	}
 
 	public function validateParameter($parameter, $data) {
-		$this->assertTrue(is_array($data), 'Parameter ' . $parameter . ' is invalid');
+		$this->assertTrue(is_array($data), 'Parameter '.$parameter.' is invalid');
 		$this->assertArrayHasKey('since', $data);
-		$this->assertNotEmpty($data['since'], 'Since of parameter ' . $parameter . ' must not be empty');
+		$this->assertNotEmpty($data['since'], 'Since of parameter '.$parameter.' must not be empty');
 		$this->assertArrayHasKey('required', $data);
-		$this->assertTrue(is_bool($data['required']), 'Required of parameter ' . $parameter . ' must be a boolean');
+		$this->assertTrue(is_bool($data['required']), 'Required of parameter '.$parameter.' must be a boolean');
 		if ($parameter === 'id' || $parameter === 'name') {
-			$this->assertTrue($data['required'], 'Parameter ' . $parameter . ' must be required');
+			$this->assertTrue($data['required'], 'Parameter '.$parameter.' must be required');
 		}
 
 		$this->assertArrayHasKey('description', $data);
-		$this->assertNotEquals('', $data['description'], 'Description of parameter ' . $parameter . ' must not be empty');
+		$this->assertNotEquals('', $data['description'], 'Description of parameter '.$parameter.' must not be empty');
 		$this->assertArrayHasKey('example', $data);
-		$this->assertNotEquals('', $data['example'], 'Example of parameter ' . $parameter . ' must not be empty');
+		$this->assertNotEquals('', $data['example'], 'Example of parameter '.$parameter.' must not be empty');
 	}
 }

tests/lib/Collaboration/Collaborators/LookupPluginTest.php

Spacing +9 added lines, -9 removed lines

@@ -52,7 +52,7 @@ 
 		$this->userSession->expects($this->any())->method('getUser')
 			->willReturn($user);
 		$this->cloudIdManager->expects($this->any())->method('resolveCloudId')
-			->willReturnCallback(function ($cloudId) {
+			->willReturnCallback(function($cloudId) {
 				if ($cloudId === 'user@myNextcloud.net') {
 					return new CloudId('user@myNextcloud.net', 'user', 'myNextcloud.net');
 				}
@@ -162,8 +162,8 @@ 
 		$client = $this->createMock(IClient::class);
 		$client->expects($this->once())
 			->method('get')
-			->willReturnCallback(function ($url) use ($searchParams, $response) {
-				$this->assertSame(strpos($url, $searchParams['server'] . '/users?search='), 0);
+			->willReturnCallback(function($url) use ($searchParams, $response) {
+				$this->assertSame(strpos($url, $searchParams['server'].'/users?search='), 0);
 				$this->assertNotFalse(strpos($url, urlencode($searchParams['search'])));
 				return $response;
 			});
@@ -226,8 +226,8 @@ 
 			$client = $this->createMock(IClient::class);
 			$client->expects($this->once())
 				->method('get')
-				->willReturnCallback(function ($url) use ($searchParams, $response) {
-					$this->assertSame(strpos($url, $searchParams['server'] . '/users?search='), 0);
+				->willReturnCallback(function($url) use ($searchParams, $response) {
+					$this->assertSame(strpos($url, $searchParams['server'].'/users?search='), 0);
 					$this->assertNotFalse(strpos($url, urlencode($searchParams['search'])));
 					return $response;
 				});
@@ -323,7 +323,7 @@ 
 						'extra' => ['federationId' => $fedIDs[2]],
 					],
 				]
-			],// GS , Lookup
+			], // GS , Lookup
 				true, true
 			],
 			[[
@@ -365,7 +365,7 @@ 
 						'extra' => ['federationId' => $fedIDs[2]],
 					],
 				]
-			],// GS , Lookup
+			], // GS , Lookup
 				true, false
 			],
 			[[
@@ -404,7 +404,7 @@ 
 						'extra' => ['federationId' => $fedIDs[2]],
 					],
 				]
-			],// GS , Lookup
+			], // GS , Lookup
 				false, true
 			],
 			[[
@@ -443,7 +443,7 @@ 
 						'extra' => ['federationId' => $fedIDs[2]],
 					],
 				]
-			],// GS , Lookup
+			], // GS , Lookup
 				false, false
 			],
 		];

tests/lib/Collaboration/Collaborators/SearchTest.php

Spacing +11 added lines, -11 removed lines

@@ -49,7 +49,7 @@ 
 		$userPlugin = $this->createMock(ISearchPlugin::class);
 		$userPlugin->expects($this->any())
 			->method('search')
-			->willReturnCallback(function () use ($searchResult, $mockedUserResult, $expectedMoreResults) {
+			->willReturnCallback(function() use ($searchResult, $mockedUserResult, $expectedMoreResults) {
 				$type = new SearchResultType('users');
 				$searchResult->addResultSet($type, $mockedUserResult);
 				return $expectedMoreResults;
@@ -58,7 +58,7 @@ 
 		$groupPlugin = $this->createMock(ISearchPlugin::class);
 		$groupPlugin->expects($this->any())
 			->method('search')
-			->willReturnCallback(function () use ($searchResult, $mockedGroupsResult, $expectedMoreResults) {
+			->willReturnCallback(function() use ($searchResult, $mockedGroupsResult, $expectedMoreResults) {
 				$type = new SearchResultType('groups');
 				$searchResult->addResultSet($type, $mockedGroupsResult);
 				return $expectedMoreResults;
@@ -67,7 +67,7 @@ 
 		$remotePlugin = $this->createMock(ISearchPlugin::class);
 		$remotePlugin->expects($this->any())
 			->method('search')
-			->willReturnCallback(function () use ($searchResult, $mockedRemotesResult, $expectedMoreResults) {
+			->willReturnCallback(function() use ($searchResult, $mockedRemotesResult, $expectedMoreResults) {
 				if ($mockedRemotesResult !== null) {
 					$type = new SearchResultType('remotes');
 					$searchResult->addResultSet($type, $mockedRemotesResult['results'], $mockedRemotesResult['exact']);
@@ -81,7 +81,7 @@ 
 		$mailPlugin = $this->createMock(ISearchPlugin::class);
 		$mailPlugin->expects($this->any())
 			->method('search')
-			->willReturnCallback(function () use ($searchResult, $mockedMailResult, $expectedMoreResults) {
+			->willReturnCallback(function() use ($searchResult, $mockedMailResult, $expectedMoreResults) {
 				$type = new SearchResultType('emails');
 				$searchResult->addResultSet($type, $mockedMailResult);
 				return $expectedMoreResults;
@@ -89,7 +89,7 @@ 
 
 		$this->container->expects($this->any())
 			->method('resolve')
-			->willReturnCallback(function ($class) use ($searchResult, $userPlugin, $groupPlugin, $remotePlugin, $mailPlugin) {
+			->willReturnCallback(function($class) use ($searchResult, $userPlugin, $groupPlugin, $remotePlugin, $mailPlugin) {
 				if ($class === SearchResult::class) {
 					return $searchResult;
 				} elseif ($class === $userPlugin) {
@@ -214,24 +214,24 @@ 
 				[IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_REMOTE, IShare::TYPE_EMAIL], // plugins
 				1, // page
 				10, // per page
-				[  // user result
+				[// user result
 					['label' => 'test One', 'value' => ['shareType' => IShare::TYPE_USER, 'shareWith' => 'test1']],
 				],
-				[  // group result
+				[// group result
 					['label' => 'testgroup1', 'value' => ['shareType' => IShare::TYPE_GROUP, 'shareWith' => 'testgroup1']],
 				],
-				[  // remote result
+				[// remote result
 					'results' => [
-						['label' => 'testz@remote.tld', 'uuid' => 'f3d78140-abcc-46df-b58d-c7cc1176aadf','value' => ['shareType' => IShare::TYPE_REMOTE, 'shareWith' => 'testz@remote.tld']]
+						['label' => 'testz@remote.tld', 'uuid' => 'f3d78140-abcc-46df-b58d-c7cc1176aadf', 'value' => ['shareType' => IShare::TYPE_REMOTE, 'shareWith' => 'testz@remote.tld']]
 					],
 					'exact' => [],
 					'exactIdMatch' => false,
 				],
-				[  //  mail result
+				[//  mail result
 					['label' => 'test Two', 'uuid' => 'b2321e9e-31af-43ac-a406-583fb26d1964', 'value' => ['shareType' => IShare::TYPE_EMAIL, 'shareWith' => 'test2@remote.tld']],
 					['label' => 'test One', 'uuid' => 'f3d78140-abcc-46df-b58d-c7cc1176aadf', 'value' => ['shareType' => IShare::TYPE_EMAIL, 'shareWith' => 'testz@remote.tld']],
 				],
-				[  // expected result
+				[// expected result
 					'exact' => ['users' => [], 'groups' => [], 'remotes' => [], 'emails' => []],
 					'users' => [
 						['label' => 'test One', 'value' => ['shareType' => IShare::TYPE_USER, 'shareWith' => 'test1']],

tests/lib/Session/CryptoSessionDataTest.php

Spacing +3 added lines, -3 removed lines

@@ -24,12 +24,12 @@
 		$this->crypto = $this->createMock(ICrypto::class);
 		$this->crypto->expects($this->any())
 			->method('encrypt')
-			->willReturnCallback(function ($input) {
-				return '#' . $input . '#';
+			->willReturnCallback(function($input) {
+				return '#'.$input.'#';
 			});
 		$this->crypto->expects($this->any())
 			->method('decrypt')
-			->willReturnCallback(function ($input) {
+			->willReturnCallback(function($input) {
 				if ($input === '') {
 					return '';
 				}

tests/lib/Session/CryptoWrappingTest.php

Spacing +3 added lines, -3 removed lines

@@ -32,12 +32,12 @@ 
 			->getMock();
 		$this->crypto->expects($this->any())
 			->method('encrypt')
-			->willReturnCallback(function ($input) {
+			->willReturnCallback(function($input) {
 				return $input;
 			});
 		$this->crypto->expects($this->any())
 			->method('decrypt')
-			->willReturnCallback(function ($input) {
+			->willReturnCallback(function($input) {
 				if ($input === '') {
 					return '';
 				}
@@ -54,7 +54,7 @@ 
 		$this->wrappedSession->expects($this->once())
 			->method('get')
 			->with('encrypted_session_data')
-			->willReturnCallback(function () use ($encryptedValue) {
+			->willReturnCallback(function() use ($encryptedValue) {
 				return $encryptedValue;
 			});