nextcloud / server

tests/lib/Share20/DefaultShareProviderTest.php

Spacing +32 added lines, -32 removed lines

@@ -717,7 +717,7 @@ 
 		$share2 = $this->provider->create($share);
 
 		$this->assertNotNull($share2->getId());
-		$this->assertSame('ocinternal:' . $share2->getId(), $share2->getFullId());
+		$this->assertSame('ocinternal:'.$share2->getId(), $share2->getFullId());
 		$this->assertSame(IShare::TYPE_USER, $share2->getShareType());
 		$this->assertSame('sharedWith', $share2->getSharedWith());
 		$this->assertSame('sharedBy', $share2->getSharedBy());
@@ -787,7 +787,7 @@ 
 		$share2 = $this->provider->create($share);
 
 		$this->assertNotNull($share2->getId());
-		$this->assertSame('ocinternal:' . $share2->getId(), $share2->getFullId());
+		$this->assertSame('ocinternal:'.$share2->getId(), $share2->getFullId());
 		$this->assertSame(IShare::TYPE_GROUP, $share2->getShareType());
 		$this->assertSame('sharedWith', $share2->getSharedWith());
 		$this->assertSame('sharedBy', $share2->getSharedBy());
@@ -856,7 +856,7 @@ 
 		$share2 = $this->provider->create($share);
 
 		$this->assertNotNull($share2->getId());
-		$this->assertSame('ocinternal:' . $share2->getId(), $share2->getFullId());
+		$this->assertSame('ocinternal:'.$share2->getId(), $share2->getFullId());
 		$this->assertSame(IShare::TYPE_LINK, $share2->getShareType());
 		$this->assertSame('sharedBy', $share2->getSharedBy());
 		$this->assertSame('shareOwner', $share2->getShareOwner());
@@ -1068,7 +1068,7 @@ 
 
 		$groups = [];
 		foreach (range(0, 100) as $i) {
-			$groups[] = 'group' . $i;
+			$groups[] = 'group'.$i;
 		}
 
 		$groups[] = 'sharedWith';
@@ -1187,7 +1187,7 @@ 
 		$this->assertCount(1, $share);
 
 		$share = $share[0];
-		$this->assertSame((string)$id, $share->getId());
+		$this->assertSame((string) $id, $share->getId());
 		$this->assertSame('sharedWith', $share->getSharedWith());
 		$this->assertSame('shareOwner', $share->getShareOwner());
 		$this->assertSame('sharedBy', $share->getSharedBy());
@@ -1322,7 +1322,7 @@ 
 
 		$groups = [];
 		foreach (range(0, 100) as $i) {
-			$groups[] = 'group' . $i;
+			$groups[] = 'group'.$i;
 		}
 
 		$groups[] = 'sharedWith';
@@ -1855,13 +1855,13 @@ 
 		$users = [];
 		for ($i = 0; $i < 6; $i++) {
 			$user = $this->createMock(IUser::class);
-			$user->method('getUID')->willReturn('user' . $i);
-			$user->method('getDisplayName')->willReturn('user' . $i);
-			$users['user' . $i] = $user;
+			$user->method('getUID')->willReturn('user'.$i);
+			$user->method('getDisplayName')->willReturn('user'.$i);
+			$users['user'.$i] = $user;
 		}
 
 		$this->userManager->method('get')->willReturnCallback(
-			function ($userId) use ($users) {
+			function($userId) use ($users) {
 				return $users[$userId];
 			}
 		);
@@ -1913,12 +1913,12 @@ 
 		$users = [];
 		for ($i = 0; $i < 6; $i++) {
 			$user = $this->createMock(IUser::class);
-			$user->method('getUID')->willReturn('user' . $i);
-			$users['user' . $i] = $user;
+			$user->method('getUID')->willReturn('user'.$i);
+			$users['user'.$i] = $user;
 		}
 
 		$this->userManager->method('get')->willReturnCallback(
-			function ($userId) use ($users) {
+			function($userId) use ($users) {
 				return $users[$userId];
 			}
 		);
@@ -1979,12 +1979,12 @@ 
 		$users = [];
 		for ($i = 0; $i < 6; $i++) {
 			$user = $this->createMock(IUser::class);
-			$user->method('getUID')->willReturn('user' . $i);
-			$users['user' . $i] = $user;
+			$user->method('getUID')->willReturn('user'.$i);
+			$users['user'.$i] = $user;
 		}
 
 		$this->userManager->method('get')->willReturnCallback(
-			function ($userId) use ($users) {
+			function($userId) use ($users) {
 				return $users[$userId];
 			}
 		);
@@ -2036,12 +2036,12 @@ 
 		$users = [];
 		for ($i = 0; $i < 6; $i++) {
 			$user = $this->createMock(IUser::class);
-			$user->method('getUID')->willReturn('user' . $i);
-			$users['user' . $i] = $user;
+			$user->method('getUID')->willReturn('user'.$i);
+			$users['user'.$i] = $user;
 		}
 
 		$this->userManager->method('get')->willReturnCallback(
-			function ($userId) use ($users) {
+			function($userId) use ($users) {
 				return $users[$userId];
 			}
 		);
@@ -2049,13 +2049,13 @@ 
 		$groups = [];
 		for ($i = 0; $i < 2; $i++) {
 			$group = $this->createMock(IGroup::class);
-			$group->method('getGID')->willReturn('group' . $i);
-			$group->method('getDisplayName')->willReturn('group-displayname' . $i);
-			$groups['group' . $i] = $group;
+			$group->method('getGID')->willReturn('group'.$i);
+			$group->method('getDisplayName')->willReturn('group-displayname'.$i);
+			$groups['group'.$i] = $group;
 		}
 
 		$this->groupManager->method('get')->willReturnCallback(
-			function ($groupId) use ($groups) {
+			function($groupId) use ($groups) {
 				return $groups[$groupId];
 			}
 		);
@@ -2115,12 +2115,12 @@ 
 		$users = [];
 		for ($i = 0; $i < 6; $i++) {
 			$user = $this->createMock(IUser::class);
-			$user->method('getUID')->willReturn('user' . $i);
-			$users['user' . $i] = $user;
+			$user->method('getUID')->willReturn('user'.$i);
+			$users['user'.$i] = $user;
 		}
 
 		$this->userManager->method('get')->willReturnCallback(
-			function ($userId) use ($users) {
+			function($userId) use ($users) {
 				return $users[$userId];
 			}
 		);
@@ -2128,13 +2128,13 @@ 
 		$groups = [];
 		for ($i = 0; $i < 2; $i++) {
 			$group = $this->createMock(IGroup::class);
-			$group->method('getGID')->willReturn('group' . $i);
-			$group->method('getDisplayName')->willReturn('group-displayname' . $i);
-			$groups['group' . $i] = $group;
+			$group->method('getGID')->willReturn('group'.$i);
+			$group->method('getDisplayName')->willReturn('group-displayname'.$i);
+			$groups['group'.$i] = $group;
 		}
 
 		$this->groupManager->method('get')->willReturnCallback(
-			function ($groupId) use ($groups) {
+			function($groupId) use ($groups) {
 				return $groups[$groupId];
 			}
 		);
@@ -2192,12 +2192,12 @@ 
 		$this->assertSame('user0', $shares[0]['share_with']);
 		$this->assertSame('user4', $shares[0]['uid_initiator']);
 		$this->assertSame('user5', $shares[0]['uid_owner']);
-		$this->assertSame(1, (int)$shares[0]['permissions']);
+		$this->assertSame(1, (int) $shares[0]['permissions']);
 
 		$this->assertSame('user3', $shares[1]['share_with']);
 		$this->assertSame('user4', $shares[1]['uid_initiator']);
 		$this->assertSame('user5', $shares[1]['uid_owner']);
-		$this->assertSame(0, (int)$shares[1]['permissions']);
+		$this->assertSame(0, (int) $shares[1]['permissions']);
 
 
 		$stmt->closeCursor();

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');
 	}
 }