nextcloud / server

tests/lib/LargeFileHelperTest.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 	public function testFormatUnsignedIntegerFloat(): void {
 		$this->assertSame(
 			'9007199254740992',
-			$this->helper->formatUnsignedInteger((float)9007199254740992)
+			$this->helper->formatUnsignedInteger((float) 9007199254740992)
 		);
 	}
 

tests/lib/Authentication/TwoFactorAuth/ProviderLoaderTest.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
 	public function testGetProviders(): void {
 		$provider = $this->createMock(IProvider::class);
 		$provider->method('getId')->willReturn('test');
-		\OC::$server->registerService('\\OCA\\TwoFactorTest\\Provider', function () use ($provider) {
+		\OC::$server->registerService('\\OCA\\TwoFactorTest\\Provider', function() use ($provider) {
 			return $provider;
 		});
 		$this->appManager->expects($this->once())
@@ -97,7 +97,7 @@ 
 		$provider = $this->createMock(IProvider::class);
 		$provider->method('getId')->willReturn('test');
 
-		\OC::$server->registerService('\\OCA\\TwoFactorTest\\Provider', function () use ($provider) {
+		\OC::$server->registerService('\\OCA\\TwoFactorTest\\Provider', function() use ($provider) {
 			return $provider;
 		});
 

tests/lib/Authentication/TwoFactorAuth/MandatoryTwoFactorTest.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
 				['twofactor_enforced_excluded_groups', [], []],
 			]);
 		$this->groupManager->method('isInGroup')
-			->willReturnCallback(function ($user, $group) {
+			->willReturnCallback(function($user, $group) {
 				return $user === 'user123' && $group === 'twofactorers';
 			});
 
@@ -129,7 +129,7 @@ 
 				['twofactor_enforced_excluded_groups', [], ['yoloers']],
 			]);
 		$this->groupManager->method('isInGroup')
-			->willReturnCallback(function ($user, $group) {
+			->willReturnCallback(function($user, $group) {
 				return $user === 'user123' && $group === 'yoloers';
 			});
 

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
 			],
 		];